- Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: core: stateless vms monitoring - vmAnalyzers
by ahadas＠redhat.com 28 Jan '16

28 Jan '16

Arik Hadas has submitted this change and it was merged. Change subject: core: stateless vms monitoring - vmAnalyzers ...................................................................... core: stateless vms monitoring - vmAnalyzers As part of the effort to make VmsMonitoring stateless, the analyzers are now stored as a variable instead of holding them as a member of VmsMonitoring. Change-Id: I302073e3910583dac2af78c2cfee87e126eea791 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VmsMonitoring.java 1 file changed, 25 insertions(+), 28 deletions(-) Approvals: Shahar Havivi: Looks good to me, approved Jenkins CI: Passed CI tests Arik Hadas: Verified -- To view, visit https://gerrit.ovirt.org/52812 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I302073e3910583dac2af78c2cfee87e126eea791 Gerrit-PatchSet: 6 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: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-3.6.3]: packaging: spec: Fix semanage requirement in fedora 23
by tnisan＠redhat.com 28 Jan '16

28 Jan '16

Tal Nisan has submitted this change and it was merged. Change subject: packaging: spec: Fix semanage requirement in fedora 23 ...................................................................... packaging: spec: Fix semanage requirement in fedora 23 semanage was packaged in policycoreutils-python until fedora 23, then it moved to policycoreutils-python-utils. Fix the spec file accordingly. Change-Id: I1236ea2f611437634c331d8bc89c74287a7ff0e8 Bug-Url: https://bugzilla.redhat.com/1302035 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> (cherry picked from commit b73ec29a23126016dd6dfde414b9ecab94ab87b4) --- M ovirt-engine.spec.in 1 file changed, 8 insertions(+), 1 deletion(-) Approvals: Sandro Bonazzola: Looks good to me, approved Jenkins CI: Passed CI tests Yedidyah Bar David: Verified -- To view, visit https://gerrit.ovirt.org/52811 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1236ea2f611437634c331d8bc89c74287a7ff0e8 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6.3 Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Jenkins CI 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: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-3.6]: packaging: setup: Do not require ovirt-log-collector
by tnisan＠redhat.com 28 Jan '16

28 Jan '16

Tal Nisan has submitted this change and it was merged. Change subject: packaging: setup: Do not require ovirt-log-collector ...................................................................... packaging: setup: Do not require ovirt-log-collector Change-Id: Iebe2155181199672239dd1063c1a69a4ab4cc3c6 Bug-Url: https://bugzilla.redhat.com/1301902 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M packaging/setup/ovirt_engine_setup/engine/constants.py 1 file changed, 0 insertions(+), 1 deletion(-) Approvals: Sandro Bonazzola: Looks good to me, approved Jenkins CI: Passed CI tests Yedidyah Bar David: Verified -- To view, visit https://gerrit.ovirt.org/52745 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iebe2155181199672239dd1063c1a69a4ab4cc3c6 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Jenkins CI 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: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-live[ovirt-live-3.6]: build: ovirt-live-3.6.3_rc1
by sbonazzo＠redhat.com 28 Jan '16

28 Jan '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: build: ovirt-live-3.6.3_rc1 ...................................................................... build: ovirt-live-3.6.3_rc1 Change-Id: Ia73c10ac46f70447d26cab67acedd76b596bae21 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M centos-7/Makefile M centos-7/kickstart/ovirt-live-ovirt-custom.ks.in 2 files changed, 2 insertions(+), 2 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/52845 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia73c10ac46f70447d26cab67acedd76b596bae21 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-live Gerrit-Branch: ovirt-live-3.6 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: webadmin: fix white space.
by tjelinek＠redhat.com 28 Jan '16

28 Jan '16

Tomas Jelinek has submitted this change and it was merged. Change subject: webadmin: fix white space. ...................................................................... webadmin: fix white space. • reformatted code so that it adheres to code conventions Change-Id: I3d9acf372d9c7bcd1149b732c1901579cdcdc9b4 Signed-off-by: Martin Mucha <mmucha(a)redhat.com> --- M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/macpool/MacPoolWidget.ui.xml 1 file changed, 20 insertions(+), 19 deletions(-) Approvals: Tomas Jelinek: Looks good to me, approved Martin Mucha: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/51632 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3d9acf372d9c7bcd1149b732c1901579cdcdc9b4 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-3.6]: packaging: spec: Fix semanage requirement in fedora 23
by tnisan＠redhat.com 28 Jan '16

28 Jan '16

