)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"6b24399eb1fa70cc724bf650c735e5267ba271d4","unresolved":false,"context_lines":[{"line_number":21,"context_line":"7. attachment APIs"},{"line_number":22,"context_line":"8. message APIs"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"Partial-Implements: blueprint policy-in-code"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"Change-Id: Iedde7a9a774a60e760b47d5eb2973f42d79227d8"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":14,"id":"7f287b81_3e3bd59c","line":24,"updated":"2017-08-25 03:21:10.000000000","message":"We should track this work - either with the blueprint keyword here or in the Gerrit interface with the topic branch highlighted in the goal: \n\nhttps://governance.openstack.org/tc/goals/queens/policy-in-code.html#gerrit-topic","commit_id":"531193ed60cd0f83fffa9539194e60af2e1b77a2"}],"cinder/policies/snapshot_actions.py":[{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"6b24399eb1fa70cc724bf650c735e5267ba271d4","unresolved":false,"context_lines":[{"line_number":29,"context_line":"        operations\u003d["},{"line_number":30,"context_line":"            {"},{"line_number":31,"context_line":"                \u0027method\u0027: \u0027POST\u0027,"},{"line_number":32,"context_line":"                \u0027path\u0027: \u0027/snapshots/{snapshot_id}/action (os-reset_status)\u0027"},{"line_number":33,"context_line":"            }"},{"line_number":34,"context_line":"        ]),"},{"line_number":35,"context_line":"    policy.DocumentedRuleDefault("}],"source_content_type":"text/x-python","patch_set":14,"id":"7f287b81_fee0adfc","line":32,"updated":"2017-08-25 03:21:10.000000000","message":"What is os-reset_status? Is this describing an additional query parameter or something in the request body?","commit_id":"531193ed60cd0f83fffa9539194e60af2e1b77a2"}],"cinder/policy.py":[{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"6b24399eb1fa70cc724bf650c735e5267ba271d4","unresolved":false,"context_lines":[{"line_number":146,"context_line":"def authorize(context, action, target, do_raise\u003dTrue, exc\u003dNone):"},{"line_number":147,"context_line":"    \"\"\"Verifies that the action is valid on the target in this context."},{"line_number":148,"context_line":""},{"line_number":149,"context_line":"       :param context: nova context"},{"line_number":150,"context_line":"       :param action: string representing the action to be checked"},{"line_number":151,"context_line":"           this should be colon separated for clarity."},{"line_number":152,"context_line":"           i.e. ``compute:create_instance``,"}],"source_content_type":"text/x-python","patch_set":14,"id":"7f287b81_3e0995dc","line":149,"updated":"2017-08-25 03:21:10.000000000","message":"This should probably be cinder.","commit_id":"531193ed60cd0f83fffa9539194e60af2e1b77a2"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"6b24399eb1fa70cc724bf650c735e5267ba271d4","unresolved":false,"context_lines":[{"line_number":163,"context_line":"                   the exception class. If not specified,"},{"line_number":164,"context_line":"                   :class:`PolicyNotAuthorized` will be used."},{"line_number":165,"context_line":""},{"line_number":166,"context_line":"       :raises nova.exception.PolicyNotAuthorized: if verification fails"},{"line_number":167,"context_line":"           and do_raise is True. Or if \u0027exc\u0027 is specified it will raise an"},{"line_number":168,"context_line":"           exception of that type."},{"line_number":169,"context_line":""}],"source_content_type":"text/x-python","patch_set":14,"id":"7f287b81_7e695d89","line":166,"updated":"2017-08-25 03:21:10.000000000","message":"This should probably be cinder.","commit_id":"531193ed60cd0f83fffa9539194e60af2e1b77a2"}],"etc/cinder/policy.json":[{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"6b24399eb1fa70cc724bf650c735e5267ba271d4","unresolved":false,"context_lines":[{"line_number":1,"context_line":"{"},{"line_number":2,"context_line":"    \"admin_or_owner\":  \"is_admin:True or (role:admin and is_admin_project:True) or  project_id:%(project_id)s\","},{"line_number":3,"context_line":"    \"default\": \"rule:admin_or_owner\","},{"line_number":4,"context_line":""},{"line_number":5,"context_line":"    \"admin_api\": \"is_admin:True or (role:admin and is_admin_project:True)\","}],"source_content_type":"application/json","patch_set":14,"id":"7f287b81_5e6101a9","side":"PARENT","line":2,"updated":"2017-08-25 03:21:10.000000000","message":"These can be removed in this patch because they\u0027re being defined in base, correct?","commit_id":"4b45a359fb0d05ac752b8275fde9c388bd7b6d7a"}]}