Royce Lv has posted comments on this change.
Change subject: fix crash when sync with libvirt network ......................................................................
Patch Set 1: I would prefer that you didn't submit this
(2 inline comments)
.................................................... File vdsm/configNetwork.py Line 218: Line 219: # make directory (if it doesn't exist) and assign it to vdsm Line 220: if not os.path.exists(dirName): Line 221: os.makedirs(dirName) Line 222: os.chown(dirName, vdsm_uid, vdsm_gid) Actually, we are little confused why we need to chown here,it is vdsmd who create this directory Line 223: Line 224: open(backup, 'w').write(content) Line 225: os.chown(backup, vdsm_uid, 0) Line 226: logging.debug("Persistently backed up %s "
Line 221: os.makedirs(dirName) Line 222: os.chown(dirName, vdsm_uid, vdsm_gid) Line 223: Line 224: open(backup, 'w').write(content) Line 225: os.chown(backup, vdsm_uid, 0) and this one ,too Line 226: logging.debug("Persistently backed up %s " Line 227: "(until next 'set safe config')", backup) Line 228: Line 229: def _networkBackup(self, network):
-- To view, visit http://gerrit.ovirt.org/7156 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment Gerrit-Change-Id: I4a3ea0303656cba87bf17b14929fe59bf0d46337 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Zhou Zheng Sheng zhshzhou@linux.vnet.ibm.com Gerrit-Reviewer: Igor Lvovsky ilvovsky@redhat.com Gerrit-Reviewer: Royce Lv lvroyce@linux.vnet.ibm.com Gerrit-Reviewer: oVirt Jenkins CI Server