Change in vdsm[master]: Adding sudo rules to multipath and removing unused rules

danken at redhat.com danken at redhat.com
Tue Feb 26 15:51:09 UTC 2013


Dan Kenigsberg has submitted this change and it was merged.

Change subject: Adding sudo rules to multipath and removing unused rules
......................................................................


Adding sudo rules to multipath and removing unused rules

When vdsm tries to cp multipath.conf file as part of setupMultipath that
tries to keep the last multipath.conf file into multipath.conf.1, we get an
error that sudo command requires a password.

This patch add rules to allow operations on absolute path of multipath.conf
file.

Removing old code that related to directories that vdsm doesn't use anymore.

Bud-Id: https://bugzilla.redhat.com/show_bug.cgi?id=907389
Change-Id: Ie732400423f12b0a783bd251b4e2cc2e2409ffec
Signed-off-by: Pavel Zhukov <pzhukov at redhat.com>
Signed-off-by: Yaniv Bronhaim <ybronhei at redhat.com>
---
M vdsm/sudoers.vdsm.in
1 file changed, 4 insertions(+), 7 deletions(-)

Approvals:
  Yaniv Bronhaim: Verified
  Dan Kenigsberg: Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ie732400423f12b0a783bd251b4e2cc2e2409ffec
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Pavel Zhukov <pzhukov at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Pavel Zhukov <pzhukov 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