)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":28522,"name":"Hervé Beraud","email":"herveberaud.pro@gmail.com","username":"hberaud"},"change_message_id":"ccecefb3e3ba7e1a0774473035307c0b5fa685e6","unresolved":true,"context_lines":[{"line_number":1,"context_line":"Parent:     88ec0a44 (Bump pyupgrade target to 3.10+)"},{"line_number":2,"context_line":"Author:     damani42 \u003c56308939+damani42@users.noreply.github.com\u003e"},{"line_number":3,"context_line":"AuthorDate: 2025-11-03 15:55:24 +0100"},{"line_number":4,"context_line":"Commit:     Daniel Bengtsson \u003cdbengt@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2025-11-03 15:59:05 +0100"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"c079940e_d3ac5213","line":2,"range":{"start_line":2,"start_character":22,"end_line":2,"end_character":64},"updated":"2025-11-03 15:26:07.000000000","message":"I think you should have your real email address.","commit_id":"ef57d038fdd08c09b4e04556b3125535df53db19"},{"author":{"_account_id":31245,"name":"Daniel Bengtsson","email":"dbengt@redhat.com","username":"damani42"},"change_message_id":"d7dc3a661ac03e3e2b62c319690be01081ea9bba","unresolved":false,"context_lines":[{"line_number":1,"context_line":"Parent:     88ec0a44 (Bump pyupgrade target to 3.10+)"},{"line_number":2,"context_line":"Author:     damani42 \u003c56308939+damani42@users.noreply.github.com\u003e"},{"line_number":3,"context_line":"AuthorDate: 2025-11-03 15:55:24 +0100"},{"line_number":4,"context_line":"Commit:     Daniel Bengtsson \u003cdbengt@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2025-11-03 15:59:05 +0100"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"7f2fed77_3fa12c29","line":2,"range":{"start_line":2,"start_character":22,"end_line":2,"end_character":64},"in_reply_to":"c079940e_d3ac5213","updated":"2025-11-03 22:09:34.000000000","message":"Done","commit_id":"ef57d038fdd08c09b4e04556b3125535df53db19"},{"author":{"_account_id":28522,"name":"Hervé Beraud","email":"herveberaud.pro@gmail.com","username":"hberaud"},"change_message_id":"ccecefb3e3ba7e1a0774473035307c0b5fa685e6","unresolved":true,"context_lines":[{"line_number":9,"context_line":"The previous conditional:"},{"line_number":10,"context_line":"    proc_id \u003d self.pg if self.pg !\u003d 0 else os.getpid()"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"was added to handle the crun container runtime edge case where"},{"line_number":13,"context_line":"os.getpgrp() may return 0. However, using `or` here provides the"},{"line_number":14,"context_line":"same behavior and correctly handles any falsy value (e.g. None or 0),"},{"line_number":15,"context_line":"while being shorter and more idiomatic Python."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"8c78b7a4_d39c6bc7","line":12,"range":{"start_line":12,"start_character":24,"end_line":12,"end_character":28},"updated":"2025-11-03 15:26:07.000000000","message":"Not sure if this is a typo.","commit_id":"ef57d038fdd08c09b4e04556b3125535df53db19"},{"author":{"_account_id":31245,"name":"Daniel Bengtsson","email":"dbengt@redhat.com","username":"damani42"},"change_message_id":"d84227471d953e3c9bd52ae2f30b522a3dab482e","unresolved":false,"context_lines":[{"line_number":9,"context_line":"The previous conditional:"},{"line_number":10,"context_line":"    proc_id \u003d self.pg if self.pg !\u003d 0 else os.getpid()"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"was added to handle the crun container runtime edge case where"},{"line_number":13,"context_line":"os.getpgrp() may return 0. However, using `or` here provides the"},{"line_number":14,"context_line":"same behavior and correctly handles any falsy value (e.g. None or 0),"},{"line_number":15,"context_line":"while being shorter and more idiomatic Python."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"647f7c8d_375c985d","line":12,"range":{"start_line":12,"start_character":24,"end_line":12,"end_character":28},"in_reply_to":"2d54e86b_d95bfe6b","updated":"2025-11-03 22:11:23.000000000","message":"Done","commit_id":"ef57d038fdd08c09b4e04556b3125535df53db19"},{"author":{"_account_id":31245,"name":"Daniel Bengtsson","email":"dbengt@redhat.com","username":"damani42"},"change_message_id":"8691ae0b69478ebc1b8ccca5baf6641d54ed0d17","unresolved":true,"context_lines":[{"line_number":9,"context_line":"The previous conditional:"},{"line_number":10,"context_line":"    proc_id \u003d self.pg if self.pg !\u003d 0 else os.getpid()"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"was added to handle the crun container runtime edge case where"},{"line_number":13,"context_line":"os.getpgrp() may return 0. However, using `or` here provides the"},{"line_number":14,"context_line":"same behavior and correctly handles any falsy value (e.g. None or 0),"},{"line_number":15,"context_line":"while being shorter and more idiomatic Python."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"2d54e86b_d95bfe6b","line":12,"range":{"start_line":12,"start_character":24,"end_line":12,"end_character":28},"in_reply_to":"8c78b7a4_d39c6bc7","updated":"2025-11-03 22:11:00.000000000","message":"Not it\u0027s not a typo.","commit_id":"ef57d038fdd08c09b4e04556b3125535df53db19"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":28522,"name":"Hervé Beraud","email":"herveberaud.pro@gmail.com","username":"hberaud"},"change_message_id":"ccecefb3e3ba7e1a0774473035307c0b5fa685e6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"c2413b08_c5be6452","updated":"2025-11-03 15:26:07.000000000","message":"LGTM. This change is, I think, low-risk and would allow a more broader falsy values detections.\n\nI just have a couple of comments for the commit message.","commit_id":"ef57d038fdd08c09b4e04556b3125535df53db19"}],"oslo_messaging/_drivers/amqpdriver.py":[{"author":{"_account_id":28522,"name":"Hervé Beraud","email":"herveberaud.pro@gmail.com","username":"hberaud"},"change_message_id":"ccecefb3e3ba7e1a0774473035307c0b5fa685e6","unresolved":true,"context_lines":[{"line_number":75,"context_line":"        # does not exist. This is only hit in an edge case with the crun"},{"line_number":76,"context_line":"        # container runtime (default for podman), see issue for more details:"},{"line_number":77,"context_line":"        # https://github.com/containers/crun/issues/1642"},{"line_number":78,"context_line":"        proc_id \u003d self.pg or os.getpid()"},{"line_number":79,"context_line":"        with open(f\u0027/proc/{proc_id}/stat\u0027) as f:"},{"line_number":80,"context_line":"            self.start_time \u003d int(f.read().split()[21])"},{"line_number":81,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"f81e5cab_56c2667d","line":78,"range":{"start_line":78,"start_character":8,"end_line":78,"end_character":40},"updated":"2025-11-03 15:26:07.000000000","message":"The change is low-risk and include more falsy values detections.","commit_id":"ef57d038fdd08c09b4e04556b3125535df53db19"},{"author":{"_account_id":31245,"name":"Daniel Bengtsson","email":"dbengt@redhat.com","username":"damani42"},"change_message_id":"4798a1d06118e4fe0c83c5e495f4e4ef22b8200e","unresolved":false,"context_lines":[{"line_number":75,"context_line":"        # does not exist. This is only hit in an edge case with the crun"},{"line_number":76,"context_line":"        # container runtime (default for podman), see issue for more details:"},{"line_number":77,"context_line":"        # https://github.com/containers/crun/issues/1642"},{"line_number":78,"context_line":"        proc_id \u003d self.pg or os.getpid()"},{"line_number":79,"context_line":"        with open(f\u0027/proc/{proc_id}/stat\u0027) as f:"},{"line_number":80,"context_line":"            self.start_time \u003d int(f.read().split()[21])"},{"line_number":81,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"22b1518f_4b93d06b","line":78,"range":{"start_line":78,"start_character":8,"end_line":78,"end_character":40},"in_reply_to":"1216b117_02fefcdd","updated":"2025-11-03 22:11:16.000000000","message":"Done","commit_id":"ef57d038fdd08c09b4e04556b3125535df53db19"},{"author":{"_account_id":31245,"name":"Daniel Bengtsson","email":"dbengt@redhat.com","username":"damani42"},"change_message_id":"19dc5b31fb1e00d9e2653baf200a18337236a567","unresolved":true,"context_lines":[{"line_number":75,"context_line":"        # does not exist. This is only hit in an edge case with the crun"},{"line_number":76,"context_line":"        # container runtime (default for podman), see issue for more details:"},{"line_number":77,"context_line":"        # https://github.com/containers/crun/issues/1642"},{"line_number":78,"context_line":"        proc_id \u003d self.pg or os.getpid()"},{"line_number":79,"context_line":"        with open(f\u0027/proc/{proc_id}/stat\u0027) as f:"},{"line_number":80,"context_line":"            self.start_time \u003d int(f.read().split()[21])"},{"line_number":81,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"1216b117_02fefcdd","line":78,"range":{"start_line":78,"start_character":8,"end_line":78,"end_character":40},"in_reply_to":"f81e5cab_56c2667d","updated":"2025-11-03 22:09:13.000000000","message":"Yes exactly it\u0027s a better condition.","commit_id":"ef57d038fdd08c09b4e04556b3125535df53db19"}]}