August 2015 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: webadmin: cancel vm conversion
by ahadas＠redhat.com 05 Aug '15

05 Aug '15

Arik Hadas has submitted this change and it was merged. Change subject: webadmin: cancel vm conversion ...................................................................... webadmin: cancel vm conversion Change-Id: I27cdefc477530c9033e4f8cc54e482ff6d07eda8 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmListModel.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/tab/MainTabVirtualMachineView.java 3 files changed, 46 insertions(+), 0 deletions(-) Approvals: Tomas Jelinek: Looks good to me, approved Arik Hadas: Verified; Passed CI tests -- To view, visit https://gerrit.ovirt.org/44207 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I27cdefc477530c9033e4f8cc54e482ff6d07eda8 Gerrit-PatchSet: 10 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: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: Remove vm button is disabled in vms context menu if multiple...
by shavivi＠redhat.com 05 Aug '15

05 Aug '15

Shahar Havivi has submitted this change and it was merged. Change subject: Remove vm button is disabled in vms context menu if multiple vms selected ...................................................................... Remove vm button is disabled in vms context menu if multiple vms selected Enable VMs removal if some of the selected VMs are down and not if all VMs are down. Change-Id: I8c5cb4bd46390b137917279756a2e77d4b6bbcf1 Bug-Url: https://bugzilla.redhat.com/1218527 Signed-off-by: Shahar Havivi <shaharh(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmListModel.java 1 file changed, 11 insertions(+), 10 deletions(-) Approvals: Tomas Jelinek: Looks good to me, approved Shahar Havivi: Verified; Passed CI tests -- To view, visit https://gerrit.ovirt.org/43897 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8c5cb4bd46390b137917279756a2e77d4b6bbcf1 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in jenkins[master]: ovirt-engine-sdk-java: fixed publisher for master
by sbonazzo＠redhat.com 05 Aug '15

05 Aug '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: ovirt-engine-sdk-java: fixed publisher for master ...................................................................... ovirt-engine-sdk-java: fixed publisher for master Change-Id: I544f9f8c62788a9a1fa3c3f0aafd96f1ce9c7e12 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M jobs/confs/yaml/jobs/ovirt/publish-rpms-nightly-master.yaml 1 file changed, 3 insertions(+), 3 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me; Ready for merge Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/44390 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I544f9f8c62788a9a1fa3c3f0aafd96f1ce9c7e12 Gerrit-PatchSet: 1 Gerrit-Project: jenkins 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: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in jenkins[master]: publish-rpms-nightly-master: updated description
by sbonazzo＠redhat.com 05 Aug '15

05 Aug '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: publish-rpms-nightly-master: updated description ...................................................................... publish-rpms-nightly-master: updated description Updated description to make it clear that the job is under yaml control. Change-Id: I048ee4d7540fa934b72992fdb6368edd743771c1 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M jobs/confs/yaml/jobs/ovirt/publish-rpms-nightly-master.yaml 1 file changed, 5 insertions(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified; Looks good to me; Ready for merge Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/44389 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I048ee4d7540fa934b72992fdb6368edd743771c1 Gerrit-PatchSet: 1 Gerrit-Project: jenkins 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: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-live[ovirt-live-3.6]: build: ovirt-live-3.6_beta1.1
by sbonazzo＠redhat.com 05 Aug '15

05 Aug '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: build: ovirt-live-3.6_beta1.1 ...................................................................... build: ovirt-live-3.6_beta1.1 Change-Id: Iea80e718c2fb25b87d648b50c128ef5915f33262 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, 4 insertions(+), 4 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/44381 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iea80e718c2fb25b87d648b50c128ef5915f33262 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-live Gerrit-Branch: ovirt-live-3.6 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: exttool: usage fixups
by masayag＠redhat.com 05 Aug '15

05 Aug '15

Moti Asayag has submitted this change and it was merged. Change subject: exttool: usage fixups ...................................................................... exttool: usage fixups Change-Id: Ie53622f5ff657982080c3030639ab64957be513f Signed-off-by: Ondra Machacek <omachace(a)redhat.com> --- M backend/manager/extensions-tool/src/main/resources/org/ovirt/engine/exttool/aaa/arguments.properties M backend/manager/extensions-tool/src/main/resources/org/ovirt/engine/exttool/core/info.properties 2 files changed, 5 insertions(+), 5 deletions(-) Approvals: Alon Bar-Lev: Looks good to me, but someone else must approve Ondra Machacek: Verified Moti Asayag: Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/44377 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie53622f5ff657982080c3030639ab64957be513f Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in otopi[master]: packagers: add dnf support
by Alon Bar-Lev 04 Aug '15

04 Aug '15

