)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"4cd71c1ef6b9b3ede7c8a82233dd96414e777f27","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Anthony Washington \u003canthony.washington@intel.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2017-03-20 17:11:16 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Minor cleanup for 435609"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"With the merge of the [0] and the idea to add"},{"line_number":10,"context_line":"more description to the generated policies. This patch changes"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1a1ced50_16db131c","line":7,"updated":"2017-03-22 20:10:50.000000000","message":"This is technically implementing a new specification if we not only change the name but also fill in the description and operation attributes of the new DocumentedRuleClass. The way this is worded makes it seem like a follow-on thing to the policy-in-code implementation, but it\u0027s also adding other stuff specific to another spec. This would at least need to be reworded and link to the appropriate spec.","commit_id":"1936495c784a34055d6a70ae7647bf99f4d30492"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"4cd71c1ef6b9b3ede7c8a82233dd96414e777f27","unresolved":false,"context_lines":[{"line_number":12,"context_line":"which supports the more descriptive approach."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"[0] : https://review.openstack.org/#/c/439070/"},{"line_number":15,"context_line":"Partially-Implements: bp policy-in-code"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: Ieff3682b8b561d76e362b4df9917d1a399f5b420"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1a1ced50_f63d972b","line":15,"updated":"2017-03-22 20:10:50.000000000","message":"this should be: \n\n  bp policy-docs","commit_id":"1936495c784a34055d6a70ae7647bf99f4d30492"}],"keystone/common/policies/endpoint.py":[{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"4cd71c1ef6b9b3ede7c8a82233dd96414e777f27","unresolved":false,"context_lines":[{"line_number":20,"context_line":"        check_str\u003dbase.RULE_ADMIN_REQUIRED,"},{"line_number":21,"context_line":"        description\u003d\u0027Show endpoint details\u0027,"},{"line_number":22,"context_line":"        operations\u003d[{\u0027path\u0027: \u0027/v3/endpoints/{endpoint_id}\u0027,"},{"line_number":23,"context_line":"                     \u0027method\u0027: \u0027GET\u0027}]),"},{"line_number":24,"context_line":"    policy.DocumentedRuleDefault("},{"line_number":25,"context_line":"        name\u003dbase.IDENTITY % \u0027list_endpoints\u0027,"},{"line_number":26,"context_line":"        check_str\u003dbase.RULE_ADMIN_REQUIRED,"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a1ced50_967b837a","line":23,"updated":"2017-03-22 20:10:50.000000000","message":"I\u0027d keep this as a separate patch - since it\u0027s also addressing a separate spec.","commit_id":"1936495c784a34055d6a70ae7647bf99f4d30492"}]}