- Engine-commits - oVirt List Archives

Change in ovirt-node-plugin-vdsm[ovirt-3.5]: Add 03-vdsm-sebool-config handler
by fabiand＠redhat.com 23 Oct '14

23 Oct '14

Fabian Deutsch has submitted this change and it was merged. Change subject: Add 03-vdsm-sebool-config handler ...................................................................... Add 03-vdsm-sebool-config handler Due to some changes the booleans can not be set during the build of Node anymore. The now added hook handler ensures that the relevant booleans are opened when Node is booted. Change-Id: Idc95892994ee8760017b5dbd764142dad39a4e8b Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1156038 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1150243 Signed-off-by: Fabian Deutsch <fabiand(a)fedoraproject.org> --- M configure.ac M hooks/Makefile.am A hooks/on-boot/03-vdsm-sebool-config M ovirt-node-plugin-vdsm.spec.in 4 files changed, 21 insertions(+), 1 deletion(-) Approvals: Fabian Deutsch: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/34401 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Idc95892994ee8760017b5dbd764142dad39a4e8b Gerrit-PatchSet: 2 Gerrit-Project: ovirt-node-plugin-vdsm Gerrit-Branch: ovirt-3.5 Gerrit-Owner: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-node-plugin-vdsm[master]: Add 03-vdsm-sebool-config handler
by fabiand＠redhat.com 23 Oct '14

23 Oct '14

