Change in vdsm[master]: multipath: Fix multipath.conf permissions

danken at redhat.com danken at redhat.com
Mon Aug 17 07:01:09 UTC 2015


Dan Kenigsberg has submitted this change and it was merged.

Change subject: multipath: Fix multipath.conf permissions
......................................................................


multipath: Fix multipath.conf permissions

/etc/multipath.conf permissions are 0644 before vdsm configures it.
After configuration, the permissions become 0600. This is unintended
change caused by using temporary file, which is created using 0600.
Now we set the permissions before copying the temporary file.

Change-Id: Ib9205b59eb989325bef00d5e34247f7c74e4e587
Signed-off-by: Nir Soffer <nsoffer at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/44868
Continuous-Integration: Jenkins CI
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
---
M lib/vdsm/tool/configurators/multipath.py
1 file changed, 1 insertion(+), 0 deletions(-)

Approvals:
  Nir Soffer: Verified
  Jenkins CI: Passed CI tests
  Dan Kenigsberg: Looks good to me, approved



-- 
To view, visit https://gerrit.ovirt.org/44868
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ib9205b59eb989325bef00d5e34247f7c74e4e587
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski at gmail.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: Yeela Kaplan <ykaplan at redhat.com>
Gerrit-Reviewer: automation at ovirt.org


More information about the vdsm-patches mailing list