Change in vdsm[master]: gluster: VDSM Gluster verbs to manage UFO/swift configuratio...
barumuga at redhat.com
barumuga at redhat.com
Thu Mar 21 15:57:43 UTC 2013
Bala.FA has posted comments on this change.
Change subject: gluster: VDSM Gluster verbs to manage UFO/swift configuration
......................................................................
Patch Set 17: I would prefer that you didn't submit this
(5 inline comments)
Sorry. Some review could have be done much early.
....................................................
File vdsm/gluster/swift.py
Line 43: elif configOption in items:
Line 44: return {configOption: items[configOption]}
Line 45: else:
Line 46: errMsg = "%s: Invalid Swift Config option" % configOption
Line 47: raise ge.GlusterSwiftConfigInvalidOptionException(err=[errMsg])
Can you pass configOption to GlusterSwiftConfigInvalidOptionException() and form the error message there?
Line 48:
Line 49:
Line 50: def _openSwiftConfigFile(serverType):
Line 51: config = ConfigParser()
Line 50: def _openSwiftConfigFile(serverType):
Line 51: config = ConfigParser()
Line 52: if serverType not in SWIFT_CONFIG_FILES:
Line 53: errMsg = "%s: Invalid Swift Server type" % serverType
Line 54: raise ge.GlusterSwiftConfigInvalidServerException(err=[errMsg])
same here
Line 55:
Line 56: config_file = SWIFT_CONFIG_FILES[serverType]
Line 57: try:
Line 58: config.readfp(open(config_file))
Line 57: try:
Line 58: config.readfp(open(config_file))
Line 59: except IOError as e:
Line 60: errMsg = "[Errno %s] %s: '%s'" % (e.errno, e.strerror, e.filename)
Line 61: raise ge.GlusterSwiftConfigOpenFailedException(err=[errMsg])
How about err=unicode(e)?
Line 62:
Line 63: return config
Line 64:
Line 65:
Line 78: # mail.python.org/pipermail/python-list/2005-February/342893.html
Line 79: shutil.move(tempConfigFile.name, config_file)
Line 80: except IOError as e:
Line 81: errMsg = "[Errno %s] %s: '%s'" % (e.errno, e.strerror, e.filename)
Line 82: raise ge.GlusterSwiftConfigWriteFailedException(err=[errMsg])
same here
Line 83:
Line 84:
Line 85: def _configGet(serverType, section=None, configOption=None):
Line 86: """
Line 111: configOption)
Line 112: elif section:
Line 113: if not config.has_section(section):
Line 114: errMsg = "%s: Invalid Swift Config Section" % section
Line 115: raise ge.GlusterSwiftConfigInvalidSectionException(err=[errMsg])
construct error message inside the exception
Line 116:
Line 117: items = dict(config.items(section))
Line 118: configValues[section] = _getConfigOptions(items, configOption)
Line 119: else:
--
To view, visit http://gerrit.ovirt.org/10864
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ie966fb515275a0768f67cbbe2055a07002355327
Gerrit-PatchSet: 17
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Aravinda VK <avishwan at redhat.com>
Gerrit-Reviewer: Aravinda VK <avishwan at redhat.com>
Gerrit-Reviewer: Bala.FA <barumuga at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi at redhat.com>
Gerrit-Reviewer: Shireesh Anjal <sanjal at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
More information about the vdsm-patches
mailing list