September 2015 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: webadmin: modifying network_not_sync.png
by alkaplan＠redhat.com 21 Sep '15

21 Sep '15

Alona Kaplan has submitted this change and it was merged. Change subject: webadmin: modifying network_not_sync.png ...................................................................... webadmin: modifying network_not_sync.png Change-Id: If67893e6aa3d2437a31631ccc0573c2756b853d7 Signed-off-by: Eliraz Levi <elevi(a)redhat.com> Signed-off-by: Alona Kaplan <alkaplan(a)redhat.com> --- M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/webadmin/images/network/network_not_sync.png 1 file changed, 0 insertions(+), 0 deletions(-) Approvals: Alona Kaplan: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/44203 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If67893e6aa3d2437a31631ccc0573c2756b853d7 Gerrit-PatchSet: 14 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Eliraz Levi <elevi(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Eliraz Levi <elevi(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: webadmin: modifying out of sync icon position
by alkaplan＠redhat.com 21 Sep '15

21 Sep '15

Alona Kaplan has submitted this change and it was merged. Change subject: webadmin: modifying out of sync icon position ...................................................................... webadmin: modifying out of sync icon position Host setup network popup, moving the out of sync icon from roles area to statuses area Bug-Url: https://bugzilla.redhat.com/1248647 Change-Id: I412187575ca16503b460351259ff1935f3af27cb Signed-off-by: Eliraz Levi <elevi(a)redhat.com> Signed-off-by: Alona Kaplan <alkaplan(a)redhat.com> --- M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/HostSetupNetworksPopupView.ui.xml M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/panels/NetworkPanel.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/panels/NetworkPanelsStyle.java 3 files changed, 12 insertions(+), 2 deletions(-) Approvals: Alona Kaplan: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/44197 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I412187575ca16503b460351259ff1935f3af27cb Gerrit-PatchSet: 14 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Eliraz Levi <elevi(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Eliraz Levi <elevi(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: webadmin: adding out of sync indication
by alkaplan＠redhat.com 21 Sep '15

21 Sep '15

Alona Kaplan has submitted this change and it was merged. Change subject: webadmin: adding out of sync indication ...................................................................... webadmin: adding out of sync indication Bug-Url: https://bugzilla.redhat.com/1058675 Change-Id: I202527cce11cee02756c63ea338bbd888402381f Signed-off-by: Eliraz Levi <elevi(a)redhat.com> Signed-off-by: Alona Kaplan <alkaplan(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostSetupNetworksModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/network/LogicalNetworkModel.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/ApplicationConstants.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/panels/ItemInfoPopup.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/panels/NetworkItemPanel.java A frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/panels/NetworkOutOfSyncPanel.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/panels/NetworkPanel.java 7 files changed, 268 insertions(+), 27 deletions(-) Approvals: Alona Kaplan: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/42396 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I202527cce11cee02756c63ea338bbd888402381f Gerrit-PatchSet: 20 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Eliraz Levi <elevi(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Einav Cohen <ecohen(a)redhat.com> Gerrit-Reviewer: Eliraz Levi <elevi(a)redhat.com> Gerrit-Reviewer: Greg Sheremeta <gshereme(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in jenkins[master]: Add standard ci jobs for lago
by dcaroest＠redhat.com 21 Sep '15

21 Sep '15

David Caro has submitted this change and it was merged. Change subject: Add standard ci jobs for lago ...................................................................... Add standard ci jobs for lago This includes the node restrictions needed by the functional tests Change-Id: I0e6b609d8678c51ac0e8b6748665d64a6922b546 Signed-off-by: Dima Kuznetsov <dkuznets(a)redhat.com> Signed-off-by: David Caro <dcaroest(a)redhat.com> --- A jobs/confs/projects/lago/lago_standard.yaml A jobs/confs/yaml/scms/lago.yaml 2 files changed, 27 insertions(+), 0 deletions(-) Approvals: Barak Korren: Looks good to me Jenkins CI: Passed CI tests David Caro: Verified; Ready for merge -- To view, visit https://gerrit.ovirt.org/41396 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0e6b609d8678c51ac0e8b6748665d64a6922b546 Gerrit-PatchSet: 11 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Dima Kuznetsov <dmitrykuzn(a)gmail.com> Gerrit-Reviewer: Anatoly Litovsky <tlitovsk(a)redhat.com> Gerrit-Reviewer: Anton Marchukov <amarchuk(a)redhat.com> Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Dima Kuznetsov <dmitrykuzn(a)gmail.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Max Kovgan <mvk(a)redhat.com> Gerrit-Reviewer: Paz Dangur <pdangur(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-engine[ovirt-engine-3.6]: core: add missing audit log on migration failure
by tnisan＠redhat.com 21 Sep '15

21 Sep '15

Tal Nisan has submitted this change and it was merged. Change subject: core: add missing audit log on migration failure ...................................................................... core: add missing audit log on migration failure In the following scenario: 1. Migrate VM 2. VM fails to migrate to the first host 3. There are no other hosts to migrate to Assuming the number of retries for a VM migration is not set to 1, the audit log VM_MIGRATION_FAILED was not generated but VM_MIGRATION_FAILED_NO_VDS_TO_RUN_ON was generated instead. This patch fixes it by changing the audit log VM_MIGRATION_FAILED_NO_VDS_TO_RUN_ON to be a warning that only specifies that no host was found and after it VM_MIGRATION_FAILED is generated to summarize that the migration failed. Change-Id: I8aaa2004102269fed78eb737a0c1806821e711ef Bug-Url: https://bugzilla.redhat.com/1193025 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/MigrateVmCommand.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/AuditLogType.java M backend/manager/modules/dal/src/main/resources/bundles/AuditLogMessages.properties 3 files changed, 4 insertions(+), 4 deletions(-) Approvals: Jenkins CI: Passed CI tests Omer Frenkel: Looks good to me, approved Arik Hadas: Verified -- To view, visit https://gerrit.ovirt.org/46421 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8aaa2004102269fed78eb737a0c1806821e711ef Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: packaging: spec: fix jdk requirements on CentOS
by sbonazzo＠redhat.com 21 Sep '15

21 Sep '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: spec: fix jdk requirements on CentOS ...................................................................... packaging: spec: fix jdk requirements on CentOS CentOS 7.1 doesn't provide java-devel >= 1.8. Requiring java-1.8.0-openjdk-devel >= 1:1.8.0. Also requiring java-openjdk >= 1:1.8.0 instead of 1.8.0 avoiding 1:1.7.0 to satisfy 0:1.8.0. Change-Id: I9b3802c1863d57e60d85eeec8c1d4a0363adb6ba Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M ovirt-engine.spec.in 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Juan Hernandez: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/46446 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9b3802c1863d57e60d85eeec8c1d4a0363adb6ba Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-hosted-engine-setup[ovirt-hosted-engine-setup-1.3]: packaging: setup: remove connectStoragePool on iSCSI additio...
by sbonazzo＠redhat.com 21 Sep '15

21 Sep '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: remove connectStoragePool on iSCSI additional hosts ...................................................................... packaging: setup: remove connectStoragePool on iSCSI additional hosts On hosted-engine from oVirt 3.6 the hosted-engine storage domain is not anymore connected to the bootstrap storage pool so the spUUID is now blanck on additional hosts. Removing initial connectStoragePool on additional iSCSI/FC hosts, adding a getStorageDomainStats call to rely on a side effect of that and get /rhev/data-center/... populated. Change-Id: I3bec0e6ad5489a90ebfad198f7934cf776562f6a Bug-Url: https://bugzilla.redhat.com/1247942 Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M src/plugins/ovirt-hosted-engine-setup/storage/storage.py 1 file changed, 20 insertions(+), 4 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Simone Tiraboschi: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/46450 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3bec0e6ad5489a90ebfad198f7934cf776562f6a Gerrit-PatchSet: 2 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: ovirt-hosted-engine-setup-1.3 Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-hosted-engine-setup[master]: packaging: setup: remove connectStoragePool on iSCSI additio...
by sbonazzo＠redhat.com 21 Sep '15

21 Sep '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: remove connectStoragePool on iSCSI additional hosts ...................................................................... packaging: setup: remove connectStoragePool on iSCSI additional hosts On hosted-engine from oVirt 3.6 the hosted-engine storage domain is not anymore connected to the bootstrap storage pool so the spUUID is now blanck on additional hosts. Removing initial connectStoragePool on additional iSCSI/FC hosts, adding a getStorageDomainStats call to rely on a side effect of that and get /rhev/data-center/... populated. Change-Id: I3bec0e6ad5489a90ebfad198f7934cf776562f6a Bug-Url: https://bugzilla.redhat.com/1247942 Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M src/plugins/ovirt-hosted-engine-setup/storage/storage.py 1 file changed, 20 insertions(+), 4 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Simone Tiraboschi: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/44531 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3bec0e6ad5489a90ebfad198f7934cf776562f6a Gerrit-PatchSet: 7 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: master Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Nir Soffer <nsoffer(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.6]: core: Adjust websockify to the new 0.6.0 API
by tnisan＠redhat.com 21 Sep '15

21 Sep '15

Tal Nisan has submitted this change and it was merged. Change subject: core: Adjust websockify to the new 0.6.0 API ...................................................................... core: Adjust websockify to the new 0.6.0 API python-websockify v0.6.0 changed the client API and this patch addresses these things. Upgrading to websockify 0.6.0 improves 2 things: 1, Logging - library uses standard python logging facilities. 2, Killing zombies - library takes care of killling zombie processes (we had to handle it ourselves). Change-Id: I13f94f91a1bdee0bfa93dc263de7e51395f7d7ce Signed-off-by: Frantisek Kobzik <fkobzik(a)redhat.com> Signed-off-by: Vinzenz Feenstra <vfeenstr(a)redhat.com> Backport-To: 3.6 Bug-Url: https://bugzilla.redhat.com/1092611 Bug-Url: https://bugzilla.redhat.com/1098574 --- M ovirt-engine.spec.in M packaging/services/ovirt-websocket-proxy/ovirt-websocket-proxy.py 2 files changed, 59 insertions(+), 73 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Jenkins CI: Passed CI tests Vinzenz Feenstra: Verified -- To view, visit https://gerrit.ovirt.org/46440 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I13f94f91a1bdee0bfa93dc263de7e51395f7d7ce Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Vinzenz Feenstra <vfeenstr(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.6]: webadmin: Add event for host low memory
by tnisan＠redhat.com 21 Sep '15

21 Sep '15

Tal Nisan has submitted this change and it was merged. Change subject: webadmin: Add event for host low memory ...................................................................... webadmin: Add event for host low memory Change-Id: I0ed2b9ca9e93a3d92a147a3552ee911842d01baf Bug-Url: https://bugzilla.redhat.com/1264109 Signed-off-by: Moti Asayag <masayag(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/VdcEventNotificationUtils.java M frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/LocalizedEnums.java M frontend/webadmin/modules/uicompat/src/main/resources/org/ovirt/engine/ui/uicompat/LocalizedEnums.properties 3 files changed, 4 insertions(+), 0 deletions(-) Approvals: Jenkins CI: Passed CI tests Moti Asayag: Verified Oved Ourfali: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/46425 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0ed2b9ca9e93a3d92a147a3552ee911842d01baf Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0