Change in vdsm[master]: mkimage: make sure proper owner/group is set on payload dire...

[Code Review]

From Dan Kenigsberg <danken(a)redhat.com&gt;:

Dan Kenigsberg has submitted this change and it was merged.

Change subject: mkimage: make sure proper owner/group is set on payload directory
......................................................................


mkimage: make sure proper owner/group is set on payload directory

A minor fix to the creation of payload directory. In RPM package the
directory has vdsm/qemu as owner/group, but when the directory is
deleted and later recreated by supervdsm, owner/group remained set to
root. It is unlikely that this could have caused any real issues as only
supervdsm writes into that directory at the moment.

Change-Id: I26b03c250954d3fbd89292d962eb3329a50bafe5
Signed-off-by: Tomáš Golembiovský <tgolembi(a)redhat.com&gt;
---
M lib/vdsm/mkimage.py
1 file changed, 3 insertions(+), 0 deletions(-)

Approvals:
  Jenkins CI: Passed CI tests
  Francesco Romani: Looks good to me, approved
  Tomas Golembiovsky: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I26b03c250954d3fbd89292d962eb3329a50bafe5
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Tomas Golembiovsky <tgolembi(a)redhat.com&gt;
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com&gt;
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com&gt;
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Tomas Golembiovsky <tgolembi(a)redhat.com&gt;
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org&gt;