)]}'
{"neutron/conf/policies/metering.py":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"fd6dc4a74cbeadfca324adf07dcda21aefa0707d","unresolved":true,"context_lines":[{"line_number":29,"context_line":"rules \u003d ["},{"line_number":30,"context_line":"    policy.DocumentedRuleDefault("},{"line_number":31,"context_line":"        name\u003d\u0027create_metering_label\u0027,"},{"line_number":32,"context_line":"        check_str\u003dbase.SYSTEM_ADMIN,"},{"line_number":33,"context_line":"        scope_types\u003d[\u0027system\u0027, \u0027project\u0027],"},{"line_number":34,"context_line":"        description\u003d\u0027Create a metering label\u0027,"},{"line_number":35,"context_line":"        operations\u003d["}],"source_content_type":"text/x-python","patch_set":4,"id":"88516983_2f6b5912","line":32,"range":{"start_line":32,"start_character":23,"end_line":32,"end_character":35},"updated":"2020-12-24 15:49:07.000000000","message":"Shouldn\u0027t be PROJECT_MEMBER?","commit_id":"1bca8d110295fae601b168cfc523b43321abb91f"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"92b269cd5b5781d350eb18509a4ce8d6620a31d5","unresolved":true,"context_lines":[{"line_number":29,"context_line":"rules \u003d ["},{"line_number":30,"context_line":"    policy.DocumentedRuleDefault("},{"line_number":31,"context_line":"        name\u003d\u0027create_metering_label\u0027,"},{"line_number":32,"context_line":"        check_str\u003dbase.SYSTEM_ADMIN,"},{"line_number":33,"context_line":"        scope_types\u003d[\u0027system\u0027, \u0027project\u0027],"},{"line_number":34,"context_line":"        description\u003d\u0027Create a metering label\u0027,"},{"line_number":35,"context_line":"        operations\u003d["}],"source_content_type":"text/x-python","patch_set":4,"id":"c60eda76_a50b67b7","line":32,"range":{"start_line":32,"start_character":23,"end_line":32,"end_character":35},"in_reply_to":"88516983_2f6b5912","updated":"2021-01-21 16:17:45.000000000","message":"Deprecated rule is ADMIN_ONLY so I think it should be SYSTEM_ADMIN here","commit_id":"1bca8d110295fae601b168cfc523b43321abb91f"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"5e24afb67c370ba6cb91dd0bf4ca79c0f3e1cbbf","unresolved":true,"context_lines":[{"line_number":29,"context_line":"rules \u003d ["},{"line_number":30,"context_line":"    policy.DocumentedRuleDefault("},{"line_number":31,"context_line":"        name\u003d\u0027create_metering_label\u0027,"},{"line_number":32,"context_line":"        check_str\u003dbase.SYSTEM_ADMIN,"},{"line_number":33,"context_line":"        scope_types\u003d[\u0027system\u0027, \u0027project\u0027],"},{"line_number":34,"context_line":"        description\u003d\u0027Create a metering label\u0027,"},{"line_number":35,"context_line":"        operations\u003d["}],"source_content_type":"text/x-python","patch_set":4,"id":"9791b985_e2477d8f","line":32,"range":{"start_line":32,"start_character":23,"end_line":32,"end_character":35},"in_reply_to":"c60eda76_a50b67b7","updated":"2021-01-22 04:49:20.000000000","message":"++\n\nWe can always relax policies in the future, if appropriate and depending in the API, what information it exposes, and if any of that information can be used to violate tenancy (e.g., could a project user misuse this information?)","commit_id":"1bca8d110295fae601b168cfc523b43321abb91f"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"fd6dc4a74cbeadfca324adf07dcda21aefa0707d","unresolved":true,"context_lines":[{"line_number":46,"context_line":"    ),"},{"line_number":47,"context_line":"    policy.DocumentedRuleDefault("},{"line_number":48,"context_line":"        name\u003d\u0027get_metering_label\u0027,"},{"line_number":49,"context_line":"        check_str\u003dbase.SYSTEM_READER,"},{"line_number":50,"context_line":"        scope_types\u003d[\u0027system\u0027, \u0027project\u0027],"},{"line_number":51,"context_line":"        description\u003d\u0027Get a metering label\u0027,"},{"line_number":52,"context_line":"        operations\u003d["}],"source_content_type":"text/x-python","patch_set":4,"id":"2c92722c_002dff1a","line":49,"range":{"start_line":49,"start_character":18,"end_line":49,"end_character":36},"updated":"2020-12-24 15:49:07.000000000","message":"shouldn\u0027t be SYSTEM_OR_PROJECT_READER?","commit_id":"1bca8d110295fae601b168cfc523b43321abb91f"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"92b269cd5b5781d350eb18509a4ce8d6620a31d5","unresolved":true,"context_lines":[{"line_number":46,"context_line":"    ),"},{"line_number":47,"context_line":"    policy.DocumentedRuleDefault("},{"line_number":48,"context_line":"        name\u003d\u0027get_metering_label\u0027,"},{"line_number":49,"context_line":"        check_str\u003dbase.SYSTEM_READER,"},{"line_number":50,"context_line":"        scope_types\u003d[\u0027system\u0027, \u0027project\u0027],"},{"line_number":51,"context_line":"        description\u003d\u0027Get a metering label\u0027,"},{"line_number":52,"context_line":"        operations\u003d["}],"source_content_type":"text/x-python","patch_set":4,"id":"d1e136d8_d865d9ee","line":49,"range":{"start_line":49,"start_character":18,"end_line":49,"end_character":36},"in_reply_to":"2c92722c_002dff1a","updated":"2021-01-21 16:17:45.000000000","message":"deprecated rule was ADMIN_ONLY so, as it\u0027s \"GET\" IMO it should be SYSTEM_READER","commit_id":"1bca8d110295fae601b168cfc523b43321abb91f"}]}