)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"9b57cd82657923b7450c18dae9cf8851e63b401e","unresolved":false,"context_lines":[{"line_number":16,"context_line":"make sure that the list can still run even if the api_version check is"},{"line_number":17,"context_line":"unauthorized."},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Closes-Bug: #1493974"},{"line_number":20,"context_line":"Change-Id: I038b84bad5b747a0688aef989f1337aee835b945"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"da20952f_54361db5","line":19,"updated":"2015-09-09 19:23:39.000000000","message":"I ended up re-opening the original bug so we keep the patches together, so could you please update this to \"Closes-Bug #1491579\"","commit_id":"61e11cfafb7e73eaa4f6c872901785b0ce3bbb9c"}],"novaclient/v2/versions.py":[{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"9b57cd82657923b7450c18dae9cf8851e63b401e","unresolved":false,"context_lines":[{"line_number":81,"context_line":"            return self._list(version_url, \"versions\")"},{"line_number":82,"context_line":"        except exc.Unauthorized:"},{"line_number":83,"context_line":"            # NOTE(gtmanfred): see note above in get_current for RAX"},{"line_number":84,"context_line":"            return []"}],"source_content_type":"text/x-python","patch_set":6,"id":"da20952f_74d2415d","line":84,"updated":"2015-09-09 19:23:39.000000000","message":"I think instead of this, we should fix the existing workaround in the get_current() function. Right now it\u0027s try-catching only inside _is_session_client() and we can just move it outside for example:\n\n try:\n     if self._is_session_client():\n     else:\n except exc.Unauthorized:","commit_id":"61e11cfafb7e73eaa4f6c872901785b0ce3bbb9c"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"30f92e415621bafad398ef9b2b37c8a40d4146c5","unresolved":false,"context_lines":[{"line_number":64,"context_line":"            # definitely non-compliant behavior. However, there is"},{"line_number":65,"context_line":"            # no defcore test for this yet. Remove this code block"},{"line_number":66,"context_line":"            # once we land things in defcore."},{"line_number":67,"context_line":"            return None"},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"    def list(self):"},{"line_number":70,"context_line":"        \"\"\"List all versions.\"\"\""}],"source_content_type":"text/x-python","patch_set":7,"id":"da20952f_1443c56d","line":67,"updated":"2015-09-09 19:40:09.000000000","message":"I guess we could have renamed s/get_current/_get_current/ and\n\n def get_current(self):\n     try:\n         return self._get_current()\n     except exc.Unauthorized:\n\nbut I didn\u0027t think of it until now. I think it\u0027s fine either way","commit_id":"d14726f5f57f69d7e78a5ce56d26af2e7c414e3d"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"5a6d5b4a5fa85938096875aa71e8dafb80b4b41d","unresolved":false,"context_lines":[{"line_number":59,"context_line":""},{"line_number":60,"context_line":"    def get_current(self):"},{"line_number":61,"context_line":"        try:"},{"line_number":62,"context_line":"            self._get_current()"},{"line_number":63,"context_line":"        except exc.Unauthorized:"},{"line_number":64,"context_line":"            # NOTE(sdague): RAX\u0027s repose configuration blocks access to the"},{"line_number":65,"context_line":"            # versioned endpoint, which is definitely non-compliant behavior."}],"source_content_type":"text/x-python","patch_set":9,"id":"da20952f_ef90b898","line":62,"updated":"2015-09-09 19:46:22.000000000","message":"I think you need to return this","commit_id":"4739006f95d1c6c6c7c0a3e685b0642cd1dce261"}]}
