)]}'
{"neutron/conf/policies/security_group.py":[{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"e484fd09f9e5387cfaeb3ce658ee5850604ffc70","unresolved":true,"context_lines":[{"line_number":138,"context_line":"    ),"},{"line_number":139,"context_line":"    policy.DocumentedRuleDefault("},{"line_number":140,"context_line":"        name\u003d\u0027get_security_group_rule\u0027,"},{"line_number":141,"context_line":"        check_str\u003dbase.SYSTEM_OR_PROJECT_READER,"},{"line_number":142,"context_line":"        scope_types\u003d[\u0027system\u0027, \u0027project\u0027],"},{"line_number":143,"context_line":"        description\u003d\u0027Get a security group rule\u0027,"},{"line_number":144,"context_line":"        operations\u003d["}],"source_content_type":"text/x-python","patch_set":2,"id":"884c3c81_5b65b50d","line":141,"updated":"2020-12-08 10:01:58.000000000","message":"I think that we should add here some rule like \"SG_OWNER\" and add it to the check_str, wdyt?","commit_id":"5e982d4e218159edee16bd7c42240f583ac86c14"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"a5bb0f4f8f659e1b9e5e8c0f25c1a1b5bc8876c8","unresolved":false,"context_lines":[{"line_number":138,"context_line":"    ),"},{"line_number":139,"context_line":"    policy.DocumentedRuleDefault("},{"line_number":140,"context_line":"        name\u003d\u0027get_security_group_rule\u0027,"},{"line_number":141,"context_line":"        check_str\u003dbase.SYSTEM_OR_PROJECT_READER,"},{"line_number":142,"context_line":"        scope_types\u003d[\u0027system\u0027, \u0027project\u0027],"},{"line_number":143,"context_line":"        description\u003d\u0027Get a security group rule\u0027,"},{"line_number":144,"context_line":"        operations\u003d["}],"source_content_type":"text/x-python","patch_set":2,"id":"3ab45043_c3b78939","line":141,"in_reply_to":"884c3c81_5b65b50d","updated":"2020-12-17 12:08:38.000000000","message":"Done","commit_id":"5e982d4e218159edee16bd7c42240f583ac86c14"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"367be093ab3e6ecf8ef95af395142fdb0c662bce","unresolved":true,"context_lines":[{"line_number":30,"context_line":""},{"line_number":31,"context_line":"rules \u003d ["},{"line_number":32,"context_line":"    policy.RuleDefault("},{"line_number":33,"context_line":"        name\u003d\u0027admin_or_sg_owner\u0027,"},{"line_number":34,"context_line":"        check_str\u003dbase.policy_or("},{"line_number":35,"context_line":"            \u0027rule:context_is_admin\u0027,"},{"line_number":36,"context_line":"            \u0027tenant_id:%(security_group:tenant_id)s\u0027),"}],"source_content_type":"text/x-python","patch_set":3,"id":"91e223f2_e6d7609a","line":33,"range":{"start_line":33,"start_character":14,"end_line":33,"end_character":31},"updated":"2020-12-16 09:31:12.000000000","message":"Just asking to learn: shouldn\u0027t we migrate those rules too?","commit_id":"7b2b52c62d19040871f0baff6c45b7b699528192"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"e3fec1c4ece588f612ac4250c61e71c7a2ac4193","unresolved":true,"context_lines":[{"line_number":30,"context_line":""},{"line_number":31,"context_line":"rules \u003d ["},{"line_number":32,"context_line":"    policy.RuleDefault("},{"line_number":33,"context_line":"        name\u003d\u0027admin_or_sg_owner\u0027,"},{"line_number":34,"context_line":"        check_str\u003dbase.policy_or("},{"line_number":35,"context_line":"            \u0027rule:context_is_admin\u0027,"},{"line_number":36,"context_line":"            \u0027tenant_id:%(security_group:tenant_id)s\u0027),"},{"line_number":37,"context_line":"        description\u003d\u0027Rule for admin or security group owner access\u0027),"},{"line_number":38,"context_line":"    policy.RuleDefault("},{"line_number":39,"context_line":"        name\u003d\u0027admin_owner_or_sg_owner\u0027,"}],"source_content_type":"text/x-python","patch_set":4,"id":"ffd3da95_3e1933a2","line":36,"range":{"start_line":33,"start_character":8,"end_line":36,"end_character":54},"updated":"2021-01-22 15:22:26.000000000","message":"nit: These are useful but not related changes, so I understand them but make it harder to see the \"important\" changes","commit_id":"e302cba68a13a3b92180c37ac9331f9855e921f1"}]}