)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"eaf7b230b543f47a02922a7c7ff84f88cfaf405e","unresolved":false,"context_lines":[{"line_number":10,"context_line":"Essentially, matched the [app:service_v3] entry with "},{"line_number":11,"context_line":"the value in the ini file and removed the [pipeline:api_v3] "},{"line_number":12,"context_line":"comment in order to avoid confusion.  Everything else "},{"line_number":13,"context_line":"related to the v3 pipeline appears to be accurate."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Closes Bug: 1504891"},{"line_number":16,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"5a2ca52d_0e99711f","line":13,"updated":"2015-10-14 18:12:23.000000000","message":"You can also line wrap the body at 72 chars.","commit_id":"507cee99d8eab4c14821b853931aead1032de85f"},{"author":{"_account_id":18338,"name":"Ron De Rose","email":"ronald.de.rose@intel.com","username":"derosenet"},"change_message_id":"5e74e67c620ecacc1b72fc030f6f1f6e373537e1","unresolved":false,"context_lines":[{"line_number":10,"context_line":"Essentially, matched the [app:service_v3] entry with "},{"line_number":11,"context_line":"the value in the ini file and removed the [pipeline:api_v3] "},{"line_number":12,"context_line":"comment in order to avoid confusion.  Everything else "},{"line_number":13,"context_line":"related to the v3 pipeline appears to be accurate."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Closes Bug: 1504891"},{"line_number":16,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"5a2ca52d_b1b6a288","line":13,"in_reply_to":"5a2ca52d_0e99711f","updated":"2015-10-14 18:42:56.000000000","message":"Done","commit_id":"507cee99d8eab4c14821b853931aead1032de85f"},{"author":{"_account_id":17123,"name":"Tom Cocozzello (tjcocozz)","email":"tjcocozz@us.ibm.com","username":"tcocozzello"},"change_message_id":"fe09d2ef2123494009e802da443a3bf2d0e60091","unresolved":false,"context_lines":[{"line_number":12,"context_line":"comment in order to avoid confusion.  Everything else "},{"line_number":13,"context_line":"related to the v3 pipeline appears to be accurate."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Closes Bug: 1504891"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: I57f6a2a5287b2f6aa108586c1ad07b9a6235d53d"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"5a2ca52d_ae571d01","line":15,"updated":"2015-10-14 17:55:23.000000000","message":"Closes-Bug\n\nProbably should remove the extra white spaces at the end of your lines too.","commit_id":"507cee99d8eab4c14821b853931aead1032de85f"},{"author":{"_account_id":18338,"name":"Ron De Rose","email":"ronald.de.rose@intel.com","username":"derosenet"},"change_message_id":"3500adb9f5bcd9cb0ac98096f7bbab805ffcb48a","unresolved":false,"context_lines":[{"line_number":12,"context_line":"comment in order to avoid confusion.  Everything else "},{"line_number":13,"context_line":"related to the v3 pipeline appears to be accurate."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Closes Bug: 1504891"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: I57f6a2a5287b2f6aa108586c1ad07b9a6235d53d"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"5a2ca52d_f14dfa72","line":15,"in_reply_to":"5a2ca52d_0ecbb11f","updated":"2015-10-14 18:17:52.000000000","message":"Done","commit_id":"507cee99d8eab4c14821b853931aead1032de85f"},{"author":{"_account_id":18338,"name":"Ron De Rose","email":"ronald.de.rose@intel.com","username":"derosenet"},"change_message_id":"5e74e67c620ecacc1b72fc030f6f1f6e373537e1","unresolved":false,"context_lines":[{"line_number":12,"context_line":"comment in order to avoid confusion.  Everything else "},{"line_number":13,"context_line":"related to the v3 pipeline appears to be accurate."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Closes Bug: 1504891"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: I57f6a2a5287b2f6aa108586c1ad07b9a6235d53d"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"5a2ca52d_71b1ea8e","line":15,"in_reply_to":"5a2ca52d_ae571d01","updated":"2015-10-14 18:42:56.000000000","message":"Done","commit_id":"507cee99d8eab4c14821b853931aead1032de85f"},{"author":{"_account_id":18338,"name":"Ron De Rose","email":"ronald.de.rose@intel.com","username":"derosenet"},"change_message_id":"3500adb9f5bcd9cb0ac98096f7bbab805ffcb48a","unresolved":false,"context_lines":[{"line_number":12,"context_line":"comment in order to avoid confusion.  Everything else "},{"line_number":13,"context_line":"related to the v3 pipeline appears to be accurate."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Closes Bug: 1504891"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: I57f6a2a5287b2f6aa108586c1ad07b9a6235d53d"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"5a2ca52d_d10c7e3d","line":15,"in_reply_to":"5a2ca52d_ae571d01","updated":"2015-10-14 18:17:52.000000000","message":"Done","commit_id":"507cee99d8eab4c14821b853931aead1032de85f"},{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"eaf7b230b543f47a02922a7c7ff84f88cfaf405e","unresolved":false,"context_lines":[{"line_number":12,"context_line":"comment in order to avoid confusion.  Everything else "},{"line_number":13,"context_line":"related to the v3 pipeline appears to be accurate."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Closes Bug: 1504891"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: I57f6a2a5287b2f6aa108586c1ad07b9a6235d53d"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"5a2ca52d_0ecbb11f","line":15,"in_reply_to":"5a2ca52d_ae571d01","updated":"2015-10-14 18:12:23.000000000","message":"With the space, the bots picked it up well enough to mark the bug as In Progress, but git-review didn\u0027t include the Change-Id with it because it didn\u0027t think it was a footer, and gerrit is refusing to link the entirety of \"Closes Bug\" to the bug report, due to the space.\n\nWhen you revise this, you can include them together, like so:\n\n  Closes-Bug: 1504891\n  Change-Id: I57f6a2a5287b2f6aa108586c1ad07b9a6235d53d\n\n(order doesn\u0027t matter)","commit_id":"507cee99d8eab4c14821b853931aead1032de85f"},{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"679474d56ef82a8b5c8280c42b7876e6f037ecb0","unresolved":false,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Closes Bug: 1504891"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: I57f6a2a5287b2f6aa108586c1ad07b9a6235d53d"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"5a2ca52d_ae3b9dea","line":15,"updated":"2015-10-14 18:14:46.000000000","message":"There\u0027s a few comments in the previous patchset on your commit message that still apply.","commit_id":"2244832d234397caf7e321f24e9628002f32d6f6"}],"doc/source/http-api.rst":[{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"eaf7b230b543f47a02922a7c7ff84f88cfaf405e","unresolved":false,"context_lines":[{"line_number":73,"context_line":".. code-block:: ini"},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"    [app:service_v3]"},{"line_number":76,"context_line":"    use \u003d egg:keystone#service_v3"},{"line_number":77,"context_line":""},{"line_number":78,"context_line":"Then define a v3 pipeline, which terminates with the v3 application you defined"},{"line_number":79,"context_line":"above:"}],"source_content_type":"text/x-rst","patch_set":3,"id":"5a2ca52d_ce293939","line":76,"updated":"2015-10-14 18:12:23.000000000","message":"++","commit_id":"507cee99d8eab4c14821b853931aead1032de85f"}],"etc/keystone-paste.ini":[{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"86882eee8c4d397cbaef8d63caf61f7f0710159d","unresolved":false,"context_lines":[{"line_number":75,"context_line":""},{"line_number":76,"context_line":"[pipeline:api_v3]"},{"line_number":77,"context_line":"# The last item in this pipeline must be service_v3 or an equivalent"},{"line_number":78,"context_line":"# application. It cannot be a filter."},{"line_number":79,"context_line":"pipeline \u003d sizelimit url_normalize request_id build_auth_context token_auth admin_token_auth json_body ec2_extension_v3 s3_extension simple_cert_extension revoke_extension federation_extension oauth1_extension endpoint_filter_extension service_v3"},{"line_number":80,"context_line":""},{"line_number":81,"context_line":"[app:public_version_service]"}],"source_content_type":"text/x-properties","patch_set":3,"id":"5a2ca52d_eb4ae782","side":"PARENT","line":78,"updated":"2015-10-14 17:48:25.000000000","message":"revert this change","commit_id":"8500d76e354b797e0f094127a7b7a1a264a3b646"},{"author":{"_account_id":18338,"name":"Ron De Rose","email":"ronald.de.rose@intel.com","username":"derosenet"},"change_message_id":"ad6ba8b923ba6473a55017f73af02d836988515d","unresolved":false,"context_lines":[{"line_number":75,"context_line":""},{"line_number":76,"context_line":"[pipeline:api_v3]"},{"line_number":77,"context_line":"# The last item in this pipeline must be service_v3 or an equivalent"},{"line_number":78,"context_line":"# application. It cannot be a filter."},{"line_number":79,"context_line":"pipeline \u003d sizelimit url_normalize request_id build_auth_context token_auth admin_token_auth json_body ec2_extension_v3 s3_extension simple_cert_extension revoke_extension federation_extension oauth1_extension endpoint_filter_extension service_v3"},{"line_number":80,"context_line":""},{"line_number":81,"context_line":"[app:public_version_service]"}],"source_content_type":"text/x-properties","patch_set":3,"id":"5a2ca52d_b10f82e0","side":"PARENT","line":78,"in_reply_to":"5a2ca52d_4eff69aa","updated":"2015-10-14 18:30:27.000000000","message":"Apologize, I added the comment back. And didn\u0027t see the comment it the docs/ directory.","commit_id":"8500d76e354b797e0f094127a7b7a1a264a3b646"},{"author":{"_account_id":18338,"name":"Ron De Rose","email":"ronald.de.rose@intel.com","username":"derosenet"},"change_message_id":"3500adb9f5bcd9cb0ac98096f7bbab805ffcb48a","unresolved":false,"context_lines":[{"line_number":75,"context_line":""},{"line_number":76,"context_line":"[pipeline:api_v3]"},{"line_number":77,"context_line":"# The last item in this pipeline must be service_v3 or an equivalent"},{"line_number":78,"context_line":"# application. It cannot be a filter."},{"line_number":79,"context_line":"pipeline \u003d sizelimit url_normalize request_id build_auth_context token_auth admin_token_auth json_body ec2_extension_v3 s3_extension simple_cert_extension revoke_extension federation_extension oauth1_extension endpoint_filter_extension service_v3"},{"line_number":80,"context_line":""},{"line_number":81,"context_line":"[app:public_version_service]"}],"source_content_type":"text/x-properties","patch_set":3,"id":"5a2ca52d_ced4d9be","side":"PARENT","line":78,"in_reply_to":"5a2ca52d_eb4ae782","updated":"2015-10-14 18:17:52.000000000","message":"Done","commit_id":"8500d76e354b797e0f094127a7b7a1a264a3b646"},{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"eaf7b230b543f47a02922a7c7ff84f88cfaf405e","unresolved":false,"context_lines":[{"line_number":75,"context_line":""},{"line_number":76,"context_line":"[pipeline:api_v3]"},{"line_number":77,"context_line":"# The last item in this pipeline must be service_v3 or an equivalent"},{"line_number":78,"context_line":"# application. It cannot be a filter."},{"line_number":79,"context_line":"pipeline \u003d sizelimit url_normalize request_id build_auth_context token_auth admin_token_auth json_body ec2_extension_v3 s3_extension simple_cert_extension revoke_extension federation_extension oauth1_extension endpoint_filter_extension service_v3"},{"line_number":80,"context_line":""},{"line_number":81,"context_line":"[app:public_version_service]"}],"source_content_type":"text/x-properties","patch_set":3,"id":"5a2ca52d_4eff69aa","side":"PARENT","line":78,"in_reply_to":"5a2ca52d_eb4ae782","updated":"2015-10-14 18:12:23.000000000","message":"Out of band, i mentioned that this comment didn\u0027t need to be in the docs/ directory, but it should still be here.","commit_id":"8500d76e354b797e0f094127a7b7a1a264a3b646"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"86882eee8c4d397cbaef8d63caf61f7f0710159d","unresolved":false,"context_lines":[{"line_number":56,"context_line":""},{"line_number":57,"context_line":"[app:public_service]"},{"line_number":58,"context_line":"use \u003d egg:keystone#public_service"},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"[app:service_v3]"},{"line_number":61,"context_line":"use \u003d egg:keystone#service_v3"},{"line_number":62,"context_line":""}],"source_content_type":"text/x-properties","patch_set":3,"id":"5a2ca52d_0b4e4396","line":59,"updated":"2015-10-14 17:48:25.000000000","message":"the docs were talking about this value","commit_id":"507cee99d8eab4c14821b853931aead1032de85f"},{"author":{"_account_id":18338,"name":"Ron De Rose","email":"ronald.de.rose@intel.com","username":"derosenet"},"change_message_id":"3500adb9f5bcd9cb0ac98096f7bbab805ffcb48a","unresolved":false,"context_lines":[{"line_number":56,"context_line":""},{"line_number":57,"context_line":"[app:public_service]"},{"line_number":58,"context_line":"use \u003d egg:keystone#public_service"},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"[app:service_v3]"},{"line_number":61,"context_line":"use \u003d egg:keystone#service_v3"},{"line_number":62,"context_line":""}],"source_content_type":"text/x-properties","patch_set":3,"id":"5a2ca52d_4e8f69cc","line":59,"in_reply_to":"5a2ca52d_0b4e4396","updated":"2015-10-14 18:17:52.000000000","message":"Done","commit_id":"507cee99d8eab4c14821b853931aead1032de85f"},{"author":{"_account_id":18338,"name":"Ron De Rose","email":"ronald.de.rose@intel.com","username":"derosenet"},"change_message_id":"ad6ba8b923ba6473a55017f73af02d836988515d","unresolved":false,"context_lines":[{"line_number":56,"context_line":""},{"line_number":57,"context_line":"[app:public_service]"},{"line_number":58,"context_line":"use \u003d egg:keystone#public_service"},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"[app:service_v3]"},{"line_number":61,"context_line":"use \u003d egg:keystone#service_v3"},{"line_number":62,"context_line":""}],"source_content_type":"text/x-properties","patch_set":3,"id":"5a2ca52d_51162ef2","line":59,"in_reply_to":"5a2ca52d_4e8f69cc","updated":"2015-10-14 18:30:27.000000000","message":"Okay, so this value now matches the ini file.","commit_id":"507cee99d8eab4c14821b853931aead1032de85f"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"86882eee8c4d397cbaef8d63caf61f7f0710159d","unresolved":false,"context_lines":[{"line_number":69,"context_line":"pipeline \u003d sizelimit url_normalize request_id build_auth_context token_auth admin_token_auth json_body ec2_extension user_crud_extension public_service"},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"[pipeline:admin_api]"},{"line_number":72,"context_line":"# The last item in this pipeline must be admin_service or an equivalent"},{"line_number":73,"context_line":"# application. It cannot be a filter."},{"line_number":74,"context_line":"pipeline \u003d sizelimit url_normalize request_id build_auth_context token_auth admin_token_auth json_body ec2_extension s3_extension crud_extension admin_service"},{"line_number":75,"context_line":""}],"source_content_type":"text/x-properties","patch_set":3,"id":"5a2ca52d_4b543b20","line":72,"updated":"2015-10-14 17:48:25.000000000","message":"these pipelines still have the comment, and moreover, it\u0027s still valid","commit_id":"507cee99d8eab4c14821b853931aead1032de85f"},{"author":{"_account_id":18338,"name":"Ron De Rose","email":"ronald.de.rose@intel.com","username":"derosenet"},"change_message_id":"3500adb9f5bcd9cb0ac98096f7bbab805ffcb48a","unresolved":false,"context_lines":[{"line_number":69,"context_line":"pipeline \u003d sizelimit url_normalize request_id build_auth_context token_auth admin_token_auth json_body ec2_extension user_crud_extension public_service"},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"[pipeline:admin_api]"},{"line_number":72,"context_line":"# The last item in this pipeline must be admin_service or an equivalent"},{"line_number":73,"context_line":"# application. It cannot be a filter."},{"line_number":74,"context_line":"pipeline \u003d sizelimit url_normalize request_id build_auth_context token_auth admin_token_auth json_body ec2_extension s3_extension crud_extension admin_service"},{"line_number":75,"context_line":""}],"source_content_type":"text/x-properties","patch_set":3,"id":"5a2ca52d_aed15dcf","line":72,"in_reply_to":"5a2ca52d_4b543b20","updated":"2015-10-14 18:17:52.000000000","message":"Reverted the comment changes in the ini file","commit_id":"507cee99d8eab4c14821b853931aead1032de85f"},{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"679474d56ef82a8b5c8280c42b7876e6f037ecb0","unresolved":false,"context_lines":[{"line_number":74,"context_line":"pipeline \u003d sizelimit url_normalize request_id build_auth_context token_auth admin_token_auth json_body ec2_extension s3_extension crud_extension admin_service"},{"line_number":75,"context_line":""},{"line_number":76,"context_line":"[pipeline:api_v3]"},{"line_number":77,"context_line":"# The last item in this pipeline must be service_v3 or an equivalent \t\t"},{"line_number":78,"context_line":"# application. It cannot be a filter."},{"line_number":79,"context_line":"pipeline \u003d sizelimit url_normalize request_id build_auth_context token_auth admin_token_auth json_body ec2_extension_v3 s3_extension simple_cert_extension revoke_extension federation_extension oauth1_extension endpoint_filter_extension service_v3"},{"line_number":80,"context_line":""}],"source_content_type":"text/x-properties","patch_set":4,"id":"5a2ca52d_ee4af582","line":77,"updated":"2015-10-14 18:14:46.000000000","message":"Still need to undo ^\n\nIn your editor, enable visual markers for both trailing whitespace and tabs.","commit_id":"2244832d234397caf7e321f24e9628002f32d6f6"},{"author":{"_account_id":18338,"name":"Ron De Rose","email":"ronald.de.rose@intel.com","username":"derosenet"},"change_message_id":"e0c045441f502be4dd36a6938adf9a3da5143567","unresolved":false,"context_lines":[{"line_number":74,"context_line":"pipeline \u003d sizelimit url_normalize request_id build_auth_context token_auth admin_token_auth json_body ec2_extension s3_extension crud_extension admin_service"},{"line_number":75,"context_line":""},{"line_number":76,"context_line":"[pipeline:api_v3]"},{"line_number":77,"context_line":"# The last item in this pipeline must be service_v3 or an equivalent \t\t"},{"line_number":78,"context_line":"# application. It cannot be a filter."},{"line_number":79,"context_line":"pipeline \u003d sizelimit url_normalize request_id build_auth_context token_auth admin_token_auth json_body ec2_extension_v3 s3_extension simple_cert_extension revoke_extension federation_extension oauth1_extension endpoint_filter_extension service_v3"},{"line_number":80,"context_line":""}],"source_content_type":"text/x-properties","patch_set":4,"id":"5a2ca52d_b420d09c","line":77,"in_reply_to":"5a2ca52d_ee4af582","updated":"2015-10-14 18:52:26.000000000","message":"Done","commit_id":"2244832d234397caf7e321f24e9628002f32d6f6"}]}
