)]}'
{"placement/policies/usage.py":[{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"af3dd46c68a443c9d1cab1108f892ef308ea2dd7","unresolved":false,"context_lines":[{"line_number":64,"context_line":"            }"},{"line_number":65,"context_line":"        ],"},{"line_number":66,"context_line":"        scope_types\u003d[\u0027system\u0027]"},{"line_number":67,"context_line":"        deprecated_rule\u003ddeprecated_list_total_usages,"},{"line_number":68,"context_line":"        deprecated_reason\u003dDEPRECATED_REASON,"},{"line_number":69,"context_line":"        deprecated_since\u003dversionutils.deprecated.WALLABY)"},{"line_number":70,"context_line":"]"}],"source_content_type":"text/x-python","patch_set":1,"id":"1f621f24_6b93d750","line":67,"updated":"2020-10-29 06:00:05.000000000","message":"pep8: E901 SyntaxError: invalid syntax","commit_id":"4cd2c4851b92e0b5a8b3a61f7f3765ca2bbfbc87"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"ed59e5587de1abf906d898d43406d02d4ab61b08","unresolved":false,"context_lines":[{"line_number":64,"context_line":"            }"},{"line_number":65,"context_line":"        ],"},{"line_number":66,"context_line":"        scope_types\u003d[\u0027system\u0027]"},{"line_number":67,"context_line":"        deprecated_rule\u003ddeprecated_list_total_usages,"},{"line_number":68,"context_line":"        deprecated_reason\u003dDEPRECATED_REASON,"},{"line_number":69,"context_line":"        deprecated_since\u003dversionutils.deprecated.WALLABY)"},{"line_number":70,"context_line":"]"}],"source_content_type":"text/x-python","patch_set":1,"id":"1f621f24_38e1269c","line":67,"in_reply_to":"1f621f24_6b93d750","updated":"2020-10-29 19:41:20.000000000","message":"Done","commit_id":"4cd2c4851b92e0b5a8b3a61f7f3765ca2bbfbc87"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"ac84e796b54669ea7bf4d6188ccbd9f13a7d8981","unresolved":true,"context_lines":[{"line_number":53,"context_line":"        # TODO(mriedem): At some point we might set scope_types\u003d[\u0027project\u0027]"},{"line_number":54,"context_line":"        # so that non-admin project-scoped token users can query usages for"},{"line_number":55,"context_line":"        # their project. The context.can() target will need to change as well"},{"line_number":56,"context_line":"        # in the actual policy enforcement check in the handler code."},{"line_number":57,"context_line":"        name\u003dTOTAL_USAGES,"},{"line_number":58,"context_line":"        check_str\u003dbase.SYSTEM_READER,"},{"line_number":59,"context_line":"        description\u003d\"List total resource usages for a given project.\","}],"source_content_type":"text/x-python","patch_set":7,"id":"9f689bbe_f216009e","line":56,"updated":"2021-01-22 10:37:40.000000000","message":"While we\u0027re undertaking this work, it feels like we should resolve this issue. That seems reasonable, right?","commit_id":"127384969c5b335a2eed3de7fc9821876c8c2551"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"3e5d06f474e2a9d95468289df6960e32a22c170d","unresolved":true,"context_lines":[{"line_number":53,"context_line":"        # TODO(mriedem): At some point we might set scope_types\u003d[\u0027project\u0027]"},{"line_number":54,"context_line":"        # so that non-admin project-scoped token users can query usages for"},{"line_number":55,"context_line":"        # their project. The context.can() target will need to change as well"},{"line_number":56,"context_line":"        # in the actual policy enforcement check in the handler code."},{"line_number":57,"context_line":"        name\u003dTOTAL_USAGES,"},{"line_number":58,"context_line":"        check_str\u003dbase.SYSTEM_READER,"},{"line_number":59,"context_line":"        description\u003d\"List total resource usages for a given project.\","}],"source_content_type":"text/x-python","patch_set":7,"id":"72c3c1ba_dc2d70a0","line":56,"in_reply_to":"9f689bbe_f216009e","updated":"2021-01-22 11:45:24.000000000","message":"I had a punt at this [1] and failed miserably. If you have a chance to take a look and can figure out what daft thing(s) I\u0027ve done, I\u0027d appreciate it /o\\\n\nhttps://review.opendev.org/c/openstack/placement/+/771964","commit_id":"127384969c5b335a2eed3de7fc9821876c8c2551"}],"placement/tests/functional/gabbits/usage-legacy-rbac.yaml":[{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"f3eef995bc5d915ed98e2535bec36bbeaf8e3b73","unresolved":true,"context_lines":[{"line_number":17,"context_line":"    x-project-id: *project_id"},{"line_number":18,"context_line":"    accept: application/json"},{"line_number":19,"context_line":"    content-type: application/json"},{"line_number":20,"context_line":"    openstack-api-version: placement latest"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"tests:"},{"line_number":23,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":10,"id":"817f2fab_6a8ee422","line":20,"updated":"2021-01-26 14:44:13.000000000","message":"Same comment about the missing personas in test coverage for legacy RBAC enforcement.","commit_id":"aeff7fa7016b3b76b029d325cf1278de1b1fd42b"}],"placement/tests/functional/gabbits/usage-secure-rbac.yaml":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"9a74f29b83997da4bad98159d220153fd3a1c2e6","unresolved":true,"context_lines":[{"line_number":122,"context_line":"  response_json_paths:"},{"line_number":123,"context_line":"    usages: {}"},{"line_number":124,"context_line":""},{"line_number":125,"context_line":"# Make sure users from other projects can\u0027t snoop around for usage on projects"},{"line_number":126,"context_line":"# they have no business knowing about."},{"line_number":127,"context_line":"- name: project admin cannot get total usage for unauthorized project"},{"line_number":128,"context_line":"  GET: /usages?project_id\u003d$ENVIRON[\u0027PROJECT_ID\u0027]"},{"line_number":129,"context_line":"  request_headers: *alt_project_admin_headers"}],"source_content_type":"text/x-yaml","patch_set":11,"id":"668bfa4b_f41248e7","line":126,"range":{"start_line":125,"start_character":0,"end_line":126,"end_character":38},"updated":"2021-02-02 19:19:01.000000000","message":"+1, these are good tests.","commit_id":"7a7365c7e015c3108c3f024fa5155e8dd51e9bdd"}]}