Fabian Deutsch has submitted this change and it was merged. Change subject: Add 03-vdsm-sebool-config handler ...................................................................... Add 03-vdsm-sebool-config handler Due to some changes the booleans can not be set during the build of Node anymore. The now added hook handler ensures that the relevant booleans are opened when Node is booted. Change-Id: I6284c467dd7a706961be66d8cb34e4f55b475e26 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1156038 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1150243 Signed-off-by: Fabian Deutsch <fabiand(a)fedoraproject.org> --- M configure.ac M hooks/Makefile.am A hooks/on-boot/03-vdsm-sebool-config M ovirt-node-plugin-vdsm.spec.in 4 files changed, 21 insertions(+), 1 deletion(-) Approvals: Fabian Deutsch: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/34400 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6284c467dd7a706961be66d8cb34e4f55b475e26 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-node-plugin-vdsm Gerrit-Branch: master Gerrit-Owner: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(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[master]: engine: blank template no need default vm_init
by shavivi＠redhat.com 23 Oct '14

23 Oct '14

Shahar Havivi has submitted this change and it was merged. Change subject: engine: blank template no need default vm_init ...................................................................... engine: blank template no need default vm_init By having a default vm_init all inherit VMs are getting blank vm_init as well. Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1155951 Change-Id: Ifa39cad168d523f1a902095b28cde91bf49f14db Signed-off-by: Shahar Havivi <shaharh(a)redhat.com> --- A packaging/dbscripts/upgrade/03_06_0460_remove_vm_init_for_blank_template.sql 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: Shahar Havivi: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/34367 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ifa39cad168d523f1a902095b28cde91bf49f14db Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: webadmin: Events tab message column too narrow
by oourfali＠redhat.com 23 Oct '14

23 Oct '14

Oved Ourfali has submitted this change and it was merged. Change subject: webadmin: Events tab message column too narrow ...................................................................... webadmin: Events tab message column too narrow The message column in events tab is too narrow in basic view. There is lots of space unused and the message column should use available space in basic view. Change-Id: I10cfae5d4fec96a091981b3768648b3a908aa41a Bug-Url: https://bugzilla.redhat.com/1149491 Signed-off-by: Ravi Nori <rnori(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/table/AbstractActionTable.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/MainTabEventView.java 2 files changed, 10 insertions(+), 3 deletions(-) Approvals: Ravi Nori: Verified Lior Vernia: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/34161 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I10cfae5d4fec96a091981b3768648b3a908aa41a Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Einav Cohen <ecohen(a)redhat.com> Gerrit-Reviewer: Lior Vernia <lvernia(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core, engine, db: Enable suspend, migration and memory snaps...
by ofrenkel＠redhat.com 23 Oct '14

23 Oct '14

Omer Frenkel has submitted this change and it was merged. Change subject: core, engine, db: Enable suspend, migration and memory snapshots in ppc64 ...................................................................... core, engine, db: Enable suspend, migration and memory snapshots in ppc64 This patch enables features that are working in PowerKVM: VM suspend, migration and memory snapshots. Change-Id: Ic4e1bddae604f4266af40f45fab40592907b007a Signed-off-by: Vitor de Lima <vdelima(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.java M packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql 2 files changed, 11 insertions(+), 6 deletions(-) Approvals: Omer Frenkel: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/34375 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic4e1bddae604f4266af40f45fab40592907b007a Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Vitor de Lima <vdelima(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Vitor de Lima <vdelima(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-hosted-engine-setup[ovirt-hosted-engine-setup-1.2]: packaging: setup: clean up answer files writing
by sbonazzo＠redhat.com 23 Oct '14

23 Oct '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: clean up answer files writing ...................................................................... packaging: setup: clean up answer files writing Always write one to /var/lib/ovirt-hosted-engine-setup/answers . If user asked for one with --generate-answer, always write there. Write to /etc only at end of successful setup. Support relative paths for answer files. Bug-Url: https://bugzilla.redhat.com/1151366 Bug-Url: https://bugzilla.redhat.com/1121057 Related-To: https://bugzilla.redhat.com/1025313 Change-Id: Icb28356a0ee38b325c53ec631b21b912e044d2a9 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> (cherry picked from commit fcb1d7f0322c108719cd48146b11d6780c945031) --- M ovirt-hosted-engine-setup.spec.in M src/bin/Makefile.am M src/bin/ovirt-hosted-engine-setup M src/ovirt_hosted_engine_setup/constants.py M src/plugins/ovirt-hosted-engine-setup/core/answerfile.py M src/plugins/ovirt-hosted-engine-setup/core/remote_answerfile.py 6 files changed, 48 insertions(+), 29 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Yedidyah Bar David: Verified -- To view, visit http://gerrit.ovirt.org/34396 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icb28356a0ee38b325c53ec631b21b912e044d2a9 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: ovirt-hosted-engine-setup-1.2 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]: core: RemoveDiskSnapshots - sort non-consecutive images
by derez＠redhat.com 23 Oct '14

23 Oct '14

Daniel Erez has submitted this change and it was merged. Change subject: core: RemoveDiskSnapshots - sort non-consecutive images ...................................................................... core: RemoveDiskSnapshots - sort non-consecutive images RemoveDiskSnapshots -> initTaskHandlers: fix images sorting in case of non consecutive images (e.g. image_index_0, image_index_3, image_index_7). Change-Id: Ia736aa93a76dd8df13bfdcf97587cee82bea6700 Bug-Url: https://bugzilla.redhat.com/1151841 Signed-off-by: Daniel Erez <derez(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RemoveDiskSnapshotsCommand.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/RemoveDiskSnapshotsCommandTest.java 2 files changed, 36 insertions(+), 2 deletions(-) Approvals: Daniel Erez: Verified Maor Lipchuk: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/34380 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia736aa93a76dd8df13bfdcf97587cee82bea6700 Gerrit-PatchSet: 7 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: Fix mapping of US Eastern Standard Time Zone
by ofrenkel＠redhat.com 23 Oct '14

23 Oct '14

Omer Frenkel has submitted this change and it was merged. Change subject: core: Fix mapping of US Eastern Standard Time Zone ...................................................................... core: Fix mapping of US Eastern Standard Time Zone The WindowsJavaTimezoneMapping mapped US East. ST to java time zone GMT+5 which was not adjusted to Daylight Savings Time. Now we map it to America/New_York which takes DST into account. Also fixed another occurence of mapping to raw GMT timezone: SA Eastern Standard Time to America/Buenos_Aires (was GMT+3) but this change was not as consequential since Argentina does not have DST. Although this should be more resilient if they decided to use it in the future. Similar story for Georia -> Asia/Tbilisi. Change-Id: I0d17376caac96d1aae87cc894452d42e161637e4 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1148623 Signed-off-by: Martin Betak <mbetak(a)redhat.com> --- M backend/manager/modules/compat/src/main/java/org/ovirt/engine/core/compat/WindowsJavaTimezoneMapping.java 1 file changed, 3 insertions(+), 3 deletions(-) Approvals: Martin Betak: Verified Omer Frenkel: Looks good to me, approved Michal Skrivanek: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/34369 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0d17376caac96d1aae87cc894452d42e161637e4 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Betak <mbetak(a)redhat.com> Gerrit-Reviewer: Martin Betak <mbetak(a)redhat.com> Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com> Gerrit-Reviewer: Michal Skrivanek <mskrivan(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-hosted-engine-setup[master]: packaging: setup: clean up answer files writing
by sbonazzo＠redhat.com 23 Oct '14

23 Oct '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: clean up answer files writing ...................................................................... packaging: setup: clean up answer files writing Always write one to /var/lib/ovirt-hosted-engine-setup/answers . If user asked for one with --generate-answer, always write there. Write to /etc only at end of successful setup. Support relative paths for answer files. Bug-Url: https://bugzilla.redhat.com/1151366 Bug-Url: https://bugzilla.redhat.com/1121057 Related-To: https://bugzilla.redhat.com/1025313 Change-Id: Icb28356a0ee38b325c53ec631b21b912e044d2a9 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M ovirt-hosted-engine-setup.spec.in M src/bin/Makefile.am M src/bin/ovirt-hosted-engine-setup M src/ovirt_hosted_engine_setup/constants.py M src/plugins/ovirt-hosted-engine-setup/core/answerfile.py M src/plugins/ovirt-hosted-engine-setup/core/remote_answerfile.py 6 files changed, 48 insertions(+), 29 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Yedidyah Bar David: Verified -- To view, visit http://gerrit.ovirt.org/34379 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icb28356a0ee38b325c53ec631b21b912e044d2a9 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-setup 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[ovirt-engine-3.5]: userportal: optimize GetAllFromVms query
by tnisan＠redhat.com 23 Oct '14

23 Oct '14

Tal Nisan has submitted this change and it was merged. Change subject: userportal: optimize GetAllFromVms query ...................................................................... userportal: optimize GetAllFromVms query Rewrote the stored function to optimize the query inside GetAllFromVms Splitting out the boolean for simplicity Joining instead of using exists on a complicated view Removing distinct, as it's already inside the joined view Change-Id: I541c9172c3ba9bcf989f9660baf06ec2e1fd02d7 Bug-Url: https://bugzilla.redhat.com/1151477 Signed-off-by: Tim Speetjens <tim.speetjens(a)redhat.com> --- M packaging/dbscripts/vms_sp.sql 1 file changed, 11 insertions(+), 6 deletions(-) Approvals: Omer Frenkel: Looks good to me, approved Tim Speetjens: Looks good to me, but someone else must approve Oved Ourfali: Verified -- To view, visit http://gerrit.ovirt.org/34387 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I541c9172c3ba9bcf989f9660baf06ec2e1fd02d7 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Tim Speetjens <tim.speetjens(a)gmail.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0