[node-patches] Change in ovirt-node[master]: make sure to only edit the Domain line in idmapd.conf

mburns at redhat.com mburns at redhat.com
Wed May 15 16:33:29 UTC 2013


Michael Burns has submitted this change and it was merged.

Change subject: make sure to only edit the Domain line in idmapd.conf
......................................................................


make sure to only edit the Domain line in idmapd.conf

previously, if you set the domain to some value that is found
elsewhere in the idmapd.conf file, and then updated again, it would
replace all occurences of the value in the file.

This is especially bad if you do something like "nobody" since that
changes the user that nfsv4 runs as.

rhbz#962737

Change-Id: I6ceda931405e6c55152ac9044ff982c78095ad8b
Signed-off-by: Mike Burns <mburns at redhat.com>
---
M src/ovirtnode/network.py
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Michael Burns: Verified; Looks good to me, approved
  Ryan Barry: Looks good to me, but someone else must approve


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I6ceda931405e6c55152ac9044ff982c78095ad8b
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-node
Gerrit-Branch: master
Gerrit-Owner: Michael Burns <mburns at redhat.com>
Gerrit-Reviewer: Fabian Deutsch <fabiand at fedoraproject.org>
Gerrit-Reviewer: Michael Burns <mburns at redhat.com>
Gerrit-Reviewer: Ryan Barry <rbarry at redhat.com>



More information about the node-patches mailing list