)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5202,"name":"Erno Kuvaja","email":"jokke@usr.fi","username":"jokke"},"change_message_id":"b834a03064f1c21ed13d2b5569f82236273c6311","unresolved":false,"context_lines":[{"line_number":12,"context_line":"[1]: https://review.openstack.org/#/c/209536/3"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: I2ab94d8768114a7e5c475310ba094af653627658"},{"line_number":15,"context_line":"Related-Bug: #1481729"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"9a68dd71_825fc20d","line":15,"updated":"2016-01-20 16:43:58.000000000","message":"Fix for this bug has been merged and the bug is marked as Fix Released.","commit_id":"77e907614dd0427f60557ec5d9d3a6088224657b"},{"author":{"_account_id":14676,"name":"Kairat Kushaev","email":"kkushaev@mirantis.com","username":"kairat_kushaev"},"change_message_id":"f4d04aded99a149344a545ba7db67b19ca2554a4","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Ship the default metadata schema in the client"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Now the help message of namesapce and resource_type are wrong,"},{"line_number":10,"context_line":"we could see a \u003cunavailable\u003e error when try to get the help"},{"line_number":11,"context_line":"information, such as \"glance help md-namespace-create\"."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"5a5ae5dd_14802ff4","line":9,"updated":"2016-02-08 10:37:05.000000000","message":"namespace","commit_id":"9d1116f14b31f7f17788d9a3062d0b64d09c714e"}],"glanceclient/v2/namespace_schema.py":[{"author":{"_account_id":14676,"name":"Kairat Kushaev","email":"kkushaev@mirantis.com","username":"kairat_kushaev"},"change_message_id":"f4d04aded99a149344a545ba7db67b19ca2554a4","unresolved":false,"context_lines":[{"line_number":160,"context_line":"            },"},{"line_number":161,"context_line":"            \"type\": \"array\""},{"line_number":162,"context_line":"        },"},{"line_number":163,"context_line":"        \"self\": {"},{"line_number":164,"context_line":"            \"type\": \"string\""},{"line_number":165,"context_line":"        },"},{"line_number":166,"context_line":"        \"namespace\": {"}],"source_content_type":"text/x-python","patch_set":4,"id":"5a5ae5dd_f78a3563","line":163,"updated":"2016-02-08 10:37:05.000000000","message":"I am wondering if this should be read-only. I see it is not read-only on the server side but it looks like we need to fix both client and server.","commit_id":"9d1116f14b31f7f17788d9a3062d0b64d09c714e"},{"author":{"_account_id":15054,"name":"wangxiyuan","email":"wangxiyuan1007@gmail.com","username":"wangxiyuan"},"change_message_id":"5efd0ce9d39e2dda92d80dba555c60b6a66a9637","unresolved":false,"context_lines":[{"line_number":160,"context_line":"            },"},{"line_number":161,"context_line":"            \"type\": \"array\""},{"line_number":162,"context_line":"        },"},{"line_number":163,"context_line":"        \"self\": {"},{"line_number":164,"context_line":"            \"type\": \"string\""},{"line_number":165,"context_line":"        },"},{"line_number":166,"context_line":"        \"namespace\": {"}],"source_content_type":"text/x-python","patch_set":4,"id":"dae33548_a7648e49","line":163,"in_reply_to":"5a5ae5dd_f78a3563","updated":"2016-02-14 02:17:06.000000000","message":"Agree.","commit_id":"9d1116f14b31f7f17788d9a3062d0b64d09c714e"},{"author":{"_account_id":14676,"name":"Kairat Kushaev","email":"kkushaev@mirantis.com","username":"kairat_kushaev"},"change_message_id":"f4d04aded99a149344a545ba7db67b19ca2554a4","unresolved":false,"context_lines":[{"line_number":171,"context_line":"        \"updated_at\": {"},{"line_number":172,"context_line":"            \"type\": \"string\","},{"line_number":173,"context_line":"            \"description\": \"Date and time of the last namespace modification \""},{"line_number":174,"context_line":"                           \"(READ-ONLY)\","},{"line_number":175,"context_line":"            \"format\": \"date-time\""},{"line_number":176,"context_line":"        },"},{"line_number":177,"context_line":"        \"visibility\": {"}],"source_content_type":"text/x-python","patch_set":4,"id":"5a5ae5dd_17e1112d","line":174,"updated":"2016-02-08 10:37:05.000000000","message":"Need to specify readOnly: True","commit_id":"9d1116f14b31f7f17788d9a3062d0b64d09c714e"}],"glanceclient/v2/resource_type_schema.py":[{"author":{"_account_id":14676,"name":"Kairat Kushaev","email":"kkushaev@mirantis.com","username":"kairat_kushaev"},"change_message_id":"f4d04aded99a149344a545ba7db67b19ca2554a4","unresolved":false,"context_lines":[{"line_number":24,"context_line":"    \"properties\": {"},{"line_number":25,"context_line":"        \"created_at\": {"},{"line_number":26,"context_line":"            \"type\": \"string\","},{"line_number":27,"context_line":"            \"description\": \"Date and time of resource type association\""},{"line_number":28,"context_line":"                           \"(READ-ONLY)\","},{"line_number":29,"context_line":"            \"format\": \"date-time\""},{"line_number":30,"context_line":"        },"}],"source_content_type":"text/x-python","patch_set":4,"id":"5a5ae5dd_f400f303","line":27,"updated":"2016-02-08 10:37:05.000000000","message":"Looks like readOnly must be specified in different way:\nhttps://bugs.launchpad.net/glance/+bug/1521581","commit_id":"9d1116f14b31f7f17788d9a3062d0b64d09c714e"},{"author":{"_account_id":14676,"name":"Kairat Kushaev","email":"kkushaev@mirantis.com","username":"kairat_kushaev"},"change_message_id":"f4d04aded99a149344a545ba7db67b19ca2554a4","unresolved":false,"context_lines":[{"line_number":58,"context_line":"            \"description\": \"Resource type names should be aligned with Heat \""},{"line_number":59,"context_line":"                           \"resource types whenever possible: http://docs.\""},{"line_number":60,"context_line":"                           \"openstack.org/developer/heat/template_guide/\""},{"line_number":61,"context_line":"                           \"openstack.html\", \"maxLength\": 80"},{"line_number":62,"context_line":"        }"},{"line_number":63,"context_line":"    }"},{"line_number":64,"context_line":"}"}],"source_content_type":"text/x-python","patch_set":4,"id":"5a5ae5dd_f7179542","line":61,"updated":"2016-02-08 10:37:05.000000000","message":"Not reason for -1 but looks like this deserves a new line\u003d)","commit_id":"9d1116f14b31f7f17788d9a3062d0b64d09c714e"},{"author":{"_account_id":14676,"name":"Kairat Kushaev","email":"kkushaev@mirantis.com","username":"kairat_kushaev"},"change_message_id":"a8f2e5f1b5f22a3ffd3b9ef9d069014b4b680ce1","unresolved":false,"context_lines":[{"line_number":17,"context_line":"# NOTE(flaper87): Keep a copy of the current default schema so that"},{"line_number":18,"context_line":"# we can react on cases where there\u0027s no connection to an OpenStack"},{"line_number":19,"context_line":"# deployment. See #1481729"},{"line_number":20,"context_line":"_BASE_SCHEMA \u003d {"},{"line_number":21,"context_line":"    \"additionalProperties\": False,"},{"line_number":22,"context_line":"    \"required\": [\"name\"],"},{"line_number":23,"context_line":"    \"name\": \"resource_type_association\","}],"source_content_type":"text/x-python","patch_set":8,"id":"bae84128_64d1fb22","line":20,"updated":"2016-02-24 13:11:37.000000000","message":"I guess we need to change the name to BASE_SCHEMA too.\nI am really sorry for be too strict on this\u003d).","commit_id":"b7be51d77c3a9bbfb07bc283c3e4dbb52e8db7a6"},{"author":{"_account_id":15054,"name":"wangxiyuan","email":"wangxiyuan1007@gmail.com","username":"wangxiyuan"},"change_message_id":"f3336719c50000d096971bd794d38c4ece798b91","unresolved":false,"context_lines":[{"line_number":17,"context_line":"# NOTE(flaper87): Keep a copy of the current default schema so that"},{"line_number":18,"context_line":"# we can react on cases where there\u0027s no connection to an OpenStack"},{"line_number":19,"context_line":"# deployment. See #1481729"},{"line_number":20,"context_line":"_BASE_SCHEMA \u003d {"},{"line_number":21,"context_line":"    \"additionalProperties\": False,"},{"line_number":22,"context_line":"    \"required\": [\"name\"],"},{"line_number":23,"context_line":"    \"name\": \"resource_type_association\","}],"source_content_type":"text/x-python","patch_set":8,"id":"bae84128_392a4ee8","line":20,"in_reply_to":"bae84128_64d1fb22","updated":"2016-02-25 01:33:02.000000000","message":"Done","commit_id":"b7be51d77c3a9bbfb07bc283c3e4dbb52e8db7a6"}],"glanceclient/v2/shell.py":[{"author":{"_account_id":14676,"name":"Kairat Kushaev","email":"kkushaev@mirantis.com","username":"kairat_kushaev"},"change_message_id":"f4d04aded99a149344a545ba7db67b19ca2554a4","unresolved":false,"context_lines":[{"line_number":444,"context_line":"                schema_raw \u003d f.read()"},{"line_number":445,"context_line":"                NAMESPACE_SCHEMA \u003d json.loads(schema_raw)"},{"line_number":446,"context_line":"        else:"},{"line_number":447,"context_line":"            return namespace_schema._BASE_SCHEMA"},{"line_number":448,"context_line":"    return NAMESPACE_SCHEMA"},{"line_number":449,"context_line":""},{"line_number":450,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"5a5ae5dd_f7fd55b1","line":447,"updated":"2016-02-08 10:37:05.000000000","message":"That\u0027s not a big trouble but I am wondering about the naming we applied to the schema variable.\nAFAIK, _SOME_PROPERTY_OR_CONSTANT it is naming for protected properties and variables but we are using it outside the module. So do we need to follow this naming (even PyCharm (yep, I am not using vim\u003d) complains me about that).","commit_id":"9d1116f14b31f7f17788d9a3062d0b64d09c714e"},{"author":{"_account_id":15054,"name":"wangxiyuan","email":"wangxiyuan1007@gmail.com","username":"wangxiyuan"},"change_message_id":"e9acf873a3d64104f890e64dc9188ef82765957d","unresolved":false,"context_lines":[{"line_number":444,"context_line":"                schema_raw \u003d f.read()"},{"line_number":445,"context_line":"                NAMESPACE_SCHEMA \u003d json.loads(schema_raw)"},{"line_number":446,"context_line":"        else:"},{"line_number":447,"context_line":"            return namespace_schema._BASE_SCHEMA"},{"line_number":448,"context_line":"    return NAMESPACE_SCHEMA"},{"line_number":449,"context_line":""},{"line_number":450,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"dae33548_c7d27234","line":447,"in_reply_to":"5a5ae5dd_f7fd55b1","updated":"2016-02-14 01:19:53.000000000","message":"Yes, \u0027_\u0027 prefix means that it\u0027s a  private one.","commit_id":"9d1116f14b31f7f17788d9a3062d0b64d09c714e"}]}