Change in vdsm[ovirt-3.5]: netconf: own /var/run/vdsm/netconf as well

danken at redhat.com danken at redhat.com
Tue Dec 29 08:07:23 UTC 2015


Dan Kenigsberg has submitted this change and it was merged.

Change subject: netconf: own /var/run/vdsm/netconf as well
......................................................................


netconf: own /var/run/vdsm/netconf as well

commit 1fa3b3 made /var/run/vdsm/netconf/nets owned by vdsm, but
sometimes we have /var/run/vdsm/netconf/ still owned by root.

Bug-Url: https://bugzilla.redhat.com/1294075
Change-Id: I94191063b798ce61660a2868010fa9654ce48c08
Signed-off-by: Tolik Litovsky <tlitovsk at redhat.com>
Signed-off-by: Dan Kenigsberg <danken at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/51133
Continuous-Integration: Jenkins CI
Reviewed-by: Edward Haas <edwardh at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/51104
---
M lib/vdsm/netconfpersistence.py
1 file changed, 1 insertion(+), 0 deletions(-)

Approvals:
  Dan Kenigsberg: Looks good to me, approved; Passed CI tests
  Anatoly Litovsky: Verified; Looks good to me, but someone else must approve

Objections:
  Jenkins CI: Failed CI tests



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I94191063b798ce61660a2868010fa9654ce48c08
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.5
Gerrit-Owner: Anatoly Litovsky <tlitovsk at redhat.com>
Gerrit-Reviewer: Anatoly Litovsky <tlitovsk at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Edward Haas <edwardh at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>


More information about the vdsm-patches mailing list