)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":14274,"name":"XinXiaohui","email":"xiaohui.xin@intel.com","username":"XinXiaohui"},"change_message_id":"be75407abec2101bc9211805bdb6eeaa927f38dd","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Use the catalog to retrieve glance_api_servers"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Currently the config option glance_api_servers must be"},{"line_number":10,"context_line":"set to give the correct list of glance API URLs. This"},{"line_number":11,"context_line":"patch enables the use of the catalog to find the glance"},{"line_number":12,"context_line":"endpoint when this config option is set to None, and"},{"line_number":13,"context_line":"changes its default to None as well."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"7a740942_c1872db3","line":10,"updated":"2015-12-09 00:23:04.000000000","message":"nits: set to give the correct list -\u003e set to a given correct","commit_id":"3aa3238cd0323ea8fc4cd4a2e5c1f990d87bfc04"}],"cinder/image/glance.py":[{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"a9f3a0a303a0aa12e2744897df12e3e5ddba7a5c","unresolved":false,"context_lines":[{"line_number":50,"context_line":"               help\u003d\u0027Info to match when looking for glance in the service \u0027"},{"line_number":51,"context_line":"                    \u0027catalog. Format is: separated values of the form: \u0027"},{"line_number":52,"context_line":"                    \u0027\u003cservice_type\u003e:\u003cservice_name\u003e:\u003cendpoint_type\u003e - \u0027"},{"line_number":53,"context_line":"                    \u0027Only used if glance_api_servers are not provided\u0027),"},{"line_number":54,"context_line":"]"},{"line_number":55,"context_line":"glance_core_properties_opts \u003d ["},{"line_number":56,"context_line":"    cfg.ListOpt(\u0027glance_core_properties\u0027,"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a8ffd7b_04602dce","line":53,"updated":"2015-12-02 18:41:57.000000000","message":"Suggest \u0027.\u0027 at EOL to be consistent with other opts.","commit_id":"4317b918975cc892e6e2efc31db01512d414fabd"},{"author":{"_account_id":17275,"name":"Nate Potter","email":"nathaniel.potter@intel.com","username":"npotter"},"change_message_id":"8a2ad59d4c3bceebba489452c84dcf3a4a1d1607","unresolved":false,"context_lines":[{"line_number":50,"context_line":"               help\u003d\u0027Info to match when looking for glance in the service \u0027"},{"line_number":51,"context_line":"                    \u0027catalog. Format is: separated values of the form: \u0027"},{"line_number":52,"context_line":"                    \u0027\u003cservice_type\u003e:\u003cservice_name\u003e:\u003cendpoint_type\u003e - \u0027"},{"line_number":53,"context_line":"                    \u0027Only used if glance_api_servers are not provided\u0027),"},{"line_number":54,"context_line":"]"},{"line_number":55,"context_line":"glance_core_properties_opts \u003d ["},{"line_number":56,"context_line":"    cfg.ListOpt(\u0027glance_core_properties\u0027,"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a8ffd7b_eab5abec","line":53,"in_reply_to":"9a8ffd7b_04602dce","updated":"2015-12-02 20:57:09.000000000","message":"Done","commit_id":"4317b918975cc892e6e2efc31db01512d414fabd"},{"author":{"_account_id":11903,"name":"Anthony Lee","email":"anthony.mic.lee@hpe.com","username":"leeantho"},"change_message_id":"139d85f20f05bf57b626909c919bd528382d0e66","unresolved":false,"context_lines":[{"line_number":119,"context_line":"        try:"},{"line_number":120,"context_line":"            service_type, service_name, endpoint_type \u003d info.split(\u0027:\u0027)"},{"line_number":121,"context_line":"        except ValueError:"},{"line_number":122,"context_line":"            raise exception.InvalidConfigurationOption(_("},{"line_number":123,"context_line":"                \"Failed to parse the configuration option \""},{"line_number":124,"context_line":"                \"\u0027glance_catalog_info\u0027, must be in the form \""},{"line_number":125,"context_line":"                \"\u003cservice_type\u003e:\u003cservice_name\u003e:\u003cendpoint_type\u003e\"))"}],"source_content_type":"text/x-python","patch_set":4,"id":"7a740942_1a7edae4","line":122,"updated":"2015-12-03 18:55:33.000000000","message":"The variable \u0027_\u0027 is not defined anywhere. It needs to be imported from cinder.i18n above on line 39.\n\nAlso InvalidConfigurationOption does not exist in cinder.exceptions.\n\nThere is an InvalidConfigurationValue exception that can be used.","commit_id":"497efb0d7b4637a181901718e0e6b32320bb3743"},{"author":{"_account_id":17275,"name":"Nate Potter","email":"nathaniel.potter@intel.com","username":"npotter"},"change_message_id":"025878b6968d80cde4b014c8aee62c5bbdd84f3e","unresolved":false,"context_lines":[{"line_number":119,"context_line":"        try:"},{"line_number":120,"context_line":"            service_type, service_name, endpoint_type \u003d info.split(\u0027:\u0027)"},{"line_number":121,"context_line":"        except ValueError:"},{"line_number":122,"context_line":"            raise exception.InvalidConfigurationOption(_("},{"line_number":123,"context_line":"                \"Failed to parse the configuration option \""},{"line_number":124,"context_line":"                \"\u0027glance_catalog_info\u0027, must be in the form \""},{"line_number":125,"context_line":"                \"\u003cservice_type\u003e:\u003cservice_name\u003e:\u003cendpoint_type\u003e\"))"}],"source_content_type":"text/x-python","patch_set":4,"id":"7a740942_dcaa8f85","line":122,"in_reply_to":"7a740942_1a7edae4","updated":"2015-12-03 19:03:09.000000000","message":"Thanks, fixed that in my latest patch set.","commit_id":"497efb0d7b4637a181901718e0e6b32320bb3743"},{"author":{"_account_id":9171,"name":"Vipin Balachandran","email":"vipin.bl@gmail.com","username":"vbala"},"change_message_id":"b48140926ba7ff1e564f6222bca8bbc5014e46cb","unresolved":false,"context_lines":[{"line_number":117,"context_line":"    if CONF.glance_api_servers is None:"},{"line_number":118,"context_line":"        info \u003d CONF.glance_catalog_info"},{"line_number":119,"context_line":"        try:"},{"line_number":120,"context_line":"            service_type, service_name, endpoint_type \u003d info.split(\u0027:\u0027)"},{"line_number":121,"context_line":"        except ValueError:"},{"line_number":122,"context_line":"            raise exception.InvalidConfigurationValue(_("},{"line_number":123,"context_line":"                \"Failed to parse the configuration option \""}],"source_content_type":"text/x-python","patch_set":5,"id":"1af94dfe_31d221d3","line":120,"range":{"start_line":120,"start_character":26,"end_line":120,"end_character":38},"updated":"2016-03-16 17:26:34.000000000","message":"Looks like it is unused. You may want to prefix it with underscore.","commit_id":"3aa3238cd0323ea8fc4cd4a2e5c1f990d87bfc04"}]}