Change in vdsm[master]: upgrade: make sure that the lease files have correct permiss...

danken at redhat.com danken at redhat.com
Tue Nov 27 20:23:28 UTC 2012


Dan Kenigsberg has submitted this change and it was merged.

Change subject: upgrade: make sure that the lease files have correct permission
......................................................................


upgrade: make sure that the lease files have correct permission

Fix a bug where we tried to initialize sanlock lock before calling
the function to set proper permissions.

Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=870024
Change-Id: If28c48d2aeedd092ff3b8100dd64cd8ba673e94f
Signed-off-by: Haim Ateya <hateya at redhat.com>
---
M vdsm/storage/imageRepository/formatConverter.py
1 file changed, 4 insertions(+), 3 deletions(-)

Approvals:
  Yaniv Bronhaim: Looks good to me, but someone else must approve
  Saggi Mizrahi: Looks good to me, but someone else must approve
  Haim Ateya: Verified
  Federico Simoncelli: Looks good to me, approved
  Dan Kenigsberg: 


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

Gerrit-MessageType: merged
Gerrit-Change-Id: If28c48d2aeedd092ff3b8100dd64cd8ba673e94f
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Haim Ateya <hateya at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Haim Ateya <hateya at redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list