)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"e62eba0b968d3ad7ab7d6b3bea10cda9422876af","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Use ConfigParser instead of SafeConfigParser"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The SafeConfigParser class has been renamed to ConfigParser in Python"},{"line_number":10,"context_line":"3.2 [1]. This alias will be removed in future versions.So we can use"},{"line_number":11,"context_line":"ConfigParser directly instead."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: Ie0292271229ee77d1bc57dd897ebbbb3d3c059aa"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"7a8ec9b2_72d4871f","line":10,"updated":"2016-09-21 16:11:02.000000000","message":"what does [1] refer to?","commit_id":"19476931b5273c336fe9b3b490522e456cc49f35"}],"tests/functional/test_swiftclient.py":[{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"666ae61d4127d54f89c5982639f217385effe6d8","unresolved":false,"context_lines":[{"line_number":42,"context_line":"    def _get_config(self):"},{"line_number":43,"context_line":"        config_file \u003d os.environ.get(\u0027SWIFT_TEST_CONFIG_FILE\u0027,"},{"line_number":44,"context_line":"                                     \u0027/etc/swift/test.conf\u0027)"},{"line_number":45,"context_line":"        config \u003d configparser.ConfigParser({\u0027auth_version\u0027: \u00271\u0027})"},{"line_number":46,"context_line":"        config.read(config_file)"},{"line_number":47,"context_line":"        self.config \u003d config"},{"line_number":48,"context_line":"        if config.has_section(\u0027func_test\u0027):"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a8ec9b2_68cb331f","line":45,"updated":"2016-09-19 23:13:38.000000000","message":"From the py2 docs, SafeConfigParser is like ConfigParser, except that it\n\n\u003e implements a more-sane variant of the magical interpolation feature.\n\nAny idea what this means in practice?\n\nGiven that it\u0027s for a testing conf I\u0027m not too worried about breaking compatibility, but I wonder if it\u0027s worth switching to RawConfigParser and disabling interpolation all together.\n\nExcept the py3 docs refer to it as \"legacy\", so we may need some version check to switch between that and ConfigParser(interpolation\u003dNone), lest they mark *that* one as deprecated too. Ugh.\n\nSee https://docs.python.org/2/library/configparser.html#ConfigParser.SafeConfigParser and https://docs.python.org/3/library/configparser.html#configparser.RawConfigParser","commit_id":"19476931b5273c336fe9b3b490522e456cc49f35"},{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"e62eba0b968d3ad7ab7d6b3bea10cda9422876af","unresolved":false,"context_lines":[{"line_number":42,"context_line":"    def _get_config(self):"},{"line_number":43,"context_line":"        config_file \u003d os.environ.get(\u0027SWIFT_TEST_CONFIG_FILE\u0027,"},{"line_number":44,"context_line":"                                     \u0027/etc/swift/test.conf\u0027)"},{"line_number":45,"context_line":"        config \u003d configparser.ConfigParser({\u0027auth_version\u0027: \u00271\u0027})"},{"line_number":46,"context_line":"        config.read(config_file)"},{"line_number":47,"context_line":"        self.config \u003d config"},{"line_number":48,"context_line":"        if config.has_section(\u0027func_test\u0027):"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a8ec9b2_f2fa57bc","line":45,"in_reply_to":"7a8ec9b2_68cb331f","updated":"2016-09-21 16:11:02.000000000","message":"No idea, but what I noticed is that the \"Safe\" is requiring option values to be strings when using the set() method\n\nhttps://docs.python.org/2.7/library/configparser.html#safeconfigparser-objects","commit_id":"19476931b5273c336fe9b3b490522e456cc49f35"}]}