November 2014 - Engine-commits - oVirt List Archives

Change in mom[master]: Add missing log for getStatistics in XMLRPC interface
by alitke＠redhat.com 10 Nov '14

10 Nov '14

Adam Litke has submitted this change and it was merged. Change subject: Add missing log for getStatistics in XMLRPC interface ...................................................................... Add missing log for getStatistics in XMLRPC interface All methods exported to XMLRPC interface has own info log, when is invoked. Only getStatistics was missing this log info. Change-Id: I0021fae11cb9c9efcd2c59e567ec3263a3693d21 Signed-off-by: Martin Pavlasek <mpavlase(a)redhat.com> Reviewed-on: http://gerrit.ovirt.org/34712 Reviewed-by: Adam Litke <alitke(a)redhat.com> --- M mom/MOMFuncs.py 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Adam Litke: Looks good to me, approved Martin Pavlásek: Verified -- To view, visit http://gerrit.ovirt.org/34712 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0021fae11cb9c9efcd2c59e567ec3263a3693d21 Gerrit-PatchSet: 2 Gerrit-Project: mom Gerrit-Branch: master Gerrit-Owner: Martin Pavlásek <mpavlase(a)redhat.com> Gerrit-Reviewer: Adam Litke <alitke(a)redhat.com> Gerrit-Reviewer: Martin Pavlásek <mpavlase(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-dwh[ovirt-engine-dwh-3.5]: packaging: setup: pep8 fixes
by ydary＠redhat.com 10 Nov '14

10 Nov '14

Yaniv Dary has submitted this change and it was merged. Change subject: packaging: setup: pep8 fixes ...................................................................... packaging: setup: pep8 fixes One is minor, other is real - missing import. Affects only certain kinds of failures, therefore not noticed in testing. Bug-Url: https://bugzilla.redhat.com/1162186 Change-Id: I7f4108fd9e1e24e5199e0ed53710dbeb8cc2fa44 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> (cherry picked from commit 72bba548ef8a6f30cbfb6d60b1581779420e2b9a) --- M packaging/setup/plugins/ovirt-engine-common/ovirt-engine-dwh/db/engine_connection.py M packaging/setup/plugins/ovirt-engine-remove/ovirt-engine-dwh/db/single_etl.py 2 files changed, 3 insertions(+), 2 deletions(-) Approvals: Yaniv Dary: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/35001 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7f4108fd9e1e24e5199e0ed53710dbeb8cc2fa44 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-dwh Gerrit-Branch: ovirt-engine-dwh-3.5 Gerrit-Owner: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-dwh[master]: packaging: setup: pep8 fixes
by ydary＠redhat.com 10 Nov '14

10 Nov '14

Yaniv Dary has submitted this change and it was merged. Change subject: packaging: setup: pep8 fixes ...................................................................... packaging: setup: pep8 fixes One is minor, other is real - missing import. Affects only certain kinds of failures, therefore not noticed in testing. Bug-Url: https://bugzilla.redhat.com/1162186 Change-Id: I7f4108fd9e1e24e5199e0ed53710dbeb8cc2fa44 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-common/ovirt-engine-dwh/db/engine_connection.py M packaging/setup/plugins/ovirt-engine-remove/ovirt-engine-dwh/db/single_etl.py 2 files changed, 3 insertions(+), 2 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, but someone else must approve Simone Tiraboschi: Looks good to me, but someone else must approve Yedidyah Bar David: Verified Yaniv Dary: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/34992 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7f4108fd9e1e24e5199e0ed53710dbeb8cc2fa44 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-dwh Gerrit-Branch: master Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-release[master]: bump version / release
by sbonazzo＠redhat.com 10 Nov '14

10 Nov '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: bump version / release ...................................................................... bump version / release bump version release for allowing people to get latest changes including deps for EL7 Change-Id: Ib2361d328d372d024700a1631d6b4e381333ebd3 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M ovirt-release-3.5/configure.ac M ovirt-release-3.5/ovirt-release35.spec.in M ovirt-release-master/configure.ac M ovirt-release-master/ovirt-release-master.spec.in 4 files changed, 10 insertions(+), 4 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Simone Tiraboschi: Looks good to me, but someone else must approve Yedidyah Bar David: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/34996 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib2361d328d372d024700a1631d6b4e381333ebd3 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-release Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-release[master]: packaging: add missing dependencies for engine on el7
by sbonazzo＠redhat.com 10 Nov '14

10 Nov '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: add missing dependencies for engine on el7 ...................................................................... packaging: add missing dependencies for engine on el7 Change-Id: I77367f4540ddfdacf1d65328b7d8f757ed1efc06 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M ovirt-release-3.5/ovirt-el7-deps.repo.in M ovirt-release-master/ovirt-el7-deps.repo.in 2 files changed, 2 insertions(+), 2 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Yedidyah Bar David: Verified -- To view, visit http://gerrit.ovirt.org/34993 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I77367f4540ddfdacf1d65328b7d8f757ed1efc06 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-release Gerrit-Branch: master Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: webadmin: CPU Type and CPU Architecture fields in Cluster di...
by tjelinek＠redhat.com 10 Nov '14

