Dan Kenigsberg has posted comments on this change.
Change subject: gluster: VDSM Gluster verbs to manage UFO/swift configuration ......................................................................
Patch Set 14: I would prefer that you didn't submit this
(1 inline comment)
.................................................... File vdsm/gluster/cli.py Line 920: try: Line 921: config.readfp(open(config_file)) Line 922: except IOError as e: Line 923: errMsg = "%s: %s" % (e.errno, e.strerror) Line 924: raise ge.GlusterSwiftConfigOpenFailedException(err=[errMsg]) please avoid str() in favor of unicode(). str() explodes on ಕನ್ನಡ characters. Line 925: Line 926: return config Line 927: Line 928:
-- 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: 14 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Aravinda VK avishwan@redhat.com Gerrit-Reviewer: Aravinda VK avishwan@redhat.com Gerrit-Reviewer: Bala.FA barumuga@redhat.com Gerrit-Reviewer: Dan Kenigsberg danken@redhat.com Gerrit-Reviewer: Federico Simoncelli fsimonce@redhat.com Gerrit-Reviewer: Saggi Mizrahi smizrahi@redhat.com Gerrit-Reviewer: Shireesh Anjal sanjal@redhat.com Gerrit-Reviewer: oVirt Jenkins CI Server