)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"66b2a46322cf36b09c0a3298021afdc1fc320459","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Migrate identity /v3-ext docs from api-ref repo"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Docs at [1] have already been converted from WADL (SGML / XML) to RST"},{"line_number":10,"context_line":"using fairy-slipper [2]."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"This commit polish the results from the conversion and migrate the v3"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"7aa08908_9e4cc977","line":9,"range":{"start_line":9,"start_character":17,"end_line":9,"end_character":24},"updated":"2016-06-16 18:40:58.000000000","message":"I don\u0027t know what \"already\" means here... somebody else did it?","commit_id":"3b394426c176903596422b3c61eaf661fcc4b761"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"d12affbd31b64a57a142f0aa21ee8af1a9203f0e","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Migrate identity /v3-ext docs from api-ref repo"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Docs at [1] have already been converted from WADL (SGML / XML) to RST"},{"line_number":10,"context_line":"using fairy-slipper [2]."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"This commit polish the results from the conversion and migrate the v3"},{"line_number":13,"context_line":"extension docs to our repository under \u0027api-ref/source\u0027 directory. It"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"7aa08908_3ab1e5e0","line":10,"updated":"2016-06-14 15:48:49.000000000","message":"Does this tool let you convert documentation in batches?","commit_id":"3b394426c176903596422b3c61eaf661fcc4b761"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"66b2a46322cf36b09c0a3298021afdc1fc320459","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Docs at [1] have already been converted from WADL (SGML / XML) to RST"},{"line_number":10,"context_line":"using fairy-slipper [2]."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"This commit polish the results from the conversion and migrate the v3"},{"line_number":13,"context_line":"extension docs to our repository under \u0027api-ref/source\u0027 directory. It"},{"line_number":14,"context_line":"also removes the OS-KDS docs, given it does not exist anymore. Missing"},{"line_number":15,"context_line":"parameters definitions were added."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"7aa08908_3e46754f","line":12,"range":{"start_line":12,"start_character":55,"end_line":12,"end_character":62},"updated":"2016-06-16 18:40:58.000000000","message":"migrates","commit_id":"3b394426c176903596422b3c61eaf661fcc4b761"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"66b2a46322cf36b09c0a3298021afdc1fc320459","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Docs at [1] have already been converted from WADL (SGML / XML) to RST"},{"line_number":10,"context_line":"using fairy-slipper [2]."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"This commit polish the results from the conversion and migrate the v3"},{"line_number":13,"context_line":"extension docs to our repository under \u0027api-ref/source\u0027 directory. It"},{"line_number":14,"context_line":"also removes the OS-KDS docs, given it does not exist anymore. Missing"},{"line_number":15,"context_line":"parameters definitions were added."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"7aa08908_de565105","line":12,"range":{"start_line":12,"start_character":12,"end_line":12,"end_character":18},"updated":"2016-06-16 18:40:58.000000000","message":"polishes","commit_id":"3b394426c176903596422b3c61eaf661fcc4b761"},{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"ec7a54713092c3abee5cc802fbb8e153dc83790f","unresolved":false,"context_lines":[{"line_number":11,"context_line":""},{"line_number":12,"context_line":"This commit polish the results from the conversion and migrate the v3"},{"line_number":13,"context_line":"extension docs to our repository under \u0027api-ref/source\u0027 directory. It"},{"line_number":14,"context_line":"also removes the OS-KDS docs, given it does not exist anymore. Missing"},{"line_number":15,"context_line":"parameters definitions were added."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"The API examples for the extensions were moved into \u0027/v3-ext/samples\u0027."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"7aa08908_f373384c","line":14,"updated":"2016-06-13 21:08:28.000000000","message":"As soon as a commit message includes the word \"also,\" that\u0027s a red flag that the commit is too big. In fact, there are a lot of substantial changes in this patch which would have been much easier to review as individual patches. All together, they\u0027re confusing to review.","commit_id":"3b394426c176903596422b3c61eaf661fcc4b761"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"66b2a46322cf36b09c0a3298021afdc1fc320459","unresolved":false,"context_lines":[{"line_number":16,"context_line":""},{"line_number":17,"context_line":"The API examples for the extensions were moved into \u0027/v3-ext/samples\u0027."},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Polishing the generated RST files include:"},{"line_number":20,"context_line":"- Removing unnecessary blank lines;"},{"line_number":21,"context_line":"- Removing empty references."},{"line_number":22,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"7aa08908_9ee84918","line":19,"range":{"start_line":19,"start_character":34,"end_line":19,"end_character":41},"updated":"2016-06-16 18:40:58.000000000","message":"includes","commit_id":"3b394426c176903596422b3c61eaf661fcc4b761"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"66b2a46322cf36b09c0a3298021afdc1fc320459","unresolved":false,"context_lines":[{"line_number":20,"context_line":"- Removing unnecessary blank lines;"},{"line_number":21,"context_line":"- Removing empty references."},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"Polishing the generated RST files do not include:"},{"line_number":24,"context_line":"- Modifying their content;"},{"line_number":25,"context_line":"- Modifying file names;"},{"line_number":26,"context_line":"- Wrapping lines at the maximum of 79 chars."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"7aa08908_3e8b156a","line":23,"range":{"start_line":23,"start_character":34,"end_line":23,"end_character":36},"updated":"2016-06-16 18:40:58.000000000","message":"does","commit_id":"3b394426c176903596422b3c61eaf661fcc4b761"}],"api-ref/source/v3-ext/parameters.yaml":[{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"50cfd6a39eb91542b5cd5cbc0cb6033c8f1204b3","unresolved":false,"context_lines":[{"line_number":3,"context_line":"# variables in path"},{"line_number":4,"context_line":"access_token_id:"},{"line_number":5,"context_line":"  description: |"},{"line_number":6,"context_line":"    The UUID of the access token."},{"line_number":7,"context_line":"  in: path"},{"line_number":8,"context_line":"  required: false"},{"line_number":9,"context_line":"  type: string"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"3aaa91ec_916bf6b8","line":6,"updated":"2016-06-29 14:02:49.000000000","message":"I think these should all be ID rather than UUID. That it\u0027s a UUID is an implementation detail that clients shouldn\u0027t rely on.","commit_id":"038c0e35cafaee4c7f1369c21007dfebac760bf6"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"50cfd6a39eb91542b5cd5cbc0cb6033c8f1204b3","unresolved":false,"context_lines":[{"line_number":5,"context_line":"  description: |"},{"line_number":6,"context_line":"    The UUID of the access token."},{"line_number":7,"context_line":"  in: path"},{"line_number":8,"context_line":"  required: false"},{"line_number":9,"context_line":"  type: string"},{"line_number":10,"context_line":"consumer_id:"},{"line_number":11,"context_line":"  description: |"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"3aaa91ec_6b377d4d","line":8,"updated":"2016-06-29 14:02:49.000000000","message":"why do these all have \"required: false\"? Parameters in the path are required.","commit_id":"038c0e35cafaee4c7f1369c21007dfebac760bf6"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"50cfd6a39eb91542b5cd5cbc0cb6033c8f1204b3","unresolved":false,"context_lines":[{"line_number":21,"context_line":"  type: string"},{"line_number":22,"context_line":"endpoint_id:"},{"line_number":23,"context_line":"  description: |"},{"line_number":24,"context_line":"    The endpoint ID."},{"line_number":25,"context_line":"  in: path"},{"line_number":26,"context_line":"  required: false"},{"line_number":27,"context_line":"  type: string"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"3aaa91ec_abcfa554","line":24,"updated":"2016-06-29 14:02:49.000000000","message":"why doesn\u0027t this have UUID?","commit_id":"038c0e35cafaee4c7f1369c21007dfebac760bf6"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"50cfd6a39eb91542b5cd5cbc0cb6033c8f1204b3","unresolved":false,"context_lines":[{"line_number":33,"context_line":"  type: string"},{"line_number":34,"context_line":"name:"},{"line_number":35,"context_line":"  description: |"},{"line_number":36,"context_line":"    The name of the group."},{"line_number":37,"context_line":"  in: path"},{"line_number":38,"context_line":"  required: false"},{"line_number":39,"context_line":"  type: string"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"3aaa91ec_0bf07990","line":36,"updated":"2016-06-29 14:02:49.000000000","message":"we have group names in paths?","commit_id":"038c0e35cafaee4c7f1369c21007dfebac760bf6"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"50cfd6a39eb91542b5cd5cbc0cb6033c8f1204b3","unresolved":false,"context_lines":[{"line_number":95,"context_line":"  in: body"},{"line_number":96,"context_line":"  required: true"},{"line_number":97,"context_line":"  type: string"},{"line_number":98,"context_line":"id_1:"},{"line_number":99,"context_line":"  description: |"},{"line_number":100,"context_line":"    The endpoint UUID."},{"line_number":101,"context_line":"  in: body"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"3aaa91ec_11e54637","line":98,"updated":"2016-06-29 14:02:49.000000000","message":"can we make this endpoint_id rather than id_1?\n\nThis applies to all these \"id\" parameters.\n\nI grepped through the files and I don\u0027t see this used anywhere.","commit_id":"038c0e35cafaee4c7f1369c21007dfebac760bf6"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"50cfd6a39eb91542b5cd5cbc0cb6033c8f1204b3","unresolved":false,"context_lines":[{"line_number":101,"context_line":"  in: body"},{"line_number":102,"context_line":"  required: true"},{"line_number":103,"context_line":"  type: string"},{"line_number":104,"context_line":"id_2:"},{"line_number":105,"context_line":"  description: |"},{"line_number":106,"context_line":"    The ID of the policy."},{"line_number":107,"context_line":"  in: body"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"3aaa91ec_d1fa9ebc","line":104,"updated":"2016-06-29 14:02:49.000000000","message":"I couldn\u0027t find this used anywhere.","commit_id":"038c0e35cafaee4c7f1369c21007dfebac760bf6"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"50cfd6a39eb91542b5cd5cbc0cb6033c8f1204b3","unresolved":false,"context_lines":[{"line_number":117,"context_line":"  description: |"},{"line_number":118,"context_line":"    The interface type, which describes the"},{"line_number":119,"context_line":"    visibility of the endpoint.  Value is:  - ``public``. Visible by"},{"line_number":120,"context_line":"    end users on a publicly available network   interface.  -"},{"line_number":121,"context_line":"    ``internal``. Visible by end users on an unmetered internal"},{"line_number":122,"context_line":"    network interface.  - ``admin``. Visible by administrative users"},{"line_number":123,"context_line":"    on a secure network   interface."}],"source_content_type":"text/x-yaml","patch_set":5,"id":"3aaa91ec_91dad649","line":120,"range":{"start_line":120,"start_character":45,"end_line":120,"end_character":48},"updated":"2016-06-29 14:02:49.000000000","message":"remove extra spaces.","commit_id":"038c0e35cafaee4c7f1369c21007dfebac760bf6"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"50cfd6a39eb91542b5cd5cbc0cb6033c8f1204b3","unresolved":false,"context_lines":[{"line_number":130,"context_line":"  in: body"},{"line_number":131,"context_line":"  required: true"},{"line_number":132,"context_line":"  type: object"},{"line_number":133,"context_line":"links_1:"},{"line_number":134,"context_line":"  description: |"},{"line_number":135,"context_line":"    The links for the ``endpoints`` resource."},{"line_number":136,"context_line":"  in: body"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"3aaa91ec_11ef06ea","line":133,"updated":"2016-06-29 14:02:49.000000000","message":"This should be \"endpoint_links\"\n\nAlso, I don\u0027t see this used anywhere.","commit_id":"038c0e35cafaee4c7f1369c21007dfebac760bf6"}],"api-ref/source/v3-ext/trust.inc":[{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"ec7a54713092c3abee5cc802fbb8e153dc83790f","unresolved":false,"context_lines":[{"line_number":22,"context_line":"/admin-guide/keystone_use_trusts.html\u003e`_."},{"line_number":23,"context_line":""},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"Create trust"},{"line_number":26,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":".. rest_method::  POST /v3/OS-TRUST/trusts"}],"source_content_type":"text/x-c++src","patch_set":4,"id":"7aa08908_70573e5a","line":25,"updated":"2016-06-13 21:08:28.000000000","message":"Where is the rest of the trust API documentation?\n\n  https://github.com/openstack/keystone-specs/blob/master/api/v3/identity-api-v3-os-trust-ext.rst","commit_id":"3b394426c176903596422b3c61eaf661fcc4b761"}]}