- Engine-commits - oVirt List Archives

Change in ovirt-log-collector[master]: build: ovirt-log-collector-4.0.0-1
by rmartins＠redhat.com 31 May '16

31 May '16

Rafael Martins has submitted this change and it was merged. Change subject: build: ovirt-log-collector-4.0.0-1 ...................................................................... build: ovirt-log-collector-4.0.0-1 Change-Id: I4bb7f5cca090a4b038f25ea584ef3e040618d9b5 Signed-off-by: Rafael Martins <rmartins(a)redhat.com> --- M configure.ac 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Jenkins CI: Passed CI tests Rafael Martins: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/58373 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4bb7f5cca090a4b038f25ea584ef3e040618d9b5 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-log-collector Gerrit-Branch: master Gerrit-Owner: Rafael Martins <rmartins(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Rafael Martins <rmartins(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-iso-uploader[master]: build: post ovirt-iso-uploader-4.0.0-1
by rmartins＠redhat.com 31 May '16

31 May '16

Rafael Martins has submitted this change and it was merged. Change subject: build: post ovirt-iso-uploader-4.0.0-1 ...................................................................... build: post ovirt-iso-uploader-4.0.0-1 Change-Id: I1580167c569f16de9ecd18578f417d1221e24323 Signed-off-by: Rafael Martins <rmartins(a)redhat.com> --- M configure.ac 1 file changed, 3 insertions(+), 3 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Rafael Martins: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/58374 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1580167c569f16de9ecd18578f417d1221e24323 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-iso-uploader Gerrit-Branch: master Gerrit-Owner: Rafael Martins <rmartins(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Rafael Martins <rmartins(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-hosted-engine-ha[ovirt-hosted-engine-ha-1.3]: build: post - ovirt-hosted-engine-ha-1.3.5.7
by sbonazzo＠redhat.com 31 May '16

31 May '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: build: post - ovirt-hosted-engine-ha-1.3.5.7 ...................................................................... build: post - ovirt-hosted-engine-ha-1.3.5.7 Change-Id: Iae3b6f4542de60b61f21fb2ffdd797292f8ab17f Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M configure.ac M ovirt-hosted-engine-ha.spec.in 2 files changed, 9 insertions(+), 6 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Simone Tiraboschi: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/58372 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iae3b6f4542de60b61f21fb2ffdd797292f8ab17f Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: ovirt-hosted-engine-ha-1.3 Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-hosted-engine-ha[ovirt-hosted-engine-ha-1.3]: build: ovirt-hosted-engine-ha-1.3.5.7
by sbonazzo＠redhat.com 31 May '16

31 May '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: build: ovirt-hosted-engine-ha-1.3.5.7 ...................................................................... build: ovirt-hosted-engine-ha-1.3.5.7 Change-Id: I9cced9d92821a447bf0091cd147ed1f3c12feb5a Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M configure.ac M ovirt-hosted-engine-ha.spec.in 2 files changed, 11 insertions(+), 8 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Simone Tiraboschi: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/58371 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9cced9d92821a447bf0091cd147ed1f3c12feb5a Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: ovirt-hosted-engine-ha-1.3 Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-iso-uploader[master]: build: ovirt-iso-uploader-4.0.0-1
by rmartins＠redhat.com 31 May '16

31 May '16

Rafael Martins has submitted this change and it was merged. Change subject: build: ovirt-iso-uploader-4.0.0-1 ...................................................................... build: ovirt-iso-uploader-4.0.0-1 Change-Id: I88c751060cf43bff350913f9b36ca8f384372fc4 Signed-off-by: Rafael Martins <rmartins(a)redhat.com> --- M configure.ac M ovirt-iso-uploader.spec.in 2 files changed, 5 insertions(+), 2 deletions(-) Approvals: Jenkins CI: Passed CI tests Rafael Martins: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/58347 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I88c751060cf43bff350913f9b36ca8f384372fc4 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-iso-uploader Gerrit-Branch: master Gerrit-Owner: Rafael Martins <rmartins(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Rafael Martins <rmartins(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: fix inconsistency with empty cd-rom value
by ahadas＠redhat.com 31 May '16

31 May '16

Arik Hadas has submitted this change and it was merged. Change subject: core: fix inconsistency with empty cd-rom value ...................................................................... core: fix inconsistency with empty cd-rom value By default, the CD-ROM attached to the VM is NULL. VDSM also reports NULL value in case no CD-ROM is attached. Recently RunVmCommand was changed to persist the CD-ROM that is attached to a VM as well, however, it is then set to an empty string if no CD-ROM is attached. Therefore, the monitoring gets empty string from the DB and NULL value from VDSM and decides to update VM dynamic data because of this change. But later on, the NULL value from VDSM is ignored and thus, this flow will happen on every monitoring cycle, leading to many redundant updates of VM dynamic daa. Change-Id: I5e2afff36f3ee02b57a43760ab88f0a9c3176f37 Bug-Url: https://bugzilla.redhat.com/1340722 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RunVmCommand.java A packaging/dbscripts/upgrade/04_00_0740_empty_current_cd_to_null.sql 2 files changed, 3 insertions(+), 0 deletions(-) Approvals: Jenkins CI: Passed CI tests Martin Betak: Looks good to me, but someone else must approve Arik Hadas: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/58304 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5e2afff36f3ee02b57a43760ab88f0a9c3176f37 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Betak <mbetak(a)redhat.com> Gerrit-Reviewer: Michal Skrivanek <mskrivan(a)redhat.com> Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Shmuel Leib Melamud <smelamud(a)redhat.com> Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: webadmin: EntityModelWidgetWithInfo labelPanel floats left
by tjelinek＠redhat.com 31 May '16

31 May '16

Tomas Jelinek has submitted this change and it was merged. Change subject: webadmin: EntityModelWidgetWithInfo labelPanel floats left ...................................................................... webadmin: EntityModelWidgetWithInfo labelPanel floats left ... to prevent it from spanning over whole horizontal space which caused contentWidgetContainer to be vertically shifted below label. The problem was reported in Import VMs dialog > VmWare > vCenter and Import VMs dialog > XEN (via RHEL) > URI + 'Proxy Host'. Change-Id: Ie9e3933f70c1a2fea5313ed24ec16e68214f3033 Bug-Url: https://bugzilla.redhat.com/1338723 Signed-off-by: Jakub Niedermertl <jniederm(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/EntityModelWidgetWithInfo.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/EntityModelWidgetWithInfo.ui.xml 2 files changed, 17 insertions(+), 2 deletions(-) Approvals: Tomas Jelinek: Looks good to me, approved Jakub Niedermertl: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/58293 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie9e3933f70c1a2fea5313ed24ec16e68214f3033 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Jakub Niedermertl <jniederm(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Jakub Niedermertl <jniederm(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-hosted-engine-ha[ovirt-hosted-engine-ha-1.3]: Add video device if not present in the ovf store
by msivak＠redhat.com 31 May '16

31 May '16

Martin Sivák has submitted this change and it was merged. Change subject: Add video device if not present in the ovf store ...................................................................... Add video device if not present in the ovf store Because the video device was not always reported correct in the past there are ways to import the HE VM into ovirt-engine without the video device. This means that it will not appear in the ovf store and therefore we would not ask for it on the next VM restart. Here a fallback video device configuration is provided in that case. This is only 3.6 related. On 4.0 all involved components are fixed. Change-Id: I8b412e2d93ba6269743257e93342ceaa47eb2299 Related-To: https://bugzilla.redhat.com/1326810 Related-To: https://bugzilla.redhat.com/1328921 Bug-Url: https://bugzilla.redhat.com/1339305 Signed-off-by: Roman Mohr <rmohr(a)redhat.com> --- M ovirt_hosted_engine_ha/lib/ovf/ovf2VmParams.py 1 file changed, 24 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, but someone else must approve Martin Sivák: Looks good to me, approved Roman Mohr: Verified Jenkins CI: Passed CI tests Roy Golan: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/58239 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8b412e2d93ba6269743257e93342ceaa47eb2299 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: ovirt-hosted-engine-ha-1.3 Gerrit-Owner: Roman Mohr <rmohr(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Roman Mohr <rmohr(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-hosted-engine-ha[ovirt-hosted-engine-ha-1.3]: Extract video device from OVF store
by msivak＠redhat.com 31 May '16

31 May '16

Martin Sivák has submitted this change and it was merged. Change subject: Extract video device from OVF store ...................................................................... Extract video device from OVF store To stay in sync with ovirt-engine, also extract video devices from the OVF store. Change-Id: I72f42a299578975a6a1a838ee65eff15586264a9 Related-To: https://bugzilla.redhat.com/1326810 Related-To: https://bugzilla.redhat.com/1328921 Bug-Url: https://bugzilla.redhat.com/1339305 Signed-off-by: Roman Mohr <rmohr(a)redhat.com> --- M ovirt_hosted_engine_ha/lib/ovf/ovf2VmParams.py M ovirt_hosted_engine_ha/lib/ovf/ovf2VmParams_test.py M ovirt_hosted_engine_ha/lib/ovf/ovf_test.xml 3 files changed, 29 insertions(+), 0 deletions(-) Approvals: Martin Sivák: Looks good to me, approved Simone Tiraboschi: Looks good to me, but someone else must approve Roman Mohr: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/57887 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I72f42a299578975a6a1a838ee65eff15586264a9 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: ovirt-hosted-engine-ha-1.3 Gerrit-Owner: Roman Mohr <rmohr(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Michal Skrivanek <mskrivan(a)redhat.com> Gerrit-Reviewer: Roman Mohr <rmohr(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-3.6]: aaa: fix group's permissions removal
by piotr.kliczewski＠gmail.com 31 May '16

31 May '16

Piotr Kliczewski has submitted this change and it was merged. Change subject: aaa: fix group's permissions removal ...................................................................... aaa: fix group's permissions removal Commit - Ica8610e4cf613629c0fbc0a8d7414598a8bbd899 introduced regression which disabled removal of permission assigned to a group. This patch fixes it. Change-Id: I621dcf89e71062c7bc5b315f085135805d39f1b9 Bug-Url: https://bugzilla.redhat.com/1340932 Signed-off-by: Ondra Machacek <omachace(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RemovePermissionCommand.java 1 file changed, 2 insertions(+), 1 deletion(-) Approvals: Piotr Kliczewski: Passed CI tests Ondra Machacek: Verified Martin Peřina: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/58357 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I621dcf89e71062c7bc5b315f085135805d39f1b9 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0