October 2015 - Engine-commits - oVirt List Archives

Change in ovirt-engine[ovirt-engine-3.5]: engine: unable to label nic when nic contains a vlan
by tnisan＠redhat.com 27 Oct '15

27 Oct '15

Tal Nisan has submitted this change and it was merged. Change subject: engine: unable to label nic when nic contains a vlan ...................................................................... engine: unable to label nic when nic contains a vlan When a nic has a user defined vlan on it, it is impossible to label this nic Change-Id: I54342c0c1cc683dcdf845554b3e819921312fd2f Bug-Url: https://bugzilla.redhat.com/1241055 Signed-off-by: Marcin Mirecki <mmirecki(a)redhat.com> Signed-off-by: Alona Kaplan <alkaplan(a)redhat.com> --- 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/network/AddNetworkParametersBuilder.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/AddNetworksByLabelParametersBuilder.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/NetworkParametersBuilder.java 4 files changed, 43 insertions(+), 13 deletions(-) Approvals: Alona Kaplan: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/47773 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I54342c0c1cc683dcdf845554b3e819921312fd2f Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Marcin Mirecki <mmirecki(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>

1 0

Change in ovirt-engine[ovirt-engine-3.6]: webadmin: instance images - filter cinder disks by DC on attach
by tnisan＠redhat.com 27 Oct '15

27 Oct '15

Tal Nisan has submitted this change and it was merged. Change subject: webadmin: instance images - filter cinder disks by DC on attach ...................................................................... webadmin: instance images - filter cinder disks by DC on attach New/Edit VM -> Attach disk dialog: filter cinder disks by VM's datacenter. Change-Id: Id991c722947b67e93a8dd2a7835221ade7ff5985 Bug-Url: https://bugzilla.redhat.com/1275271 Signed-off-by: Daniel Erez <derez(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/InstanceImagesAttachDiskModel.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Tal Nisan: Looks good to me, approved Jenkins CI: Passed CI tests Daniel Erez: Verified -- To view, visit https://gerrit.ovirt.org/47756 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id991c722947b67e93a8dd2a7835221ade7ff5985 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.5]: core: Windows 10 requires special CPU
by tnisan＠redhat.com 27 Oct '15

27 Oct '15

Tal Nisan has submitted this change and it was merged. Change subject: core: Windows 10 requires special CPU ...................................................................... core: Windows 10 requires special CPU Because of BSOD: SYSTEM_THREAD_EXCEPTION_NOT_HANDLES, running guest OS Windows 10 is limited westmere+ CPUs Change-Id: Iaa53264e12d8f80562987ea5bfa131e91d5737b2 Bug-Url: https://bugzilla.redhat.com/1269828 Signed-off-by: Jakub Niedermertl <jniederm(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RunVmCommand.java M packaging/conf/osinfo-defaults.properties 2 files changed, 10 insertions(+), 0 deletions(-) Approvals: Tomas Jelinek: Verified Tal Nisan: Passed CI tests Jakub Niedermertl: Verified Omer Frenkel: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/47752 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iaa53264e12d8f80562987ea5bfa131e91d5737b2 Gerrit-PatchSet: 8 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Jakub Niedermertl <jniederm(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: 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: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com>

1 0

Change in ovirt-engine[ovirt-engine-3.5]: core: hotfix daotests in VmStaticDAOTest
by tnisan＠redhat.com 27 Oct '15

27 Oct '15

Tal Nisan has submitted this change and it was merged. Change subject: core: hotfix daotests in VmStaticDAOTest ...................................................................... core: hotfix daotests in VmStaticDAOTest testGetOrderedVmGuidsForRunMultipleActionsByMigrationSupport which tested db logic for ordered VmGuids retrieval (with respect to MigrationSupport attribute) was failing because the number of tested vms was 4, but MigrationSupport has only 3 possible values. Therefore there were 2 VMs that were equal with respect to this ordering. Hot-fixed by restricting the number of test VMs to 3. Change-Id: I53df7e6003582ece29387c59312ea9a267f7a1d0 Bug-Url: https://bugzilla.redhat.com/1220116 Signed-off-by: Frantisek Kobzik <fkobzik(a)redhat.com> --- M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/VmStaticDAOTest.java 1 file changed, 3 insertions(+), 0 deletions(-) Approvals: Dudi Maroshi: Verified Jenkins CI: Passed CI tests Roy Golan: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/47405 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I53df7e6003582ece29387c59312ea9a267f7a1d0 Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Dudi Maroshi <dudi(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Dudi Maroshi <dudi(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>

1 0

Change in ovirt-engine[ovirt-engine-3.5]: core: Compensation of active snapshot ID change
by tnisan＠redhat.com 27 Oct '15

27 Oct '15

