September 2016 - Engine-commits - oVirt List Archives

Change in ovirt-engine[ovirt-engine-3.6.9]: core: Support migrating the HE VM out of cluster
by tnisan＠redhat.com 06 Sep '16

06 Sep '16

Tal Nisan has submitted this change and it was merged. Change subject: core: Support migrating the HE VM out of cluster ...................................................................... core: Support migrating the HE VM out of cluster This would allow migrating the hosted engine VM out of the cluster. This is specifically needed in case where we want to migrate old cluster version to a new cluster, with new version. Change-Id: Ia49024fb7a6d9558e28c0da5465aff2118f7bbe8 Bug-Url: https://bugzilla.redhat.com/1356127 Signed-off-by: Roy Golan <rgolan(a)redhat.com> (cherry picked from commit 40fbe22c1816b24c746402814121772a5a0f6f97) --- A backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/ChangeVMClusterCommandTest.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/validation/VmActionByVmOriginTypeValidator.java 2 files changed, 44 insertions(+), 0 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Jenkins CI: Passed CI tests Roy Golan: Verified Andrej Krejcir: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/63377 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia49024fb7a6d9558e28c0da5465aff2118f7bbe8 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6.9 Gerrit-Owner: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Andrej Krejcir <akrejcir(a)redhat.com> Gerrit-Reviewer: Jenkins CI 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-sdk[master]: Fix reading of lists of elements
by omachace＠redhat.com 06 Sep '16

06 Sep '16

Ondra Machacek has submitted this change and it was merged. Change subject: Fix reading of lists of elements ...................................................................... Fix reading of lists of elements Currently the method that read lists of elements doesn't handle correctly empty elements without close tags. For example, when reading the following cluster: <cluster> ... <required_rng_sources/> <switch_type>legacy</switch_type> </cluster> The 'legacy' value of the 'switch_type' element is interpreted as one of the values of the previous 'required_rng_sources' list. This patch fixes the method and introduces two tests to verify that it works correctly. Change-Id: I76c3625719c5e2ef327fcee7cec974d740d5f655 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M sdk/ext/ov_xml_reader.c A sdk/tests/test_cluster_reader.py M sdk/tests/test_xml_reader.py 3 files changed, 101 insertions(+), 10 deletions(-) Approvals: Ondra Machacek: Looks good to me, approved Juan Hernandez: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/63385 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I76c3625719c5e2ef327fcee7cec974d740d5f655 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: webadmin: infinite loop in new VM/pool dialog
by tjelinek＠redhat.com 06 Sep '16

06 Sep '16

Tomas Jelinek has submitted this change and it was merged. Change subject: webadmin: infinite loop in new VM/pool dialog ...................................................................... webadmin: infinite loop in new VM/pool dialog Open new VM/pool dialog and pick a template which had non-null custom compatibility version, the dialog ends up in infinite refresh loop. The reason was that the CoreVmBaseToUnitBuilder.postBuild called the model.getCustomCompatibilityVersion().setSelectedItem(vm.getCustomCompatibilityVersion()); all the time regardless of the fact it changed or not. This change leads to a full refresh of the whole dialog, including this builder. Fixed by setting this value only if changed. Change-Id: I6c2e24faf1793f9de0e973ee41425f7cd32ea922 Bug-Url: https://bugzilla.redhat.com/1373204 Signed-off-by: Tomas Jelinek <tjelinek(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/builders/vm/CoreVmBaseToUnitBuilder.java 1 file changed, 7 insertions(+), 1 deletion(-) Approvals: Tomas Jelinek: Verified Jenkins CI: Passed CI tests Martin Betak: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/63305 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6c2e24faf1793f9de0e973ee41425f7cd32ea922 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tomas Jelinek <tjelinek(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-4.0]: webadmin: Look for sound device by general type
by tnisan＠redhat.com 06 Sep '16

06 Sep '16

