)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8726,"name":"Victor Morales","email":"chipahuac@hotmail.com","username":"electrocucaracha"},"change_message_id":"1b1aa727c1d9b5940b411316a783501d237263ee","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Akihiro Motoki \u003camotoki@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-12-15 21:03:01 +0900"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"api-ref: project_id in req/resp body should be \"body\""},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"project_id and tenant_id field in request/response body are"},{"line_number":10,"context_line":"marked as \"path\" now. It should be \"body\"."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"1a6eadb0_eee5e708","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":53},"updated":"2016-12-15 22:50:10.000000000","message":"nit: The title exceeds the maximum number of characters.","commit_id":"35672b836977d00ac2a5a9e42bc4d149c73f32bb"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"2e4d39775595826a0d7f2402a0a18b4315428b83","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Akihiro Motoki \u003camotoki@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-12-15 21:03:01 +0900"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"api-ref: project_id in req/resp body should be \"body\""},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"project_id and tenant_id field in request/response body are"},{"line_number":10,"context_line":"marked as \"path\" now. It should be \"body\"."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"1a6eadb0_6b917c52","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":53},"in_reply_to":"1a6eadb0_eee5e708","updated":"2016-12-19 01:43:46.000000000","message":"This is not a strict guideline. I think the current length of the summary is acceptable.","commit_id":"35672b836977d00ac2a5a9e42bc4d149c73f32bb"},{"author":{"_account_id":8726,"name":"Victor Morales","email":"chipahuac@hotmail.com","username":"electrocucaracha"},"change_message_id":"1b1aa727c1d9b5940b411316a783501d237263ee","unresolved":false,"context_lines":[{"line_number":10,"context_line":"marked as \"path\" now. It should be \"body\"."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"- \"project_id-path\" already exists and it can be used"},{"line_number":13,"context_line":"  for \"project\" in URL path."},{"line_number":14,"context_line":"- \"project_id\" is now used for body."},{"line_number":15,"context_line":"- \"project_id-body\" is now duplicated, so it is revmoed."},{"line_number":16,"context_line":"  fwaas.inc is the only user of project_id-body and"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"1a6eadb0_0e4943d0","line":13,"range":{"start_line":13,"start_character":7,"end_line":13,"end_character":14},"updated":"2016-12-15 22:50:10.000000000","message":"shouldn\u0027t project_id?","commit_id":"35672b836977d00ac2a5a9e42bc4d149c73f32bb"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"2e4d39775595826a0d7f2402a0a18b4315428b83","unresolved":false,"context_lines":[{"line_number":10,"context_line":"marked as \"path\" now. It should be \"body\"."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"- \"project_id-path\" already exists and it can be used"},{"line_number":13,"context_line":"  for \"project\" in URL path."},{"line_number":14,"context_line":"- \"project_id\" is now used for body."},{"line_number":15,"context_line":"- \"project_id-body\" is now duplicated, so it is revmoed."},{"line_number":16,"context_line":"  fwaas.inc is the only user of project_id-body and"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"1a6eadb0_eb8bcc1c","line":13,"range":{"start_line":13,"start_character":7,"end_line":13,"end_character":14},"in_reply_to":"1a6eadb0_0e4943d0","updated":"2016-12-19 01:43:46.000000000","message":"Done","commit_id":"35672b836977d00ac2a5a9e42bc4d149c73f32bb"},{"author":{"_account_id":8726,"name":"Victor Morales","email":"chipahuac@hotmail.com","username":"electrocucaracha"},"change_message_id":"1b1aa727c1d9b5940b411316a783501d237263ee","unresolved":false,"context_lines":[{"line_number":12,"context_line":"- \"project_id-path\" already exists and it can be used"},{"line_number":13,"context_line":"  for \"project\" in URL path."},{"line_number":14,"context_line":"- \"project_id\" is now used for body."},{"line_number":15,"context_line":"- \"project_id-body\" is now duplicated, so it is revmoed."},{"line_number":16,"context_line":"  fwaas.inc is the only user of project_id-body and"},{"line_number":17,"context_line":"   it is updated accordingly."},{"line_number":18,"context_line":"- quotas.inc is updated to use \u0027project_id-path\u0027."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"1a6eadb0_6ec0f750","line":15,"range":{"start_line":15,"start_character":48,"end_line":15,"end_character":55},"updated":"2016-12-15 22:50:10.000000000","message":"s/revmoed/removed","commit_id":"35672b836977d00ac2a5a9e42bc4d149c73f32bb"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"2e4d39775595826a0d7f2402a0a18b4315428b83","unresolved":false,"context_lines":[{"line_number":12,"context_line":"- \"project_id-path\" already exists and it can be used"},{"line_number":13,"context_line":"  for \"project\" in URL path."},{"line_number":14,"context_line":"- \"project_id\" is now used for body."},{"line_number":15,"context_line":"- \"project_id-body\" is now duplicated, so it is revmoed."},{"line_number":16,"context_line":"  fwaas.inc is the only user of project_id-body and"},{"line_number":17,"context_line":"   it is updated accordingly."},{"line_number":18,"context_line":"- quotas.inc is updated to use \u0027project_id-path\u0027."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"1a6eadb0_6b789c2c","line":15,"range":{"start_line":15,"start_character":48,"end_line":15,"end_character":55},"in_reply_to":"1a6eadb0_6ec0f750","updated":"2016-12-19 01:43:46.000000000","message":"Done","commit_id":"35672b836977d00ac2a5a9e42bc4d149c73f32bb"},{"author":{"_account_id":8726,"name":"Victor Morales","email":"chipahuac@hotmail.com","username":"electrocucaracha"},"change_message_id":"1b1aa727c1d9b5940b411316a783501d237263ee","unresolved":false,"context_lines":[{"line_number":17,"context_line":"   it is updated accordingly."},{"line_number":18,"context_line":"- quotas.inc is updated to use \u0027project_id-path\u0027."},{"line_number":19,"context_line":"  Also project_id/tenant_id in response body of quotas operations"},{"line_number":20,"context_line":"  have been droped as they do not exist."},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"Note that project_id/tenant_id in request body should be marked"},{"line_number":23,"context_line":"as \"optional\" in most resources but this patch does not touch them"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"1a6eadb0_2e719f44","line":20,"range":{"start_line":20,"start_character":2,"end_line":20,"end_character":6},"updated":"2016-12-15 22:50:10.000000000","message":"Maybe I could be wrong, but isn\u0027t referring to the body? if that\u0027s the case \"has\" needs to be used instead of \"have\"","commit_id":"35672b836977d00ac2a5a9e42bc4d149c73f32bb"},{"author":{"_account_id":8726,"name":"Victor Morales","email":"chipahuac@hotmail.com","username":"electrocucaracha"},"change_message_id":"1b1aa727c1d9b5940b411316a783501d237263ee","unresolved":false,"context_lines":[{"line_number":17,"context_line":"   it is updated accordingly."},{"line_number":18,"context_line":"- quotas.inc is updated to use \u0027project_id-path\u0027."},{"line_number":19,"context_line":"  Also project_id/tenant_id in response body of quotas operations"},{"line_number":20,"context_line":"  have been droped as they do not exist."},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"Note that project_id/tenant_id in request body should be marked"},{"line_number":23,"context_line":"as \"optional\" in most resources but this patch does not touch them"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"1a6eadb0_cea66bad","line":20,"range":{"start_line":20,"start_character":12,"end_line":20,"end_character":18},"updated":"2016-12-15 22:50:10.000000000","message":"s/droped/dropped","commit_id":"35672b836977d00ac2a5a9e42bc4d149c73f32bb"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"2e4d39775595826a0d7f2402a0a18b4315428b83","unresolved":false,"context_lines":[{"line_number":17,"context_line":"   it is updated accordingly."},{"line_number":18,"context_line":"- quotas.inc is updated to use \u0027project_id-path\u0027."},{"line_number":19,"context_line":"  Also project_id/tenant_id in response body of quotas operations"},{"line_number":20,"context_line":"  have been droped as they do not exist."},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"Note that project_id/tenant_id in request body should be marked"},{"line_number":23,"context_line":"as \"optional\" in most resources but this patch does not touch them"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"1a6eadb0_ebb90ccd","line":20,"range":{"start_line":20,"start_character":2,"end_line":20,"end_character":6},"in_reply_to":"1a6eadb0_2e719f44","updated":"2016-12-19 01:43:46.000000000","message":"The subject of \u0027have\u0027 is project_id/tenant_id. I rephrased it to \u0027project_id and tenant_id\u0027.","commit_id":"35672b836977d00ac2a5a9e42bc4d149c73f32bb"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"2e4d39775595826a0d7f2402a0a18b4315428b83","unresolved":false,"context_lines":[{"line_number":17,"context_line":"   it is updated accordingly."},{"line_number":18,"context_line":"- quotas.inc is updated to use \u0027project_id-path\u0027."},{"line_number":19,"context_line":"  Also project_id/tenant_id in response body of quotas operations"},{"line_number":20,"context_line":"  have been droped as they do not exist."},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"Note that project_id/tenant_id in request body should be marked"},{"line_number":23,"context_line":"as \"optional\" in most resources but this patch does not touch them"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"1a6eadb0_4ba2009c","line":20,"range":{"start_line":20,"start_character":12,"end_line":20,"end_character":18},"in_reply_to":"1a6eadb0_cea66bad","updated":"2016-12-19 01:43:46.000000000","message":"Done","commit_id":"35672b836977d00ac2a5a9e42bc4d149c73f32bb"}],"api-ref/source/v2/fwaas.inc":[{"author":{"_account_id":5367,"name":"boden","email":"bodenvmw@gmail.com","username":"boden"},"change_message_id":"956c2473ae8b98e32e5828200afff8bcfef3efb0","unresolved":false,"context_lines":[{"line_number":70,"context_line":"   - id: firewall_policy_id-body"},{"line_number":71,"context_line":"   - name: name"},{"line_number":72,"context_line":"   - shared: shared-response"},{"line_number":73,"context_line":"   - tenant_id: project_id"},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"Response Example"},{"line_number":76,"context_line":"----------------"}],"source_content_type":"text/x-c++src","patch_set":2,"id":"1a6eadb0_ad61553b","line":73,"range":{"start_line":73,"start_character":5,"end_line":73,"end_character":14},"updated":"2016-12-15 11:49:05.000000000","message":"This should be project_id rather than tenant_id again, shouldn\u0027t it?","commit_id":"efb8a562aec40d90d45c317ecbd09de405c8ccb9"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"997a6826b8f1b123cb1cb4a3d75d9914bd024713","unresolved":false,"context_lines":[{"line_number":70,"context_line":"   - id: firewall_policy_id-body"},{"line_number":71,"context_line":"   - name: name"},{"line_number":72,"context_line":"   - shared: shared-response"},{"line_number":73,"context_line":"   - tenant_id: project_id"},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"Response Example"},{"line_number":76,"context_line":"----------------"}],"source_content_type":"text/x-c++src","patch_set":2,"id":"1a6eadb0_f0fb2279","line":73,"range":{"start_line":73,"start_character":5,"end_line":73,"end_character":14},"in_reply_to":"1a6eadb0_ad61553b","updated":"2016-12-15 12:01:50.000000000","message":"Good catch!","commit_id":"efb8a562aec40d90d45c317ecbd09de405c8ccb9"}],"api-ref/source/v2/quotas.inc":[{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"806db7165c7349509688cf6004eaa4fbff149bfd","unresolved":false,"context_lines":[{"line_number":105,"context_line":""},{"line_number":106,"context_line":".. rest_parameters:: parameters.yaml"},{"line_number":107,"context_line":""},{"line_number":108,"context_line":"   - project_id: project_id"},{"line_number":109,"context_line":"   - subnet: quota-subnet"},{"line_number":110,"context_line":"   - network: quota-network"},{"line_number":111,"context_line":"   - floatingip: quota-floatingip"}],"source_content_type":"text/x-c++src","patch_set":1,"id":"1a6eadb0_f215ef3f","line":108,"range":{"start_line":108,"start_character":17,"end_line":108,"end_character":27},"updated":"2016-12-15 09:27:51.000000000","message":"This should be \u0027project_id-path\u0027.","commit_id":"bcc1d2e326ad73f9dc955ea62bb1b944919c5904"},{"author":{"_account_id":12021,"name":"Thomas Morin","email":"tmmorin.orange@gmail.com","username":"tmorin"},"change_message_id":"947b1067108fb5c840017512e913dc94923a3048","unresolved":false,"context_lines":[{"line_number":64,"context_line":"-------"},{"line_number":65,"context_line":""},{"line_number":66,"context_line":".. rest_parameters:: parameters.yaml"},{"line_number":67,"context_line":""},{"line_number":68,"context_line":"   - project_id: project_id-path"},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"Response Parameters"}],"source_content_type":"text/x-c++src","patch_set":2,"id":"1a6eadb0_cd293639","line":67,"updated":"2016-12-15 09:55:08.000000000","message":"Why remove tenant_id, since it is still supported, right ?","commit_id":"efb8a562aec40d90d45c317ecbd09de405c8ccb9"},{"author":{"_account_id":12021,"name":"Thomas Morin","email":"tmmorin.orange@gmail.com","username":"tmorin"},"change_message_id":"edb59df361d559c9d13b8eed9a6fe83d6ff81a61","unresolved":false,"context_lines":[{"line_number":64,"context_line":"-------"},{"line_number":65,"context_line":""},{"line_number":66,"context_line":".. rest_parameters:: parameters.yaml"},{"line_number":67,"context_line":""},{"line_number":68,"context_line":"   - project_id: project_id-path"},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"Response Parameters"}],"source_content_type":"text/x-c++src","patch_set":2,"id":"1a6eadb0_dbd9e758","line":67,"in_reply_to":"1a6eadb0_109c1e5c","updated":"2016-12-15 12:15:23.000000000","message":"Thanks, sorry for the confusion.\nI hadn\u0027t spotted this was in path.","commit_id":"efb8a562aec40d90d45c317ecbd09de405c8ccb9"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"997a6826b8f1b123cb1cb4a3d75d9914bd024713","unresolved":false,"context_lines":[{"line_number":64,"context_line":"-------"},{"line_number":65,"context_line":""},{"line_number":66,"context_line":".. rest_parameters:: parameters.yaml"},{"line_number":67,"context_line":""},{"line_number":68,"context_line":"   - project_id: project_id-path"},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"Response Parameters"}],"source_content_type":"text/x-c++src","patch_set":2,"id":"1a6eadb0_109c1e5c","line":67,"in_reply_to":"1a6eadb0_cd293639","updated":"2016-12-15 12:01:50.000000000","message":"This corresponds to the URL at L.55. There is only \"project_id\" in the URL and no \"tenant_id\", so I remove tenant_id. Does it make sense to you?","commit_id":"efb8a562aec40d90d45c317ecbd09de405c8ccb9"},{"author":{"_account_id":12021,"name":"Thomas Morin","email":"tmmorin.orange@gmail.com","username":"tmorin"},"change_message_id":"947b1067108fb5c840017512e913dc94923a3048","unresolved":false,"context_lines":[{"line_number":105,"context_line":""},{"line_number":106,"context_line":".. rest_parameters:: parameters.yaml"},{"line_number":107,"context_line":""},{"line_number":108,"context_line":"   - project_id: project_id-path"},{"line_number":109,"context_line":"   - subnet: quota-subnet"},{"line_number":110,"context_line":"   - network: quota-network"},{"line_number":111,"context_line":"   - floatingip: quota-floatingip"},{"line_number":112,"context_line":"   - subnetpool: quota-subnetpool"},{"line_number":113,"context_line":"   - quotas: quotas"},{"line_number":114,"context_line":"   - security_group_rule: quota-security_group_rule"},{"line_number":115,"context_line":"   - security_group: quota-security_group"},{"line_number":116,"context_line":"   - router: quota-router"},{"line_number":117,"context_line":"   - rbac_policy: quota-rbac_policy"},{"line_number":118,"context_line":"   - port: quota-port"},{"line_number":119,"context_line":""},{"line_number":120,"context_line":"Request Example"},{"line_number":121,"context_line":"---------------"}],"source_content_type":"text/x-c++src","patch_set":2,"id":"1a6eadb0_8ddade4a","line":118,"range":{"start_line":108,"start_character":5,"end_line":118,"end_character":21},"updated":"2016-12-15 09:55:08.000000000","message":"Why remove tenant_id, since it is still supported, right ?","commit_id":"efb8a562aec40d90d45c317ecbd09de405c8ccb9"},{"author":{"_account_id":12021,"name":"Thomas Morin","email":"tmmorin.orange@gmail.com","username":"tmorin"},"change_message_id":"edb59df361d559c9d13b8eed9a6fe83d6ff81a61","unresolved":false,"context_lines":[{"line_number":105,"context_line":""},{"line_number":106,"context_line":".. rest_parameters:: parameters.yaml"},{"line_number":107,"context_line":""},{"line_number":108,"context_line":"   - project_id: project_id-path"},{"line_number":109,"context_line":"   - subnet: quota-subnet"},{"line_number":110,"context_line":"   - network: quota-network"},{"line_number":111,"context_line":"   - floatingip: quota-floatingip"},{"line_number":112,"context_line":"   - subnetpool: quota-subnetpool"},{"line_number":113,"context_line":"   - quotas: quotas"},{"line_number":114,"context_line":"   - security_group_rule: quota-security_group_rule"},{"line_number":115,"context_line":"   - security_group: quota-security_group"},{"line_number":116,"context_line":"   - router: quota-router"},{"line_number":117,"context_line":"   - rbac_policy: quota-rbac_policy"},{"line_number":118,"context_line":"   - port: quota-port"},{"line_number":119,"context_line":""},{"line_number":120,"context_line":"Request Example"},{"line_number":121,"context_line":"---------------"}],"source_content_type":"text/x-c++src","patch_set":2,"id":"1a6eadb0_9b676fb6","line":118,"range":{"start_line":108,"start_character":5,"end_line":118,"end_character":21},"in_reply_to":"1a6eadb0_10ea5eac","updated":"2016-12-15 12:15:23.000000000","message":"The part above is not a response example, but the request doc. If tenant_id is supported for request, it should appear here, right ?","commit_id":"efb8a562aec40d90d45c317ecbd09de405c8ccb9"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"997a6826b8f1b123cb1cb4a3d75d9914bd024713","unresolved":false,"context_lines":[{"line_number":105,"context_line":""},{"line_number":106,"context_line":".. rest_parameters:: parameters.yaml"},{"line_number":107,"context_line":""},{"line_number":108,"context_line":"   - project_id: project_id-path"},{"line_number":109,"context_line":"   - subnet: quota-subnet"},{"line_number":110,"context_line":"   - network: quota-network"},{"line_number":111,"context_line":"   - floatingip: quota-floatingip"},{"line_number":112,"context_line":"   - subnetpool: quota-subnetpool"},{"line_number":113,"context_line":"   - quotas: quotas"},{"line_number":114,"context_line":"   - security_group_rule: quota-security_group_rule"},{"line_number":115,"context_line":"   - security_group: quota-security_group"},{"line_number":116,"context_line":"   - router: quota-router"},{"line_number":117,"context_line":"   - rbac_policy: quota-rbac_policy"},{"line_number":118,"context_line":"   - port: quota-port"},{"line_number":119,"context_line":""},{"line_number":120,"context_line":"Request Example"},{"line_number":121,"context_line":"---------------"}],"source_content_type":"text/x-c++src","patch_set":2,"id":"1a6eadb0_10ea5eac","line":118,"range":{"start_line":108,"start_character":5,"end_line":118,"end_character":21},"in_reply_to":"1a6eadb0_8ddade4a","updated":"2016-12-15 12:01:50.000000000","message":"If you look at the response example, there is no tenant_id (and project_id) in the response body, so I drop tenant_id from the above list.","commit_id":"efb8a562aec40d90d45c317ecbd09de405c8ccb9"},{"author":{"_account_id":5367,"name":"boden","email":"bodenvmw@gmail.com","username":"boden"},"change_message_id":"21d344b31a693ef40d57868d3d7ac1f9ab85a766","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"1a6eadb0_848b8b67","updated":"2016-12-15 14:15:29.000000000","message":"It looks like \u0027List quotas for projects with non-default quota values\u0027 doesn\u0027t include a project or tenant ID in the response params, but they are shown in the sample response. Is that something that should be addressed here?","commit_id":"35672b836977d00ac2a5a9e42bc4d149c73f32bb"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"2e4d39775595826a0d7f2402a0a18b4315428b83","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"1a6eadb0_cbd1b031","in_reply_to":"1a6eadb0_848b8b67","updated":"2016-12-19 01:43:46.000000000","message":"quota-list example looks incorrect too. project_id is not included in a response of quota-list (with non-default values). It looks like a bug of the neutron quota API implementation itself. I will take care of it.\n\nRegarding the API reference, I will update the quota stuff separately.","commit_id":"35672b836977d00ac2a5a9e42bc4d149c73f32bb"}]}