Tal Nisan has submitted this change and it was merged. Change subject: core: Compensation of active snapshot ID change ...................................................................... core: Compensation of active snapshot ID change If exception is thrown inside CreateAllSnapshotsFromVmCommand, we need to compensate the change of active snapshot ID that is performed during snapshot creation. Otherwise, link between disk images and the active snapshot is lost. Change-Id: I432859105d31f74f10711f6a63b188d325a43e30 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1274717 Signed-off-by: Shmuel Melamud <smelamud(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CreateAllSnapshotsFromVmCommand.java 1 file changed, 20 insertions(+), 3 deletions(-) Approvals: Shahar Havivi: Looks good to me, approved Jenkins CI: Passed CI tests Shmuel Leib Melamud: Verified -- To view, visit https://gerrit.ovirt.org/47750 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I432859105d31f74f10711f6a63b188d325a43e30 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Shmuel Leib Melamud <smelamud(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Shmuel Leib Melamud <smelamud(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine-sdk-java[sdk_3.5]: sdk: Prepare for next development iteration
by juan.hernandez＠redhat.com 27 Oct '15

27 Oct '15

Juan Hernandez has submitted this change and it was merged. Change subject: sdk: Prepare for next development iteration ...................................................................... sdk: Prepare for next development iteration In preparation for the next development iteration this patch increases the version number, adds the SNAPSHOT suffix to the maven version and sets the RPM release number to 0.1. Change-Id: I2574be2119e3347fe28729f31b814277441e31e3 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M ovirt-engine-sdk-java/Makefile M ovirt-engine-sdk-java/pom.xml 2 files changed, 2 insertions(+), 2 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/47722 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2574be2119e3347fe28729f31b814277441e31e3 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk-java Gerrit-Branch: sdk_3.5 Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine-sdk-java[sdk_3.5]: sdk: Release 3.5.6.0
by juan.hernandez＠redhat.com 27 Oct '15

27 Oct '15

Juan Hernandez has submitted this change and it was merged. Change subject: sdk: Release 3.5.6.0 ...................................................................... sdk: Release 3.5.6.0 Change-Id: Ifb92cbc3a9f6f6902ad3c64cbaa370edcef6b1a5 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M ovirt-engine-sdk-java/Makefile M ovirt-engine-sdk-java/pom.xml M ovirt-engine-sdk-java/spec.el6.in M ovirt-engine-sdk-java/spec.el7.in M ovirt-engine-sdk-java/spec.fc20.in M ovirt-engine-sdk-java/spec.fc21.in M ovirt-engine-sdk-java/spec.fc22.in 7 files changed, 22 insertions(+), 2 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/47721 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ifb92cbc3a9f6f6902ad3c64cbaa370edcef6b1a5 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk-java Gerrit-Branch: sdk_3.5 Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine-sdk-java[sdk_3.5]: sdk: Regenerate against the latest API
by juan.hernandez＠redhat.com 27 Oct '15

27 Oct '15

Juan Hernandez has submitted this change and it was merged. Change subject: sdk: Regenerate against the latest API ...................................................................... sdk: Regenerate against the latest API This patch regenerates the SDK using the latest API obtained from the engine built from commit 451c8f5. Change-Id: I59c8ec433ae93ba17886fecc3b27d81331459932 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M ovirt-engine-sdk-java-codegen/src/main/resources/api.rsdl M ovirt-engine-sdk-java-codegen/src/main/resources/api.xsd M ovirt-engine-sdk-java/Makefile M ovirt-engine-sdk-java/src/main/java/org/ovirt/engine/sdk/decorators/VmPools.java M ovirt-engine-sdk-java/src/main/java/org/ovirt/engine/sdk/entities/ObjectFactory.java M ovirt-engine-sdk-java/src/main/java/org/ovirt/engine/sdk/entities/VersionCaps.java M ovirt-engine-sdk-java/src/main/java/org/ovirt/engine/sdk/entities/VmPool.java A ovirt-engine-sdk-java/src/main/java/org/ovirt/engine/sdk/entities/VmPoolTypes.java 8 files changed, 203 insertions(+), 2 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/47720 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I59c8ec433ae93ba17886fecc3b27d81331459932 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk-java Gerrit-Branch: sdk_3.5 Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine-sdk[sdk_3.5]: sdk: Prepare for next development iteration
by juan.hernandez＠redhat.com 27 Oct '15

27 Oct '15

Juan Hernandez has submitted this change and it was merged. Change subject: sdk: Prepare for next development iteration ...................................................................... sdk: Prepare for next development iteration In preparation for the next development iteration this patch increases the version number and sets the RPM relase number to 0.1. Change-Id: Ia2d4e0c03434eccfdf00c61140892fdae695ca8d Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M Makefile M setup.py 2 files changed, 3 insertions(+), 3 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me; Passed CI tests; Ready for merge Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/47718 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia2d4e0c03434eccfdf00c61140892fdae695ca8d Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk Gerrit-Branch: sdk_3.5 Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine-sdk[sdk_3.5]: sdk: Release 3.5.6.0
by juan.hernandez＠redhat.com 27 Oct '15

27 Oct '15

Juan Hernandez has submitted this change and it was merged. Change subject: sdk: Release 3.5.6.0 ...................................................................... sdk: Release 3.5.6.0 Change-Id: I27350f1a66527b502e09e488c94bef39d3315bf6 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M Makefile M ovirt-engine-sdk-python.spec.in M setup.py 3 files changed, 7 insertions(+), 3 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me; Passed CI tests; Ready for merge Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/47717 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I27350f1a66527b502e09e488c94bef39d3315bf6 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk Gerrit-Branch: sdk_3.5 Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0