Change in ovirt-engine[ovirt-engine-4.1]: core: SyncLunsInfoForBlockStorageDomain - metadata devices r...

Tal Nisan has submitted this change and it was merged. Change subject: core: SyncLunsInfoForBlockStorageDomain - metadata devices refresh ...................................................................... core: SyncLunsInfoForBlockStorageDomain - metadata devices refresh This patch changes the SyncLunsInfoForBlockStorageDomain command to skip executing the metadata devices refresh if no vdsId was passed to the command. refreshMetadataDevicesInfo() is executes BlockStorageDomainHelper.fillMetadataDevicesInfo() which currently doesn't use an api that support randomizing a host for execution. Currently all the flows the needs that functionallity do pass the vds id, while the command should obviously not be aware to the parameters passed by it's callers - as this patch is intended to be backported to the 4.1 branch the change here is minimal - skips the execution if no vds id was passed. In the master branch fillMetadataDevicesInfo() will be updated to supported randomizing a host which will enable us to remove that check. If we want to skip the metadata devices refresh it should be done using a specific parameter. Change-Id: I35bdcbe4d5ff55e2fbe39ca4ab28b05b4802454b Bug-Url: https://bugzilla.redhat.com/1452984 Signed-off-by: Liron Aravot <laravot(a)redhat.com&gt; --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/domain/SyncLunsInfoForBlockStorageDomainCommand.java 1 file changed, 18 insertions(+), 7 deletions(-) Approvals: Jenkins CI: Passed CI tests Maor Lipchuk: Looks good to me, approved Liron Aravot: Verified -- To view, visit https://gerrit.ovirt.org/77207 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I35bdcbe4d5ff55e2fbe39ca4ab28b05b4802454b Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.1 Gerrit-Owner: Liron Aravot <laravot(a)redhat.com&gt; Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com&gt; Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com&gt; Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com&gt; Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org&gt;