July 2016 - Engine-commits - oVirt List Archives

Change in ovirt-engine-sdk[master]: Add example of adding VM snapshot
by omachace＠redhat.com 19 Jul '16

19 Jul '16

Ondra Machacek has submitted this change and it was merged. Change subject: Add example of adding VM snapshot ...................................................................... Add example of adding VM snapshot This patch adds an example that shows how to add a snapshot to an existing virtual machine. Change-Id: I8a0c225ab96c01be9a8cb29dcdf7498f094422d7 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- A sdk/examples/add_vm_snapshot.py 1 file changed, 50 insertions(+), 0 deletions(-) Approvals: Ondra Machacek: Looks good to me, approved Juan Hernandez: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/61011 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8a0c225ab96c01be9a8cb29dcdf7498f094422d7 Gerrit-PatchSet: 2 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-sdk-java[sdk_4.0]: Add example of adding VM snapshot
by omachace＠redhat.com 19 Jul '16

19 Jul '16

Ondra Machacek has submitted this change and it was merged. Change subject: Add example of adding VM snapshot ...................................................................... Add example of adding VM snapshot This patch adds an example that shows how to add a snapshot to an existing virtual machine. Change-Id: Icc8771f3afc76d9ddba5047c00a61c6d53db6456 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> (cherry picked from commit c62f4db8bdc3031e33e6d89e584137d2cf587411) --- A sdk/src/test/java/org/ovirt/engine/sdk4/examples/AddVmSnapshot.java 1 file changed, 56 insertions(+), 0 deletions(-) Approvals: Ondra Machacek: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/61020 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icc8771f3afc76d9ddba5047c00a61c6d53db6456 Gerrit-PatchSet: 1 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: 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-sdk-java[master]: Add example of adding VM snapshot
by omachace＠redhat.com 19 Jul '16

19 Jul '16

Ondra Machacek has submitted this change and it was merged. Change subject: Add example of adding VM snapshot ...................................................................... Add example of adding VM snapshot This patch adds an example that shows how to add a snapshot to an existing virtual machine. Change-Id: Icc8771f3afc76d9ddba5047c00a61c6d53db6456 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- A sdk/src/test/java/org/ovirt/engine/sdk4/examples/AddVmSnapshot.java 1 file changed, 56 insertions(+), 0 deletions(-) Approvals: Ondra Machacek: Looks good to me, approved Juan Hernandez: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/61013 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icc8771f3afc76d9ddba5047c00a61c6d53db6456 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk-java 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]: core: after HostNetworkQos is removed, sync influenced netwo...
by masayag＠redhat.com 19 Jul '16

19 Jul '16

Moti Asayag has submitted this change and it was merged. Change subject: core: after HostNetworkQos is removed, sync influenced networks. ...................................................................... core: after HostNetworkQos is removed, sync influenced networks. Already existing in UpdateHostNetworkQosCommand was refactored a little extracted into common ancestor and used also in RemoveHostNetworkQosCommand so that removal of HostNetworkQos also sync influenced networks, so that they aren't left in out of sync state. Change-Id: Ifa8111a59d3c93cb0256727bde620d2d2928338f Bug-Url: https://bugzilla.redhat.com/1279771 Signed-off-by: Martin Mucha <mmucha(a)redhat.com> --- A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/qos/PersistentHostSetupNetworksParametersFactory.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/qos/QosCommandBase.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/qos/RefreshNetworksParametersFactory.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/qos/RemoveHostNetworkQosCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/qos/UpdateHostNetworkQosCommand.java A backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/qos/PersistentHostSetupNetworksParametersFactoryTest.java A backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/qos/RefreshNetworksParametersFactoryTest.java 7 files changed, 393 insertions(+), 125 deletions(-) Approvals: Martin Mucha: Verified Jenkins CI: Passed CI tests Moti Asayag: Looks good to me, approved Yevgeny Zaspitsky: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/59310 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ifa8111a59d3c93cb0256727bde620d2d2928338f Gerrit-PatchSet: 7 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Marcin Mirecki <mmirecki(a)redhat.com> Gerrit-Reviewer: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(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[master]: engine: remove unsused code from VmNicValidator
by masayag＠redhat.com 19 Jul '16

19 Jul '16

