)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"d0bd2c7f5e0afc1f4db7a73b3fc1417b38a29e9c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"d7d2dbc9_ade2239f","updated":"2023-02-16 21:56:37.000000000","message":"@Slawek, we need to backport this to applicable stable branches ","commit_id":"91759b17ea5d4d6563596612208e6014dee2ed88"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"fac6993ee64f2438c3cec07c41c86580aff6a186","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"1d97a814_ed5487b5","updated":"2022-11-15 16:56:00.000000000","message":"Waiting for the CI but code looks OK.","commit_id":"91759b17ea5d4d6563596612208e6014dee2ed88"}],"neutron_lib/tests/unit/db/test_utils.py":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"2d1d2d2bf19d258e84d39c404dcfebbf455e894f","unresolved":true,"context_lines":[{"line_number":155,"context_line":"class TestUtilsWithScopeEnforcement(TestUtilsLegacyPolicies):"},{"line_number":156,"context_line":""},{"line_number":157,"context_line":"    def setUp(self):"},{"line_number":158,"context_line":"        cfg.CONF.set_override(\u0027enforce_scope\u0027, True, group\u003d\u0027oslo_policy\u0027)"},{"line_number":159,"context_line":"        super().setUp()"}],"source_content_type":"text/x-python","patch_set":1,"id":"cde036eb_e0ca10fa","line":158,"range":{"start_line":158,"start_character":8,"end_line":158,"end_character":73},"updated":"2022-11-15 14:56:37.000000000","message":"Once L159 is called, the TestUtilsLegacyPolicies.setUp method will change back again the enforce_scope value. We should override this value after the super().setUp() call.","commit_id":"ea5005b5af4eaef5219dd9a763b409e253da135d"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"a7d6b432893cb1a41ae5cf0cef319e41feac1a22","unresolved":false,"context_lines":[{"line_number":155,"context_line":"class TestUtilsWithScopeEnforcement(TestUtilsLegacyPolicies):"},{"line_number":156,"context_line":""},{"line_number":157,"context_line":"    def setUp(self):"},{"line_number":158,"context_line":"        cfg.CONF.set_override(\u0027enforce_scope\u0027, True, group\u003d\u0027oslo_policy\u0027)"},{"line_number":159,"context_line":"        super().setUp()"}],"source_content_type":"text/x-python","patch_set":1,"id":"e876bcf0_d8534821","line":158,"range":{"start_line":158,"start_character":8,"end_line":158,"end_character":73},"in_reply_to":"cde036eb_e0ca10fa","updated":"2022-11-15 15:37:40.000000000","message":"Yes, done 😊","commit_id":"ea5005b5af4eaef5219dd9a763b409e253da135d"},{"author":{"_account_id":34271,"name":"Miro Tomaska","display_name":"Miro Tomaska","email":"mtomaska@redhat.com","username":"mtomaska"},"change_message_id":"73dd769027c2fcf0e175d0e8b20f986dac97df63","unresolved":true,"context_lines":[{"line_number":152,"context_line":"            utils.model_query_scope_is_project(ctx, model))"},{"line_number":153,"context_line":""},{"line_number":154,"context_line":""},{"line_number":155,"context_line":"class TestUtilsWithScopeEnforcement(TestUtilsLegacyPolicies):"},{"line_number":156,"context_line":""},{"line_number":157,"context_line":"    def setUp(self):"},{"line_number":158,"context_line":"        cfg.CONF.set_override(\u0027enforce_scope\u0027, True, group\u003d\u0027oslo_policy\u0027)"},{"line_number":159,"context_line":"        super().setUp()"}],"source_content_type":"text/x-python","patch_set":1,"id":"4ae44f4c_ab76abfd","line":159,"range":{"start_line":155,"start_character":0,"end_line":159,"end_character":23},"updated":"2022-11-11 16:06:39.000000000","message":"Not clear to me what is this lone class going to do? Maybe I am just not seeing something","commit_id":"ea5005b5af4eaef5219dd9a763b409e253da135d"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"2d1d2d2bf19d258e84d39c404dcfebbf455e894f","unresolved":false,"context_lines":[{"line_number":152,"context_line":"            utils.model_query_scope_is_project(ctx, model))"},{"line_number":153,"context_line":""},{"line_number":154,"context_line":""},{"line_number":155,"context_line":"class TestUtilsWithScopeEnforcement(TestUtilsLegacyPolicies):"},{"line_number":156,"context_line":""},{"line_number":157,"context_line":"    def setUp(self):"},{"line_number":158,"context_line":"        cfg.CONF.set_override(\u0027enforce_scope\u0027, True, group\u003d\u0027oslo_policy\u0027)"},{"line_number":159,"context_line":"        super().setUp()"}],"source_content_type":"text/x-python","patch_set":1,"id":"76849c31_5cb9d4a9","line":159,"range":{"start_line":155,"start_character":0,"end_line":159,"end_character":23},"in_reply_to":"4ae44f4c_ab76abfd","updated":"2022-11-15 14:56:37.000000000","message":"Executing the same tests defined in TestUtilsLegacyPolicies but with enforce_scope\u003dTrue","commit_id":"ea5005b5af4eaef5219dd9a763b409e253da135d"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"a7d6b432893cb1a41ae5cf0cef319e41feac1a22","unresolved":false,"context_lines":[{"line_number":152,"context_line":"            utils.model_query_scope_is_project(ctx, model))"},{"line_number":153,"context_line":""},{"line_number":154,"context_line":""},{"line_number":155,"context_line":"class TestUtilsWithScopeEnforcement(TestUtilsLegacyPolicies):"},{"line_number":156,"context_line":""},{"line_number":157,"context_line":"    def setUp(self):"},{"line_number":158,"context_line":"        cfg.CONF.set_override(\u0027enforce_scope\u0027, True, group\u003d\u0027oslo_policy\u0027)"},{"line_number":159,"context_line":"        super().setUp()"}],"source_content_type":"text/x-python","patch_set":1,"id":"6f07bd41_2e312a3e","line":159,"range":{"start_line":155,"start_character":0,"end_line":159,"end_character":23},"in_reply_to":"76849c31_5cb9d4a9","updated":"2022-11-15 15:37:40.000000000","message":"exactly as Rodolfo said. It will execute same tests with different value of that config option to ensure it works fine in both cases","commit_id":"ea5005b5af4eaef5219dd9a763b409e253da135d"}]}
