April 2016 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: core: Update of HostNetworkQos on DataCenter must trigger up...
by alkaplan＠redhat.com 21 Apr '16

21 Apr '16

Alona Kaplan has submitted this change and it was merged. Change subject: core: Update of HostNetworkQos on DataCenter must trigger update of network ...................................................................... core: Update of HostNetworkQos on DataCenter must trigger update of network before this update, if HostNetworkQos was updated on datacenter, and something other that name or description was changed, it rendered network out of sync, because on database records was updated, not network itself. Change-Id: I67882d1c68806b852ee655e46fff03aca87a04ce Bug-Url: https://bugzilla.redhat.com/1279771 Signed-off-by: Martin Mucha <mmucha(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/qos/UpdateHostNetworkQosCommand.java 1 file changed, 157 insertions(+), 0 deletions(-) Approvals: Martin Mucha: Verified Alona Kaplan: Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/52952 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I67882d1c68806b852ee655e46fff03aca87a04ce 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: Jenkins CI Gerrit-Reviewer: Marcin Mirecki <mmirecki(a)redhat.com> Gerrit-Reviewer: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: webadmin: fix provider isReadOnlyAware flag
by derez＠redhat.com 21 Apr '16

21 Apr '16

Daniel Erez has submitted this change and it was merged. Change subject: webadmin: fix provider isReadOnlyAware flag ...................................................................... webadmin: fix provider isReadOnlyAware flag ProviderType -> isReadOnlyAware method returns a wrong flag, which caused an exception when editing a non 'ExternalNetworkProvider', and the 'Read Only' checkbox was displays for all providers type. Fixed by returning the correct flag. Change-Id: Ie2c036548cb477abba617c01268eacec6af99ef3 Signed-off-by: Daniel Erez <derez(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/ProviderType.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Alona Kaplan: Looks good to me, approved Daniel Erez: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/56356 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie2c036548cb477abba617c01268eacec6af99ef3 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: services as export for woorea in XML conf
by derez＠redhat.com 21 Apr '16

21 Apr '16

Daniel Erez has submitted this change and it was merged. Change subject: core: services as export for woorea in XML conf ...................................................................... core: services as export for woorea in XML conf Changed 'services' attribute to export for woorea module in XML config. This is required to allow the ServiceLoader in OpenStackClient to find the RESTEasyConnector class. Change-Id: I6711107e352d5e3e0a94a377f7e826596025d462 Signed-off-by: Daniel Erez <derez(a)redhat.com> --- M ear/src/main/application/META-INF/jboss-deployment-structure.xml 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Martin Peřina: Looks good to me, approved Daniel Erez: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/56436 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6711107e352d5e3e0a94a377f7e826596025d462 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: StorageHelperBase#filterConnectionsByStorageType
by amureini＠redhat.com 21 Apr '16

21 Apr '16

Allon Mureinik has submitted this change and it was merged. Change subject: core: StorageHelperBase#filterConnectionsByStorageType ...................................................................... core: StorageHelperBase#filterConnectionsByStorageType Used Java 8 streaming API to reduce boilerplate code. Change-Id: I27d3bef4c9b1a581d9416d64525a17be43ffea4b Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/connection/StorageHelperBase.java 1 file changed, 2 insertions(+), 10 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Jenkins CI: Passed CI tests Allon Mureinik: Verified Ala Hino: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/56370 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I27d3bef4c9b1a581d9416d64525a17be43ffea4b Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Ala Hino <ahino(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Maor Lipchuk <mlipchuk(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: remove vm jobs in batch
by ahadas＠redhat.com 21 Apr '16

21 Apr '16

Arik Hadas has submitted this change and it was merged. Change subject: core: remove vm jobs in batch ...................................................................... core: remove vm jobs in batch VM jobs are typically (currently always) removed in groups as a result of aggregating the jobs reported from VMs that are running on the same host. Thus, it worth doing the remove in batch (like we do for updates). Change-Id: Id91f402f414c4df9dd41b36fbf232c8fe5a9788b Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/monitoring/VmJobsMonitoring.java M backend/manager/modules/vdsbroker/src/test/java/org/ovirt/engine/core/vdsbroker/monitoring/VmJobsMonitoringTest.java 2 files changed, 21 insertions(+), 38 deletions(-) Approvals: Jenkins CI: Passed CI tests Arik Hadas: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/56245 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id91f402f414c4df9dd41b36fbf232c8fe5a9788b Gerrit-PatchSet: 3 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: Add cinder call back for AddVmFromSnapshot.
by mlipchuk＠redhat.com 21 Apr '16

21 Apr '16

Maor Lipchuk has submitted this change and it was merged. Change subject: core: Add cinder call back for AddVmFromSnapshot. ...................................................................... core: Add cinder call back for AddVmFromSnapshot. When clonning a VM from Cinder snapshot the VM and the snapshot stay in lock status since AddVmFromSnapshot has no call back method. Adding the call back method should fix this issue. Change-Id: Ic4a41089fbf035ddd4b914885db5dc2a39f1a25c Bug-Url: https://bugzilla.redhat.com/1316853 Signed-off-by: Maor Lipchuk <mlipchuk(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVmFromSnapshotCommand.java 1 file changed, 15 insertions(+), 0 deletions(-) Approvals: Daniel Erez: Looks good to me, approved Jenkins CI: Passed CI tests Maor Lipchuk: Verified -- To view, visit https://gerrit.ovirt.org/56305 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic4a41089fbf035ddd4b914885db5dc2a39f1a25c Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: restapi: Use empty string as default value of parameters
by juan.hernandez＠redhat.com 21 Apr '16

21 Apr '16

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Use empty string as default value of parameters ...................................................................... restapi: Use empty string as default value of parameters Currently matrix and query parameters that don't have a value are ignored. That is incorrect, we should use an empty string as the default value. Change-Id: I43fa2304cfc1aea406ff1f0da0da20f440346ecb Bug-Url: https://bugzilla.redhat.com/1328404 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/modules/restapi/interface/common/jaxrs/src/main/java/org/ovirt/engine/api/common/util/ParametersHelper.java 1 file changed, 11 insertions(+), 5 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/56332 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I43fa2304cfc1aea406ff1f0da0da20f440346ecb Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine 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: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: Remove redundand action type from hot plug/unplug disk...
by tnisan＠redhat.com 21 Apr '16

21 Apr '16

Tal Nisan has submitted this change and it was merged. Change subject: core: Remove redundand action type from hot plug/unplug disk tests ...................................................................... core: Remove redundand action type from hot plug/unplug disk tests The getCommandActionType seems to be redundant thus removed Change-Id: I2d5f6482b0c88c6233e62ae5fc105f5de886b8e3 Signed-off-by: Tal Nisan <tnisan(a)redhat.com> --- M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/storage/disk/HotPlugDiskToVmCommandTest.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/storage/disk/HotUnPlugDiskFromVmCommandTest.java 2 files changed, 0 insertions(+), 12 deletions(-) Approvals: Tal Nisan: Verified; Passed CI tests Allon Mureinik: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/56425 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2d5f6482b0c88c6233e62ae5fc105f5de886b8e3 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: Avoid redundant VmDevice creation in HotPlugDiskToVmCo...
by tnisan＠redhat.com 21 Apr '16

21 Apr '16

Tal Nisan has submitted this change and it was merged. Change subject: core: Avoid redundant VmDevice creation in HotPlugDiskToVmCommandTest ...................................................................... core: Avoid redundant VmDevice creation in HotPlugDiskToVmCommandTest The createDiskWrongPlug method creates a redundant VmDevice for the sole purpose of making it plugged/unplugged, this can be done by simply using the already existing VmDevice created in the setup and just changing it's plugged status Also the redundant mockVmDevice(false) was removed from createVirtIODisk as an unplugged device is already mocked in the setup Change-Id: Icf50857716f63706b85a05ce8390980dc2b4659a Signed-off-by: Tal Nisan <tnisan(a)redhat.com> --- M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/storage/disk/HotPlugDiskToVmCommandTest.java 1 file changed, 3 insertions(+), 3 deletions(-) Approvals: Tal Nisan: Verified; Passed CI tests Allon Mureinik: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/56423 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icf50857716f63706b85a05ce8390980dc2b4659a Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: Remove duplicate from HotUnPlugDiskFromVmCommandTest
by tnisan＠redhat.com 21 Apr '16

21 Apr '16

Tal Nisan has submitted this change and it was merged. Change subject: core: Remove duplicate from HotUnPlugDiskFromVmCommandTest ...................................................................... core: Remove duplicate from HotUnPlugDiskFromVmCommandTest The method createVirtIODisk is exactly the same as in the super class only changes the status to plugged, the content was removed and a call to the super method was done instead along with changing the disk status to plugged Change-Id: Iccce97c33787a271161418e22254bbb417d8e2b5 Signed-off-by: Tal Nisan <tnisan(a)redhat.com> --- M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/storage/disk/HotPlugDiskToVmCommandTest.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/storage/disk/HotUnPlugDiskFromVmCommandTest.java 2 files changed, 3 insertions(+), 18 deletions(-) Approvals: Tal Nisan: Verified; Passed CI tests Allon Mureinik: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/56424 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iccce97c33787a271161418e22254bbb417d8e2b5 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0