Change in vdsm[master]: fix crash when sync with libvirt network

lvroyce at linux.vnet.ibm.com lvroyce at linux.vnet.ibm.com
Tue Aug 14 06:23:39 UTC 2012


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 at linux.vnet.ibm.com>
Gerrit-Reviewer: Igor Lvovsky <ilvovsky at redhat.com>
Gerrit-Reviewer: Royce Lv <lvroyce at linux.vnet.ibm.com>
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list