Tal Nisan has submitted this change and it was merged. Change subject: webadmin: Look for sound device by general type ...................................................................... webadmin: Look for sound device by general type The sound device in the managed device map of a VM must be located by VmDeviceGeneralType.SOUND, not by VmDeviceType.SOUND. The code that was fixed in InstanceTypeManager.postDoUpdateManagedFieldsFrom(VmBase) is called when user edits a VM and NEXT_RUN snapshot exists. After getting the information about NEXT_RUN snapshot, UI controls in the dialog are updated accordingly. The "Sound card" checkbox is always unchecked, because the code looks for a sound device incorrectly and cannot find it. When user clicks "OK" in the dialog, the difference between actual sound device configuration and the checkbox state is detected and confirmation dialog is displayed. Change-Id: I179bb4623815a90b5468e25e3432351e5458e04a Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1356568 Signed-off-by: Shmuel Melamud <smelamud(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/utils/VmDeviceCommonUtils.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/instancetypes/InstanceTypeManager.java 2 files changed, 5 insertions(+), 5 deletions(-) Approvals: Jenkins CI: Passed CI tests Shmuel Leib Melamud: Verified Arik Hadas: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/63293 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I179bb4623815a90b5468e25e3432351e5458e04a Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Shmuel Leib Melamud <smelamud(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Shmuel Leib Melamud <smelamud(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]: backend: Refactor loading of Backend Services
by rgolan＠redhat.com 06 Sep '16

06 Sep '16

Roy Golan has submitted this change and it was merged. Change subject: backend: Refactor loading of Backend Services ...................................................................... backend: Refactor loading of Backend Services Backend services (classes implementing the BackendService marker interface) were loaded from 2 different places (Backend, InitServicesOnStartupBean) using CDI instance lookup. This duplicate logic was extracted into newly created ServiceLoader whose sole purpose is to provide a simple and uniform mechanism for loading of services. Change-Id: I5454be63f248f110f7ebf333e3876332f171da19 Signed-off-by: Martin Betak <mbetak(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/InitBackendServicesOnStartupBean.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ServiceLoader.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/BackendService.java 4 files changed, 48 insertions(+), 14 deletions(-) Approvals: Martin Peřina: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Martin Betak: Verified Roy Golan: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/62288 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5454be63f248f110f7ebf333e3876332f171da19 Gerrit-PatchSet: 12 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Betak <mbetak(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: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Yevgeny Zaspitsky <yzaspits(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-3.6]: core: fix wrong imports order
by rgolan＠redhat.com 06 Sep '16

06 Sep '16

Roy Golan has submitted this change and it was merged. Change subject: core: fix wrong imports order ...................................................................... core: fix wrong imports order Change-Id: I7b8a3dd7c0d0c7d0b3baa5d1a8188d0ecce7fec9 Bug-Url: https://bugzilla.redhat.com/1356127 Signed-off-by: Roy Golan <rgolan(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ChangeVMClusterCommand.java 1 file changed, 3 insertions(+), 3 deletions(-) Objections: gerrit-hooks: Fails -- To view, visit https://gerrit.ovirt.org/63378 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7b8a3dd7c0d0c7d0b3baa5d1a8188d0ecce7fec9 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-3.6]: core: Support migrating the HE VM out of cluster
by rgolan＠redhat.com 06 Sep '16

06 Sep '16

Roy Golan has submitted this change and it was merged. Change subject: core: Support migrating the HE VM out of cluster ...................................................................... core: Support migrating the HE VM out of cluster This would allow migrating the hosted engine VM out of the cluster. This is specifically needed in case where we want to migrate old cluster version to a new cluster, with new version. Change-Id: Ia49024fb7a6d9558e28c0da5465aff2118f7bbe8 Bug-Url: https://bugzilla.redhat.com/1356127 Signed-off-by: Roy Golan <rgolan(a)redhat.com> (cherry picked from commit 40fbe22c1816b24c746402814121772a5a0f6f97) --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ChangeVMClusterCommand.java A backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/ChangeVMClusterCommandTest.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/validation/VmActionByVmOriginTypeValidator.java 3 files changed, 47 insertions(+), 3 deletions(-) -- To view, visit https://gerrit.ovirt.org/62425 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia49024fb7a6d9558e28c0da5465aff2118f7bbe8 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Jenny Tokar <jtokar(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]: core: cleanup in CpuFlagsManagerHandler
by ahadas＠redhat.com 06 Sep '16

06 Sep '16

Arik Hadas has submitted this change and it was merged. Change subject: core: cleanup in CpuFlagsManagerHandler ...................................................................... core: cleanup in CpuFlagsManagerHandler Change-Id: Id88def0777493ad663d2ff8c00da00d4ef455b85 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CpuFlagsManagerHandler.java 1 file changed, 105 insertions(+), 132 deletions(-) Approvals: Shahar Havivi: Looks good to me, approved Jenkins CI: Passed CI tests Arik Hadas: Verified -- To view, visit https://gerrit.ovirt.org/63312 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id88def0777493ad663d2ff8c00da00d4ef455b85 Gerrit-PatchSet: 2 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[master]: core: cleanup in resource manager
by ahadas＠redhat.com 06 Sep '16

06 Sep '16

Arik Hadas has submitted this change and it was merged. Change subject: core: cleanup in resource manager ...................................................................... core: cleanup in resource manager Remove unused fields. Change-Id: I53292dfaa93720f3395ce3016b34f13c248fc86c Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/ResourceManager.java 1 file changed, 0 insertions(+), 8 deletions(-) Approvals: Shahar Havivi: Looks good to me, approved Jenkins CI: Passed CI tests Arik Hadas: Verified -- To view, visit https://gerrit.ovirt.org/63328 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I53292dfaa93720f3395ce3016b34f13c248fc86c Gerrit-PatchSet: 2 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-sdk-java[sdk_4.0]: [maven-release-plugin] prepare for next development iteration
by omachace＠redhat.com 06 Sep '16

06 Sep '16

Ondra Machacek has submitted this change and it was merged. Change subject: [maven-release-plugin] prepare for next development iteration ...................................................................... [maven-release-plugin] prepare for next development iteration Change-Id: If6839b96a16ce3f6eb915134b82ce6ad130c4433 Signed-off-by: Ondra Machacek <omachace(a)redhat.com> --- M automation/build-artifacts.sh M generator/pom.xml M pom.xml M sdk/pom.xml 4 files changed, 6 insertions(+), 6 deletions(-) Approvals: Ondra Machacek: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/63327 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If6839b96a16ce3f6eb915134b82ce6ad130c4433 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine-sdk-java Gerrit-Branch: sdk_4.0 Gerrit-Owner: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0