10 Nov '14

Tomas Jelinek has submitted this change and it was merged. Change subject: webadmin: CPU Type and CPU Architecture fields in Cluster dialog fix ...................................................................... webadmin: CPU Type and CPU Architecture fields in Cluster dialog fix * CPU Architecture listbox is placed right above the CPU Type because they are related. (ClusterPopupView.ui.xml) * CPU Architecture field doesn't disappear when CPU Type is selected. (Bug fix). (ClusterPopupView.java) * CPU Architecture filters content of CPU type listbox. (ClusterModel.java, FilteredListModel.java) Bug-Url: https://bugzilla.redhat.com/1159777 Change-Id: I4c0452426f58f09703382495a9527fdbe859d80e Signed-off-by: Jakub Niedermertl <jniederm(a)redhat.com> --- A frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/FilteredListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/clusters/ClusterModel.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/cluster/ClusterPopupView.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/cluster/ClusterPopupView.ui.xml 4 files changed, 123 insertions(+), 17 deletions(-) Approvals: Tomas Jelinek: Looks good to me, approved Jakub Niedermertl: Verified -- To view, visit http://gerrit.ovirt.org/34816 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4c0452426f58f09703382495a9527fdbe859d80e Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Jakub Niedermertl <jniederm(a)redhat.com> Gerrit-Reviewer: Jakub Niedermertl <jniederm(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: Adding host to DB before calling provision
by yzaslavs＠redhat.com 10 Nov '14

10 Nov '14

Yair Zaslavsky has submitted this change and it was merged. Change subject: Adding host to DB before calling provision ...................................................................... Adding host to DB before calling provision When calling provision foreman expect the engine's host-id to allow calling the reinstallation after provisioning. Engine generates host-id while saving host info to db, therefore this patch switches between the DB. Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1158588 Change-Id: I635d87d09eec0dd052d86329d2b6b7f0a1658836 Signed-off-by: Yaniv Bronhaim <ybronhei(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVdsCommand.java 1 file changed, 11 insertions(+), 11 deletions(-) Approvals: Yaniv Bronhaim: Verified Yair Zaslavsky: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/34860 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I635d87d09eec0dd052d86329d2b6b7f0a1658836 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yaniv Bronhaim <ybronhei(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.5]: packaging:[ver2] fail the build if osinfo-defaults.propertie...
by tnisan＠redhat.com 10 Nov '14

10 Nov '14

Tal Nisan has submitted this change and it was merged. Change subject: packaging:[ver2] fail the build if osinfo-defaults.properties fails to parse ...................................................................... packaging:[ver2] fail the build if osinfo-defaults.properties fails to parse check the properties file we ship with the exact code that checks it at runtime when the engine loads. this test is hooked into the packaging process and must fail it. Change-Id: Icaecda7d278fa898838f04547d5ab44e5ca6b3d9 Bug-Url: https://bugzilla.redhat.com/1056965 Signed-off-by: Roy Golan <rgolan(a)redhat.com> --- M build/validations/pom.xml 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Sandro Bonazzola: Looks good to me, approved Roy Golan: Verified -- To view, visit http://gerrit.ovirt.org/34959 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icaecda7d278fa898838f04547d5ab44e5ca6b3d9 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: webadmin, userportal: Remove explicit include for generated s...
by vszocs＠redhat.com 10 Nov '14

10 Nov '14

Vojtech Szocs has submitted this change and it was merged. Change subject: webadmin,userportal: Remove explicit include for generated sources ...................................................................... webadmin,userportal: Remove explicit include for generated sources This patch builds on top of http://gerrit.ovirt.org/#/c/34402/ and ensures that everything under target/generated-sources/gwt directory is added to project's resources. Change-Id: Iaa38ee88c655e6b93c2a53dda34d341c961e83ae Signed-off-by: Vojtech Szocs <vszocs(a)redhat.com> --- M frontend/webadmin/modules/frontend/pom.xml 1 file changed, 0 insertions(+), 3 deletions(-) Approvals: Alexander Wels: Verified; Looks good to me, approved Vojtech Szocs: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/34708 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iaa38ee88c655e6b93c2a53dda34d341c961e83ae Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Einav Cohen <ecohen(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in jenkins[master]: build vdsm-jsonrpc-java 3.5 on el7
by dcaroest＠redhat.com 10 Nov '14

10 Nov '14

David Caro has submitted this change and it was merged. Change subject: build vdsm-jsonrpc-java 3.5 on el7 ...................................................................... build vdsm-jsonrpc-java 3.5 on el7 Change-Id: I81f2ce722a0229af2262ac0e4649b9351badb87e Bug-Url: https://bugzilla.redhat.com/1159293 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M jobs/confs/yaml/jobs/vdsm-jsonrpc-java/vdsm-jsonrpc-java_create-rpms.yaml 1 file changed, 3 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, but someone else must approve Eyal Edri: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/34923 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I81f2ce722a0229af2262ac0e4649b9351badb87e Gerrit-PatchSet: 1 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0