)]}'
{"magnum/api/controllers/root.py":[{"author":{"_account_id":7230,"name":"Vijendar Komalla","email":"vijendar.komalla@rackspace.com","username":"vijendar"},"change_message_id":"455c3cac4c8192df91b5137a8ec27cb1945d833d","unresolved":false,"context_lines":[{"line_number":69,"context_line":"        root.name \u003d \"OpenStack Magnum API\""},{"line_number":70,"context_line":"        root.description \u003d (\"Magnum is an OpenStack project which aims to \""},{"line_number":71,"context_line":"                            \"provide container management.\")"},{"line_number":72,"context_line":"        root.versions \u003d [Version.convert(\u0027v1\u0027, \"CURRENT\", \"1.1\", \"1.1\")]"},{"line_number":73,"context_line":"        return root"},{"line_number":74,"context_line":""},{"line_number":75,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"1aa78d24_b40bcb7c","line":72,"range":{"start_line":72,"start_character":59,"end_line":72,"end_character":60},"updated":"2016-07-08 22:04:54.000000000","message":"should max,min version be 1.0 instead of 1.1?","commit_id":"deb10d32a9d57f4b8b2aa370a4ddfc349b41060c"},{"author":{"_account_id":7230,"name":"Vijendar Komalla","email":"vijendar.komalla@rackspace.com","username":"vijendar"},"change_message_id":"51d60fa50747a05f915a774fa65f6977e34d4e88","unresolved":false,"context_lines":[{"line_number":69,"context_line":"        root.name \u003d \"OpenStack Magnum API\""},{"line_number":70,"context_line":"        root.description \u003d (\"Magnum is an OpenStack project which aims to \""},{"line_number":71,"context_line":"                            \"provide container management.\")"},{"line_number":72,"context_line":"        root.versions \u003d [Version.convert(\u0027v1\u0027, \"CURRENT\", \"1.1\", \"1.1\")]"},{"line_number":73,"context_line":"        return root"},{"line_number":74,"context_line":""},{"line_number":75,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"1aa78d24_2a0bafd7","line":72,"range":{"start_line":72,"start_character":59,"end_line":72,"end_character":60},"in_reply_to":"1aa78d24_03a8a6da","updated":"2016-07-11 18:49:56.000000000","message":"makes sense! Thanks Jaycen.","commit_id":"deb10d32a9d57f4b8b2aa370a4ddfc349b41060c"},{"author":{"_account_id":21469,"name":"Jaycen Grant","email":"jaycen.v.grant@intel.com","username":"jvgrant"},"change_message_id":"8494c7e5cb5541b369d973752e14507293c3b56c","unresolved":false,"context_lines":[{"line_number":69,"context_line":"        root.name \u003d \"OpenStack Magnum API\""},{"line_number":70,"context_line":"        root.description \u003d (\"Magnum is an OpenStack project which aims to \""},{"line_number":71,"context_line":"                            \"provide container management.\")"},{"line_number":72,"context_line":"        root.versions \u003d [Version.convert(\u0027v1\u0027, \"CURRENT\", \"1.1\", \"1.1\")]"},{"line_number":73,"context_line":"        return root"},{"line_number":74,"context_line":""},{"line_number":75,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"1aa78d24_03a8a6da","line":72,"range":{"start_line":72,"start_character":59,"end_line":72,"end_character":60},"in_reply_to":"1aa78d24_b40bcb7c","updated":"2016-07-08 22:10:04.000000000","message":"The base version when microversioning was added was decided to be 1.1. 1.0 was reserved for kilo. There is documentation and a comment on this in the at the top of the controller/v1/__init__.py file","commit_id":"deb10d32a9d57f4b8b2aa370a4ddfc349b41060c"}],"magnum/api/middleware/parsable_error.py":[{"author":{"_account_id":21660,"name":"Stephen Watson","email":"stephen.watson@intel.com","username":"swatson"},"change_message_id":"bd4a27ae6eb5511535facbcc663985242d1ebbe9","unresolved":false,"context_lines":[{"line_number":38,"context_line":"            except ValueError:"},{"line_number":39,"context_line":"                pass"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"            if \u0027title\u0027 in err and \u0027description\u0027 in err:"},{"line_number":42,"context_line":"                title \u003d err[\u0027title\u0027]"},{"line_number":43,"context_line":"                desc \u003d err[\u0027description\u0027]"},{"line_number":44,"context_line":"            elif \u0027faultstring\u0027 in err:"}],"source_content_type":"text/x-python","patch_set":3,"id":"1aa78d24_7e5dc2eb","line":41,"updated":"2016-07-08 20:22:04.000000000","message":"Could one of \u0027title\u0027 or \u0027description\u0027 ever be in err without the other? i.e. only \u0027title\u0027 or only \u0027description\u0027","commit_id":"deb10d32a9d57f4b8b2aa370a4ddfc349b41060c"},{"author":{"_account_id":21469,"name":"Jaycen Grant","email":"jaycen.v.grant@intel.com","username":"jvgrant"},"change_message_id":"8d32a559779a13c14315b86c2632db61375faeb1","unresolved":false,"context_lines":[{"line_number":38,"context_line":"            except ValueError:"},{"line_number":39,"context_line":"                pass"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"            if \u0027title\u0027 in err and \u0027description\u0027 in err:"},{"line_number":42,"context_line":"                title \u003d err[\u0027title\u0027]"},{"line_number":43,"context_line":"                desc \u003d err[\u0027description\u0027]"},{"line_number":44,"context_line":"            elif \u0027faultstring\u0027 in err:"}],"source_content_type":"text/x-python","patch_set":3,"id":"1aa78d24_3ef21a9b","line":41,"in_reply_to":"1aa78d24_7e5dc2eb","updated":"2016-07-08 20:28:27.000000000","message":"It is always title and description from an http error or the faultstring from an exception.  This code remains the same as before just moved up into this method to lower complexity.","commit_id":"deb10d32a9d57f4b8b2aa370a4ddfc349b41060c"}]}
