)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"117fec274e94f8c1670a1a9d29ccc8afe9c2f159","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"0d912dc7_4e537e09","updated":"2022-02-28 18:38:44.000000000","message":"LGTM\nFixes the issues with oslo.context \u003e\u003d 4.0.0","commit_id":"1383c8c92f625cfef8d8f25ce0299c0928455de3"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"19a924f49d5e21343afc5e00c0f10670e745b832","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"1b69b28e_ba4e6b24","updated":"2022-02-11 15:12:08.000000000","message":"recheck","commit_id":"1383c8c92f625cfef8d8f25ce0299c0928455de3"}],"octavia/common/context.py":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"10c2ac52064abf0c151f20d747063d39e562fdfc","unresolved":true,"context_lines":[{"line_number":30,"context_line":"        is_admin \u003d (policy.get_enforcer().check_is_admin(self) or"},{"line_number":31,"context_line":"                    CONF.api_settings.auth_strategy \u003d\u003d constants.NOAUTH)"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"        super().__init__(is_admin\u003dis_admin, user_id\u003duser_id,"},{"line_number":34,"context_line":"                         project_id\u003dproject_id, **kwargs)"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"    @property"}],"source_content_type":"text/x-python","patch_set":4,"id":"bf5ea08c_86abfa75","line":33,"range":{"start_line":33,"start_character":44,"end_line":33,"end_character":60},"updated":"2021-07-22 17:24:20.000000000","message":"Interestingly the previous implementation didn\u0027t pass use_id, which eventually made the parameter ignored. I believe we should have passed this in the first place but I\u0027d appreciate if anybody know background of that behavior.","commit_id":"60fef6eb03696c50a88924b5e6382c17627df509"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"3d140f857eba89c4dea83f52c3524960a80eea0d","unresolved":false,"context_lines":[{"line_number":30,"context_line":"        is_admin \u003d (policy.get_enforcer().check_is_admin(self) or"},{"line_number":31,"context_line":"                    CONF.api_settings.auth_strategy \u003d\u003d constants.NOAUTH)"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"        super().__init__(is_admin\u003dis_admin, user_id\u003duser_id,"},{"line_number":34,"context_line":"                         project_id\u003dproject_id, **kwargs)"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"    @property"}],"source_content_type":"text/x-python","patch_set":4,"id":"b7f0f5e2_3baae1f5","line":33,"range":{"start_line":33,"start_character":44,"end_line":33,"end_character":60},"in_reply_to":"bf5ea08c_86abfa75","updated":"2021-12-12 23:39:08.000000000","message":"Done","commit_id":"60fef6eb03696c50a88924b5e6382c17627df509"},{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"43b3951496944c8923d0db205fec79f2c28f5c4e","unresolved":true,"context_lines":[{"line_number":26,"context_line":""},{"line_number":27,"context_line":"    _session \u003d None"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"    def __init__(self, user_id\u003dNone, project_id\u003dNone, **kwargs):"},{"line_number":30,"context_line":"        super().__init__(project_id\u003dproject_id, **kwargs)"},{"line_number":31,"context_line":"        self.is_admin \u003d (policy.get_enforcer().check_is_admin(self) or"},{"line_number":32,"context_line":"                         CONF.api_settings.auth_strategy \u003d\u003d constants.NOAUTH)"}],"source_content_type":"text/x-python","patch_set":7,"id":"583a369a_5804159c","line":29,"range":{"start_line":29,"start_character":37,"end_line":29,"end_character":54},"updated":"2022-01-18 08:51:35.000000000","message":"removing project_id parameter would also work, it would have been passed in kwargs","commit_id":"885ca42a0c4f052b7b8a9ec19a68c34317f3208e"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"9a2b469a7ceb6e2ad8a245fc05a683a1b15fbb64","unresolved":false,"context_lines":[{"line_number":26,"context_line":""},{"line_number":27,"context_line":"    _session \u003d None"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"    def __init__(self, user_id\u003dNone, project_id\u003dNone, **kwargs):"},{"line_number":30,"context_line":"        super().__init__(project_id\u003dproject_id, **kwargs)"},{"line_number":31,"context_line":"        self.is_admin \u003d (policy.get_enforcer().check_is_admin(self) or"},{"line_number":32,"context_line":"                         CONF.api_settings.auth_strategy \u003d\u003d constants.NOAUTH)"}],"source_content_type":"text/x-python","patch_set":7,"id":"e0fa1395_267081bf","line":29,"range":{"start_line":29,"start_character":37,"end_line":29,"end_character":54},"in_reply_to":"5700e3b9_9053efe2","updated":"2022-02-11 12:26:48.000000000","message":"Done","commit_id":"885ca42a0c4f052b7b8a9ec19a68c34317f3208e"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"05ee2f59f7e9231e09d505a76732778936521e57","unresolved":false,"context_lines":[{"line_number":26,"context_line":""},{"line_number":27,"context_line":"    _session \u003d None"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"    def __init__(self, user_id\u003dNone, project_id\u003dNone, **kwargs):"},{"line_number":30,"context_line":"        super().__init__(project_id\u003dproject_id, **kwargs)"},{"line_number":31,"context_line":"        self.is_admin \u003d (policy.get_enforcer().check_is_admin(self) or"},{"line_number":32,"context_line":"                         CONF.api_settings.auth_strategy \u003d\u003d constants.NOAUTH)"}],"source_content_type":"text/x-python","patch_set":7,"id":"5700e3b9_9053efe2","line":29,"range":{"start_line":29,"start_character":37,"end_line":29,"end_character":54},"in_reply_to":"583a369a_5804159c","updated":"2022-02-11 11:58:07.000000000","message":"There are a few places we pass project_id without the parameter key thus we need to keep this explicit definition. I\u0027ll submit a follow-up to remove this completely.","commit_id":"885ca42a0c4f052b7b8a9ec19a68c34317f3208e"}]}