Moti Asayag has submitted this change and it was merged. Change subject: engine: remove unsused code from VmNicValidator ...................................................................... engine: remove unsused code from VmNicValidator Change-Id: I53d77774ac4fcf8dfeaedbb20de47e5b0abe15b6 Signed-off-by: Yevgeny Zaspitsky <yzaspits(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/validator/VmNicValidator.java 1 file changed, 0 insertions(+), 6 deletions(-) Approvals: Jenkins CI: Passed CI tests Moti Asayag: Looks good to me, approved Yevgeny Zaspitsky: Verified -- To view, visit https://gerrit.ovirt.org/60929 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I53d77774ac4fcf8dfeaedbb20de47e5b0abe15b6 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yevgeny Zaspitsky <yzaspits(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(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[master]: core: use only vm static data in vm-pool monitor
by ahadas＠redhat.com 19 Jul '16

19 Jul '16

Arik Hadas has submitted this change and it was merged. Change subject: core: use only vm static data in vm-pool monitor ...................................................................... core: use only vm static data in vm-pool monitor Replace the costly query of VM with a lighter query for VmStatic. Change-Id: I805e571cb4b68a8776e47f6d8bd6b4098208c122 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VmPoolMonitor.java 1 file changed, 10 insertions(+), 10 deletions(-) Approvals: Shahar Havivi: Looks good to me, approved Jenkins CI: Passed CI tests Arik Hadas: Verified -- To view, visit https://gerrit.ovirt.org/61009 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I805e571cb4b68a8776e47f6d8bd6b4098208c122 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: refine daos in vm-pool monitor
by ahadas＠redhat.com 19 Jul '16

19 Jul '16

Arik Hadas has submitted this change and it was merged. Change subject: core: refine daos in vm-pool monitor ...................................................................... core: refine daos in vm-pool monitor Inject VmDao and VmPoolDao instead of using DbFacade. Change-Id: I6d0904962b2aea4e98931053392f51effbc6aa99 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VmPoolMonitor.java 1 file changed, 9 insertions(+), 4 deletions(-) Approvals: Shahar Havivi: Looks good to me, approved Arik Hadas: Verified; Passed CI tests -- To view, visit https://gerrit.ovirt.org/61008 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6d0904962b2aea4e98931053392f51effbc6aa99 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: do not attempt to update interface statistics on destr...
by ahadas＠redhat.com 19 Jul '16

19 Jul '16

Arik Hadas has submitted this change and it was merged. Change subject: core: do not attempt to update interface statistics on destroy vm ...................................................................... core: do not attempt to update interface statistics on destroy vm Now that DestroyVm only sets the VM status to powering down and wait for the monitoring to detect that the VM is actually down, DestroyVm should not update the network interface statistics anymore. Change-Id: I23f97757c73be64b6b579ae6e0730ab1ad9a58ce Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/DestroyVmVDSCommand.java 1 file changed, 0 insertions(+), 17 deletions(-) Approvals: Shahar Havivi: Looks good to me, approved Jenkins CI: Passed CI tests Arik Hadas: Verified -- To view, visit https://gerrit.ovirt.org/60923 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I23f97757c73be64b6b579ae6e0730ab1ad9a58ce Gerrit-PatchSet: 5 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-ruby[sdk_4.0]: Add example of adding VM snapshot
by juan.hernandez＠redhat.com 19 Jul '16

19 Jul '16

Juan Hernandez has submitted this change and it was merged. Change subject: Add example of adding VM snapshot ...................................................................... Add example of adding VM snapshot This patch adds an example that shows how to add a snapshot to an existing virtual machine. Change-Id: Ia4bbbdc0bc72eeaf152e4d4bb76e89cc7f20a016 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> (cherry picked from commit 5b69af6c6995449bbc3fd0a3a0b202903d7832d5) --- A sdk/examples/add_vm_snapshot.rb 1 file changed, 51 insertions(+), 0 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/61010 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia4bbbdc0bc72eeaf152e4d4bb76e89cc7f20a016 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk-ruby Gerrit-Branch: sdk_4.0 Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: update script which makes all macs in mac_pool_ranges ...
by oourfali＠redhat.com 19 Jul '16

19 Jul '16

Oved Ourfali has submitted this change and it was merged. Change subject: core: update script which makes all macs in mac_pool_ranges in lowercase ...................................................................... core: update script which makes all macs in mac_pool_ranges in lowercase Change-Id: I14891e6849b01eca95af961491f5700419a5bec9 Signed-off-by: Martin Mucha <mmucha(a)redhat.com> --- A packaging/dbscripts/upgrade/04_01_0160_update_macs_to_lower_case.sql 1 file changed, 3 insertions(+), 0 deletions(-) Approvals: Martin Mucha: Verified Oved Ourfali: Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/58330 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I14891e6849b01eca95af961491f5700419a5bec9 Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Marcin Mirecki <mmirecki(a)redhat.com> Gerrit-Reviewer: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0