Alon Bar-Lev has submitted this change and it was merged. Change subject: packagers: add dnf support ...................................................................... packagers: add dnf support Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1249385 Change-Id: I92ce39280a42bfbaab3fee9bee0861ac56bdfe09 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M src/otopi/Makefile.am M src/otopi/constants.py A src/otopi/minidnf.py M src/otopi/miniyum.py M src/otopi/packager.py M src/plugins/otopi/packagers/Makefile.am M src/plugins/otopi/packagers/__init__.py A src/plugins/otopi/packagers/dnfpackager.py M src/plugins/otopi/packagers/yumpackager.py 9 files changed, 1,267 insertions(+), 25 deletions(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/44374 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I92ce39280a42bfbaab3fee9bee0861ac56bdfe09 Gerrit-PatchSet: 1 Gerrit-Project: otopi Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-node-plugin-vdsm[master]: engine_page: Improve formart vars naming
by dougsland＠redhat.com 04 Aug '15

04 Aug '15

Douglas Schilling Landgraf has submitted this change and it was merged. Change subject: engine_page: Improve formart vars naming ...................................................................... engine_page: Improve formart vars naming Change-Id: I492164fd38ba9f364ee7f95e2aa8866226013706 Signed-off-by: Douglas Schilling Landgraf <dougsland(a)redhat.com> Reviewed-on: https://gerrit.ovirt.org/44370 Continuous-Integration: Jenkins CI --- M src/engine_page.py 1 file changed, 4 insertions(+), 4 deletions(-) Approvals: Douglas Schilling Landgraf: Verified; Looks good to me, approved; Passed CI tests Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/44370 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I492164fd38ba9f364ee7f95e2aa8866226013706 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-node-plugin-vdsm Gerrit-Branch: master Gerrit-Owner: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-node-plugin-vdsm[master]: TUI: replace ovirt-register to vdsm-tool register
by dougsland＠redhat.com 04 Aug '15

04 Aug '15

Douglas Schilling Landgraf has submitted this change and it was merged. Change subject: TUI: replace ovirt-register to vdsm-tool register ...................................................................... TUI: replace ovirt-register to vdsm-tool register vdsm-tool register will be the registration tool for 3.6 version. Change-Id: I5ffcb62a02b1064c3d6ecea61853fe7116036e14 Signed-off-by: Douglas Schilling Landgraf <dougsland(a)redhat.com> Reviewed-on: https://gerrit.ovirt.org/44290 Continuous-Integration: Jenkins CI --- M ovirt-node-plugin-vdsm.spec.in M src/engine_page.py 2 files changed, 20 insertions(+), 45 deletions(-) Approvals: Douglas Schilling Landgraf: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/44290 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5ffcb62a02b1064c3d6ecea61853fe7116036e14 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-node-plugin-vdsm Gerrit-Branch: master Gerrit-Owner: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com> Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: Convert CpuFlagsManagerHandler to singleton
by ofrenkel＠redhat.com 04 Aug '15

04 Aug '15

Omer Frenkel has submitted this change and it was merged. Change subject: core: Convert CpuFlagsManagerHandler to singleton ...................................................................... core: Convert CpuFlagsManagerHandler to singleton Change class with static methods CpuFlagsManagerHandler to be singleton injected by SimpleDependecyInjector. This change makes possible to mock CpuFlagsManagerHandler in tests. In particular, it will be possible to mock CpuFlagsManagerHandler.getCpuId() to return non-null CPU IDs. Without this change, null check in VmHandler.isCpuSupported() (to ensure that null CPU ID will not cause NPE) will break tests. Change-Id: Icbc206b93dde4540152d2267ed956cdb188a20ec Signed-off-by: Shmuel Melamud <smelamud(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/Backend.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ChangeVDSClusterCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CpuFlagsManagerHandler.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/GetAllServerCpuListQuery.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/GetClusterUnsupportedVmsCpusQuery.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/GetHostArchitectureQuery.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/GetSupportedCpuListQuery.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/GetVdsByVdsIdQuery.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/HandleVdsCpuFlagsOrClusterChangedCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/QueriesCommandBase.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RunVmCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/SearchQuery.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVdsGroupCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsArchitectureHelper.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsGroupCommandBase.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsGroupOperationCommandBase.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VmCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VmHandler.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/policyunits/CpuLevelFilterPolicyUnit.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/validator/ClusterValidator.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/AddVmCommandTest.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/GetVdsByVdsIdQueryTest.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/UpdateVdsGroupCommandTest.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/UpdateVmCommandTest.java 25 files changed, 134 insertions(+), 50 deletions(-) Approvals: Jenkins CI: Passed CI tests Shmuel Leib Melamud: Verified Omer Frenkel: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/44034 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icbc206b93dde4540152d2267ed956cdb188a20ec Gerrit-PatchSet: 5 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: Jenkins CI Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Shmuel Leib Melamud <smelamud(a)redhat.com> Gerrit-Reviewer: Shmuel Melamud <smelamud(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0