Adam Litke has submitted this change and it was merged. Change subject: sdm: Pass the real hostid to copy_data ......................................................................
sdm: Pass the real hostid to copy_data We have been passing the hostid as None to copy_data but the correct value will need to be passed in order to support volume leases. Since we use the same hostid for all storage domains and the that hostid is the same as the id attribute of the connected storage pool we can add a simple helper to supply the proper hostid to sdm verbs. This will need to be changed if we remove storage pools or if we opt to use different hostid values for each domain. Change-Id: I367576fca126be695d7617477f92110941620a3b Signed-off-by: Adam Litke <ali...@redhat.com> Reviewed-on: https://gerrit.ovirt.org/62576 Reviewed-by: Nir Soffer <nsof...@redhat.com> Continuous-Integration: Jenkins CI --- M vdsm/storage/hsm.py 1 file changed, 12 insertions(+), 1 deletion(-) Approvals: Nir Soffer: Looks good to me, approved Adam Litke: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/62576 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I367576fca126be695d7617477f92110941620a3b Gerrit-PatchSet: 7 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Adam Litke <ali...@redhat.com> Gerrit-Reviewer: Adam Litke <ali...@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com> Gerrit-Reviewer: gerrit-hooks <automat...@ovirt.org> _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org