Alon Bar-Lev 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
break?
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 <[email protected]>
Gerrit-Reviewer: Alex Lourie <[email protected]>
Gerrit-Reviewer: Alon Bar-Lev <[email protected]>
Gerrit-Reviewer: Ofer Schreiber <[email protected]>
Gerrit-Reviewer: Sandro Bonazzola <[email protected]>
Gerrit-Reviewer: Yedidyah Bar David <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to