Dan Kenigsberg has submitted this change and it was merged. Change subject: net: netconf owner change should ignore gid ......................................................................
net: netconf owner change should ignore gid When changing ownership of the directories/files the group id does not need to change. It affects tests which run as non-root (they fail when attempting to change the gid to root (0). Change-Id: I059a443d39122490f085d4538c9adf353e17563e Signed-off-by: Edward Haas <edwa...@redhat.com> Reviewed-on: https://gerrit.ovirt.org/51148 Reviewed-by: Nir Soffer <nsof...@redhat.com> Continuous-Integration: Jenkins CI Reviewed-by: Dan Kenigsberg <dan...@redhat.com> Tested-by: Dan Kenigsberg <dan...@redhat.com> (cherry picked from commit e98cc411ac055f1e76e039237db2941883bcc52c) Reviewed-on: https://gerrit.ovirt.org/51149 --- M lib/vdsm/netconfpersistence.py 1 file changed, 3 insertions(+), 3 deletions(-) Approvals: Jenkins CI: Passed CI tests Dan Kenigsberg: Verified; Looks good to me, approved Edward Haas: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/51149 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I059a443d39122490f085d4538c9adf353e17563e Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: ovirt-3.6 Gerrit-Owner: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: Edward Haas <edwa...@redhat.com> Gerrit-Reviewer: Jenkins CI _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches