)]}'
{"README.rst":[{"author":{"_account_id":23185,"name":"Samantha Blanco","email":"samantha.blanco@att.com","username":"sblanco"},"change_message_id":"5239355dbf945ce2fdc757b95141acec4eb5acb5","unresolved":false,"context_lines":[{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Features"},{"line_number":21,"context_line":"--------"},{"line_number":22,"context_line":"* Validation of in-code and in-file policy definitions."},{"line_number":23,"context_line":"* Validation of custom policy file definitions that override default policy"},{"line_number":24,"context_line":"  definitions."},{"line_number":25,"context_line":"* Built-in positive and negative testing. Positive and negative testing"}],"source_content_type":"text/x-rst","patch_set":4,"id":"1f1a1f67_2398ed67","line":22,"range":{"start_line":22,"start_character":28,"end_line":22,"end_character":35},"updated":"2017-07-18 20:43:48.000000000","message":"Would \"default\" be better here? Something about this is confusing.","commit_id":"8df586dd1b67b1f3e836fa0d00bcff258ad780e7"},{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"93c8da10b183d47e60182c52fb5560609f849a90","unresolved":false,"context_lines":[{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Features"},{"line_number":21,"context_line":"--------"},{"line_number":22,"context_line":"* Validation of in-code and in-file policy definitions."},{"line_number":23,"context_line":"* Validation of custom policy file definitions that override default policy"},{"line_number":24,"context_line":"  definitions."},{"line_number":25,"context_line":"* Built-in positive and negative testing. Positive and negative testing"}],"source_content_type":"text/x-rst","patch_set":4,"id":"1f1a1f67_fd628dad","line":22,"range":{"start_line":22,"start_character":28,"end_line":22,"end_character":35},"in_reply_to":"1f1a1f67_2398ed67","updated":"2017-07-18 22:21:14.000000000","message":"Done","commit_id":"8df586dd1b67b1f3e836fa0d00bcff258ad780e7"},{"author":{"_account_id":23185,"name":"Samantha Blanco","email":"samantha.blanco@att.com","username":"sblanco"},"change_message_id":"5239355dbf945ce2fdc757b95141acec4eb5acb5","unresolved":false,"context_lines":[{"line_number":23,"context_line":"* Validation of custom policy file definitions that override default policy"},{"line_number":24,"context_line":"  definitions."},{"line_number":25,"context_line":"* Built-in positive and negative testing. Positive and negative testing"},{"line_number":26,"context_line":"  is performed using the same tests and role-switching."},{"line_number":27,"context_line":"* Valdation of custom roles as well as default OpenStack roles (\"admin\""},{"line_number":28,"context_line":"  and \"Member\" for example)."},{"line_number":29,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"1f1a1f67_c3a01993","line":26,"range":{"start_line":26,"start_character":2,"end_line":26,"end_character":4},"updated":"2017-07-18 20:43:48.000000000","message":"are","commit_id":"8df586dd1b67b1f3e836fa0d00bcff258ad780e7"},{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"93c8da10b183d47e60182c52fb5560609f849a90","unresolved":false,"context_lines":[{"line_number":23,"context_line":"* Validation of custom policy file definitions that override default policy"},{"line_number":24,"context_line":"  definitions."},{"line_number":25,"context_line":"* Built-in positive and negative testing. Positive and negative testing"},{"line_number":26,"context_line":"  is performed using the same tests and role-switching."},{"line_number":27,"context_line":"* Valdation of custom roles as well as default OpenStack roles (\"admin\""},{"line_number":28,"context_line":"  and \"Member\" for example)."},{"line_number":29,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"1f1a1f67_1d4e6937","line":26,"range":{"start_line":26,"start_character":2,"end_line":26,"end_character":4},"in_reply_to":"1f1a1f67_c3a01993","updated":"2017-07-18 22:21:14.000000000","message":"Done","commit_id":"8df586dd1b67b1f3e836fa0d00bcff258ad780e7"},{"author":{"_account_id":23185,"name":"Samantha Blanco","email":"samantha.blanco@att.com","username":"sblanco"},"change_message_id":"5239355dbf945ce2fdc757b95141acec4eb5acb5","unresolved":false,"context_lines":[{"line_number":24,"context_line":"  definitions."},{"line_number":25,"context_line":"* Built-in positive and negative testing. Positive and negative testing"},{"line_number":26,"context_line":"  is performed using the same tests and role-switching."},{"line_number":27,"context_line":"* Valdation of custom roles as well as default OpenStack roles (\"admin\""},{"line_number":28,"context_line":"  and \"Member\" for example)."},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"How It Works"},{"line_number":31,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":4,"id":"1f1a1f67_83a6a1aa","line":28,"range":{"start_line":27,"start_character":63,"end_line":28,"end_character":27},"updated":"2017-07-18 20:43:48.000000000","message":"I think this can be removed; either that or include an example of a custom role.","commit_id":"8df586dd1b67b1f3e836fa0d00bcff258ad780e7"},{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"93c8da10b183d47e60182c52fb5560609f849a90","unresolved":false,"context_lines":[{"line_number":24,"context_line":"  definitions."},{"line_number":25,"context_line":"* Built-in positive and negative testing. Positive and negative testing"},{"line_number":26,"context_line":"  is performed using the same tests and role-switching."},{"line_number":27,"context_line":"* Valdation of custom roles as well as default OpenStack roles (\"admin\""},{"line_number":28,"context_line":"  and \"Member\" for example)."},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"How It Works"},{"line_number":31,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":4,"id":"1f1a1f67_3d5365e4","line":28,"range":{"start_line":27,"start_character":63,"end_line":28,"end_character":27},"in_reply_to":"1f1a1f67_83a6a1aa","updated":"2017-07-18 22:21:14.000000000","message":"Done","commit_id":"8df586dd1b67b1f3e836fa0d00bcff258ad780e7"},{"author":{"_account_id":23185,"name":"Samantha Blanco","email":"samantha.blanco@att.com","username":"sblanco"},"change_message_id":"5239355dbf945ce2fdc757b95141acec4eb5acb5","unresolved":false,"context_lines":[{"line_number":33,"context_line":"determine whether a given role is allowed to perform a policy action, given a"},{"line_number":34,"context_line":"specific role and OpenStack service. The output from ``oslo.policy``, the"},{"line_number":35,"context_line":"expected result, and the actual result from test execution are compared to"},{"line_number":36,"context_line":"each other: if the two match, then the test result is successful."},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"* Documentation: https://docs.openstack.org/developer/patrole"},{"line_number":39,"context_line":"* Bugs: https://bugs.launchpad.net/patrole"}],"source_content_type":"text/x-rst","patch_set":4,"id":"1f1a1f67_03b4d1cf","line":36,"range":{"start_line":36,"start_character":19,"end_line":36,"end_character":22},"updated":"2017-07-18 20:43:48.000000000","message":"Three things are listed","commit_id":"8df586dd1b67b1f3e836fa0d00bcff258ad780e7"},{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"93c8da10b183d47e60182c52fb5560609f849a90","unresolved":false,"context_lines":[{"line_number":33,"context_line":"determine whether a given role is allowed to perform a policy action, given a"},{"line_number":34,"context_line":"specific role and OpenStack service. The output from ``oslo.policy``, the"},{"line_number":35,"context_line":"expected result, and the actual result from test execution are compared to"},{"line_number":36,"context_line":"each other: if the two match, then the test result is successful."},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"* Documentation: https://docs.openstack.org/developer/patrole"},{"line_number":39,"context_line":"* Bugs: https://bugs.launchpad.net/patrole"}],"source_content_type":"text/x-rst","patch_set":4,"id":"1f1a1f67_fd396da3","line":36,"range":{"start_line":36,"start_character":19,"end_line":36,"end_character":22},"in_reply_to":"1f1a1f67_03b4d1cf","updated":"2017-07-18 22:21:14.000000000","message":"It\u0027s actually two. The output from ``oslo.policy`` is the same as the expected result. I\u0027ll make it clearer in the next patch.","commit_id":"8df586dd1b67b1f3e836fa0d00bcff258ad780e7"}]}