- Engine-commits - oVirt List Archives

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

Change in ovirt-engine[ovirt-engine-3.6]: webadmin: don't set watchdog action to null
by tnisan＠redhat.com 21 Sep '15

21 Sep '15

Tal Nisan has submitted this change and it was merged. Change subject: webadmin: don't set watchdog action to null ...................................................................... webadmin: don't set watchdog action to null It is not correct to set it to null - it renders as an empty value which is not correct - better just to disable the field. Change-Id: I2792c7b4b50d402fa6cb9622f5d3174b2653853b Bug-Url: https://bugzilla.redhat.com/1261015 Signed-off-by: Tomas Jelinek <tjelinek(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/UnitVmModel.java 1 file changed, 0 insertions(+), 1 deletion(-) Approvals: Tomas Jelinek: Verified; Looks good to me, approved Tal Nisan: Passed CI tests Jakub Niedermertl: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/46347 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2792c7b4b50d402fa6cb9622f5d3174b2653853b Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Jakub Niedermertl <jniederm(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.6]: webadmin: watchdog action edit template not always filled
by tnisan＠redhat.com 21 Sep '15

21 Sep '15

Tal Nisan has submitted this change and it was merged. Change subject: webadmin: watchdog action edit template not always filled ...................................................................... webadmin: watchdog action edit template not always filled The updateWatchdogItems cleared the selected watchDogModel which caused the watchdog action to be set to null in watchdogModelSelectedItemChanged Change-Id: I000fc7ef8a61be34d67993c405a3bb2ff7136d3b Bug-Url: https://bugzilla.redhat.com/1261015 Signed-off-by: Tomas Jelinek <tjelinek(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/UnitVmModel.java 1 file changed, 5 insertions(+), 1 deletion(-) Approvals: Tomas Jelinek: Verified; Looks good to me, approved Tal Nisan: Passed CI tests Martin Betak: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/46346 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I000fc7ef8a61be34d67993c405a3bb2ff7136d3b Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Betak <mbetak(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.6]: core: Filter out external VMs for OVF_SOTRE disk.
by tnisan＠redhat.com 21 Sep '15

21 Sep '15

Tal Nisan has submitted this change and it was merged. Change subject: core: Filter out external VMs for OVF_SOTRE disk. ...................................................................... core: Filter out external VMs for OVF_SOTRE disk. Filter out external VMs when we store VMs to the OVF_STORE disk in the Storage Domain. External VMs represent processes which run on the Host and does not reflect entities which the user will want to preserve for disaster, therefore there is no need to preserve also external VMs in the OVF_STORE disk. Change-Id: I253bfb974f78d0e7441da020d4bfe46c8aa5ac10 Bug-Url: https://bugzilla.redhat.com/1259467 Signed-off-by: Maor Lipchuk <mlipchuk(a)redhat.com> --- M packaging/dbscripts/vms_sp.sql 1 file changed, 5 insertions(+), 3 deletions(-) Approvals: Jenkins CI: Passed CI tests Allon Mureinik: Looks good to me, approved Maor Lipchuk: Verified -- To view, visit https://gerrit.ovirt.org/46429 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I253bfb974f78d0e7441da020d4bfe46c8aa5ac10 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0