)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":23084,"name":"Bartosz Bezak","email":"bartosz@stackhpc.com","username":"b.bezak"},"change_message_id":"e96268bfa030cb59e205daf0ed7825fefdccca30","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"6cbea8fe_9f79a2df","updated":"2025-06-18 09:23:57.000000000","message":"looks good - just nit picking","commit_id":"4b6fc8813efab52b13a856c49f7b873407144db4"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"d370adc355243bfcc4d59bf4f3a0cb04283715d6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"cc329265_52c3a58f","updated":"2025-06-19 15:06:35.000000000","message":"recheck non-related","commit_id":"cf3c65fef7c51b8f54c32bfa58574f58ad4071c8"}],"docker/base/set_configs.py":[{"author":{"_account_id":23084,"name":"Bartosz Bezak","email":"bartosz@stackhpc.com","username":"b.bezak"},"change_message_id":"e96268bfa030cb59e205daf0ed7825fefdccca30","unresolved":true,"context_lines":[{"line_number":631,"context_line":"    if removed_dests:"},{"line_number":632,"context_line":"        raise StateMismatch("},{"line_number":633,"context_line":"            f\"The following config files are tracked in state but missing \""},{"line_number":634,"context_line":"            f\"from config.json.\""},{"line_number":635,"context_line":"            f\"They would be restored or removed: {sorted(removed_dests)}\""},{"line_number":636,"context_line":"        )"},{"line_number":637,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"9805630a_bae5ea88","line":634,"updated":"2025-06-18 09:23:57.000000000","message":"```suggestion\n            f\"from config.json. \"\n```","commit_id":"4b6fc8813efab52b13a856c49f7b873407144db4"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"0cd218b372089f35cc91b8cdf257e23b8d2ddd5d","unresolved":false,"context_lines":[{"line_number":631,"context_line":"    if removed_dests:"},{"line_number":632,"context_line":"        raise StateMismatch("},{"line_number":633,"context_line":"            f\"The following config files are tracked in state but missing \""},{"line_number":634,"context_line":"            f\"from config.json.\""},{"line_number":635,"context_line":"            f\"They would be restored or removed: {sorted(removed_dests)}\""},{"line_number":636,"context_line":"        )"},{"line_number":637,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"26bfaa3e_b05504d1","line":634,"in_reply_to":"9805630a_bae5ea88","updated":"2025-06-19 12:24:22.000000000","message":"Done","commit_id":"4b6fc8813efab52b13a856c49f7b873407144db4"}],"tests/test_set_config.py":[{"author":{"_account_id":23084,"name":"Bartosz Bezak","email":"bartosz@stackhpc.com","username":"b.bezak"},"change_message_id":"e96268bfa030cb59e205daf0ed7825fefdccca30","unresolved":true,"context_lines":[{"line_number":896,"context_line":""},{"line_number":897,"context_line":"class ExecuteConfigCheckStateMismatchTest(base.BaseTestCase):"},{"line_number":898,"context_line":""},{"line_number":899,"context_line":"    @mock.patch(\u0027builtins.open\u0027, new_callable\u003dmock.mock_open)"},{"line_number":900,"context_line":"    @mock.patch.object(set_configs, \u0027get_defaults_state\u0027)"},{"line_number":901,"context_line":"    def test_execute_config_check_raises_state_mismatch("},{"line_number":902,"context_line":"        self, mock_get_defaults_state, mock_open"}],"source_content_type":"text/x-python","patch_set":2,"id":"219189bb_e35525d6","line":899,"updated":"2025-06-18 09:23:57.000000000","message":"I don\u0027t think that mock_open is needed here as tests just comparing states - not opening any files","commit_id":"4b6fc8813efab52b13a856c49f7b873407144db4"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"0cd218b372089f35cc91b8cdf257e23b8d2ddd5d","unresolved":false,"context_lines":[{"line_number":896,"context_line":""},{"line_number":897,"context_line":"class ExecuteConfigCheckStateMismatchTest(base.BaseTestCase):"},{"line_number":898,"context_line":""},{"line_number":899,"context_line":"    @mock.patch(\u0027builtins.open\u0027, new_callable\u003dmock.mock_open)"},{"line_number":900,"context_line":"    @mock.patch.object(set_configs, \u0027get_defaults_state\u0027)"},{"line_number":901,"context_line":"    def test_execute_config_check_raises_state_mismatch("},{"line_number":902,"context_line":"        self, mock_get_defaults_state, mock_open"}],"source_content_type":"text/x-python","patch_set":2,"id":"314db56f_8c28873b","line":899,"in_reply_to":"219189bb_e35525d6","updated":"2025-06-19 12:24:22.000000000","message":"Done","commit_id":"4b6fc8813efab52b13a856c49f7b873407144db4"},{"author":{"_account_id":23084,"name":"Bartosz Bezak","email":"bartosz@stackhpc.com","username":"b.bezak"},"change_message_id":"e96268bfa030cb59e205daf0ed7825fefdccca30","unresolved":true,"context_lines":[{"line_number":899,"context_line":"    @mock.patch(\u0027builtins.open\u0027, new_callable\u003dmock.mock_open)"},{"line_number":900,"context_line":"    @mock.patch.object(set_configs, \u0027get_defaults_state\u0027)"},{"line_number":901,"context_line":"    def test_execute_config_check_raises_state_mismatch("},{"line_number":902,"context_line":"        self, mock_get_defaults_state, mock_open"},{"line_number":903,"context_line":"    ):"},{"line_number":904,"context_line":"        \"\"\"Test execute_config_check() when state has extra config file."},{"line_number":905,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"deb3d78f_1752f6ee","line":902,"updated":"2025-06-18 09:23:57.000000000","message":"ditto","commit_id":"4b6fc8813efab52b13a856c49f7b873407144db4"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"0cd218b372089f35cc91b8cdf257e23b8d2ddd5d","unresolved":false,"context_lines":[{"line_number":899,"context_line":"    @mock.patch(\u0027builtins.open\u0027, new_callable\u003dmock.mock_open)"},{"line_number":900,"context_line":"    @mock.patch.object(set_configs, \u0027get_defaults_state\u0027)"},{"line_number":901,"context_line":"    def test_execute_config_check_raises_state_mismatch("},{"line_number":902,"context_line":"        self, mock_get_defaults_state, mock_open"},{"line_number":903,"context_line":"    ):"},{"line_number":904,"context_line":"        \"\"\"Test execute_config_check() when state has extra config file."},{"line_number":905,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"ddd80442_f00cfe1d","line":902,"in_reply_to":"deb3d78f_1752f6ee","updated":"2025-06-19 12:24:22.000000000","message":"Done","commit_id":"4b6fc8813efab52b13a856c49f7b873407144db4"}]}