Change in vdsm[master]: hsm: refresh pool connection on connectStoragePool

danken at redhat.com danken at redhat.com
Thu Dec 19 12:50:51 UTC 2013


Dan Kenigsberg has submitted this change and it was merged.

Change subject: hsm: refresh pool connection on connectStoragePool
......................................................................


hsm: refresh pool connection on connectStoragePool

This patch ensures that connectStoragePool would trigger a refresh
when the host is already connected to the same pool.
Using connectStoragePool instead of refreshStoragePool is at the base
of the storage pool metadata removal but it also solves an existing
problem in the non-operational recovery flow where the engine sends
a connectStoragePool request.

Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1026697
Change-Id: I46d7c36b48f5d58edbc89947ccd2bcd60408a729
Signed-off-by: Federico Simoncelli <fsimonce at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/22467
Reviewed-by: Ayal Baron <abaron at redhat.com>
---
M vdsm/storage/hsm.py
1 file changed, 18 insertions(+), 19 deletions(-)

Approvals:
  Ayal Baron: Looks good to me, approved
  Federico Simoncelli: Verified



-- 
To view, visit http://gerrit.ovirt.org/22467
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I46d7c36b48f5d58edbc89947ccd2bcd60408a729
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Sergey Gotliv <sgotliv at redhat.com>
Gerrit-Reviewer: Vered Volansky <vvolansk at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list