Sandro Bonazzola has posted comments on this change.

Change subject: packaging: setup: fix local state dirs ownership
......................................................................


Patch Set 2: (1 inline comment)

....................................................
File packaging/setup/plugins/ovirt-engine-setup/system/localstate.py
Line 64:                 followlinks=False,
Line 65:             ):
Line 66:                 for name in dirs + files:
Line 67:                     if os.stat(os.path.join(root, name)).st_uid != uid:
Line 68:                         rm_tmp_dir = True
Done
Line 69:             if rm_tmp_dir:
Line 70:                 self.logger.debug(
Line 71:                     'Cleaning {tmpdir}'.format(
Line 72:                         
tmpdir=osetupcons.FileLocations.OVIRT_ENGINE_TMP_DIR,


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I180fa0b1c3ded975c052405126819d91524a829f
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Sandro Bonazzola <sbona...@redhat.com>
Gerrit-Reviewer: Alex Lourie <alou...@redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alo...@redhat.com>
Gerrit-Reviewer: Ofer Schreiber <oschr...@redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbona...@redhat.com>
Gerrit-Reviewer: Yedidyah Bar David <d...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to