Tal Nisan has submitted this change and it was merged. Change subject: packaging: spec: Fix semanage requirement in fedora 23 ...................................................................... packaging: spec: Fix semanage requirement in fedora 23 semanage was packaged in policycoreutils-python until fedora 23, then it moved to policycoreutils-python-utils. Fix the spec file accordingly. Change-Id: I1236ea2f611437634c331d8bc89c74287a7ff0e8 Bug-Url: https://bugzilla.redhat.com/1302035 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M ovirt-engine.spec.in 1 file changed, 8 insertions(+), 1 deletion(-) Approvals: Sandro Bonazzola: Looks good to me, approved Jenkins CI: Passed CI tests Yedidyah Bar David: Verified -- To view, visit https://gerrit.ovirt.org/52810 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1236ea2f611437634c331d8bc89c74287a7ff0e8 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Jenkins CI 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: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: webadmin: new pool dialog contained the sub version field
by tjelinek＠redhat.com 28 Jan '16

28 Jan '16

Tomas Jelinek has submitted this change and it was merged. Change subject: webadmin: new pool dialog contained the sub version field ...................................................................... webadmin: new pool dialog contained the sub version field The sub version field is from the template dialog - it should not be present in the new pool dialog. Change-Id: I36188c0e666377ea076caea1966004a47632f338 Bug-Url: https://bugzilla.redhat.com/1302372 Signed-off-by: Tomas Jelinek <tjelinek(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/pool/PoolNewPopupWidget.java 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Tomas Jelinek: Verified; Looks good to me, approved Jakub Niedermertl: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Martin Betak: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/52831 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I36188c0e666377ea076caea1966004a47632f338 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Jakub Niedermertl <jniederm(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Betak <mbetak(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-3.6]: webadmin: Release keys removed from the console title
by tnisan＠redhat.com 28 Jan '16

28 Jan '16

Tal Nisan has submitted this change and it was merged. Change subject: webadmin: Release keys removed from the console title ...................................................................... webadmin: Release keys removed from the console title With this fix, the console title does not contain release keys. This info is displayed by virt-viewer out of the box. Change-Id: I620d55a83ec6f0231fb6eb9f882a10fa98199f9e Bug-Url: https://bugzilla.redhat.com/1295659 Signed-off-by: Marek Libra <mlibra(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/ConsoleModel.java M frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/UIMessages.java 2 files changed, 1 insertion(+), 20 deletions(-) Approvals: Tomas Jelinek: Looks good to me, approved Jakub Niedermertl: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Marek Libra: Verified -- To view, visit https://gerrit.ovirt.org/52807 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I620d55a83ec6f0231fb6eb9f882a10fa98199f9e Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Marek Libra <mlibra(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Jakub Niedermertl <jniederm(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Marek Libra <mlibra(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-3.6]: Use the proper whitelist for MigrateVmCommand.canDoAction
by tnisan＠redhat.com 28 Jan '16

28 Jan '16

Tal Nisan has submitted this change and it was merged. Change subject: Use the proper whitelist for MigrateVmCommand.canDoAction ...................................................................... Use the proper whitelist for MigrateVmCommand.canDoAction The execute methods gets whitelist using the getVdsWhiteList method. But canDoAction used getParameters directly. This caused trouble for MigrateVmToServerCommand which overrides getVdsWhitelist as a different value was used for canDoAction and execute calls. Change-Id: Ieb03d8d7933cca6c87c255d5e846c9c3cb14ee33 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1258960 Signed-off-by: Martin Sivak <msivak(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/MigrateVmCommand.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Martin Sivák: Verified Jenkins CI: Passed CI tests Arik Hadas: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/52800 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ieb03d8d7933cca6c87c255d5e846c9c3cb14ee33 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: dao: Avoid Java 8 constructs in VmDynamicDaoImpl
by ahadas＠redhat.com 27 Jan '16

27 Jan '16

Arik Hadas has submitted this change and it was merged. Change subject: dao: Avoid Java 8 constructs in VmDynamicDaoImpl ...................................................................... dao: Avoid Java 8 constructs in VmDynamicDaoImpl Removing Java 8 constructs from VmDynamicDaoImpl, because they are not currently supported in DAOs and cause DAO tests to fail. Change-Id: I64a396986c98ddf7640737bf4b5b9b25ad84874c Signed-off-by: Shmuel Melamud <smelamud(a)redhat.com> --- M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VmDynamicDaoImpl.java 1 file changed, 15 insertions(+), 4 deletions(-) Approvals: Eli Mesika: Looks good to me, approved Jenkins CI: Passed CI tests Shmuel Leib Melamud: Verified Arik Hadas: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/52809 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I64a396986c98ddf7640737bf4b5b9b25ad84874c Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Shmuel Leib Melamud <smelamud(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Shmuel Leib Melamud <smelamud(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0