)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":15519,"name":"Iury Gregory Melo Ferreira","display_name":"Iury Gregory","email":"iurygregory@gmail.com","username":"iurygregory"},"change_message_id":"6db58275ced7ce0a8d941230767054ec51a24d22","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"9773c203_e20ac078","updated":"2022-06-07 11:56:34.000000000","message":"unit tests are a bit unhappy \u003d(, I will try to reproduce locally","commit_id":"05d5151080d3cef0d8f2d9694b1fd38d2c6077ce"},{"author":{"_account_id":15519,"name":"Iury Gregory Melo Ferreira","display_name":"Iury Gregory","email":"iurygregory@gmail.com","username":"iurygregory"},"change_message_id":"946630af64c1d52acfa2f642ac8d84e862701a0b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"e130a26a_32745d93","updated":"2022-06-07 13:15:39.000000000","message":"makes sense to me, after looking at old backports","commit_id":"8abb8ccfafed651b17ed7a1a202eb2655ac16431"},{"author":{"_account_id":23851,"name":"Riccardo Pittau","email":"elfosardo@gmail.com","username":"elfosardo"},"change_message_id":"ad035e68cdd736394b1b1856ec5e3c49922d53d5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"94f06286_ccef96cb","updated":"2022-06-07 17:40:13.000000000","message":"recheck","commit_id":"8abb8ccfafed651b17ed7a1a202eb2655ac16431"},{"author":{"_account_id":23851,"name":"Riccardo Pittau","email":"elfosardo@gmail.com","username":"elfosardo"},"change_message_id":"3579b376b997ee5307475078a8fc7e04cc6fc68c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"f22e3e4e_9ea41075","updated":"2022-06-07 15:43:03.000000000","message":"recheck","commit_id":"8abb8ccfafed651b17ed7a1a202eb2655ac16431"}],"ironic_python_agent/tests/unit/test_utils.py":[{"author":{"_account_id":15519,"name":"Iury Gregory Melo Ferreira","display_name":"Iury Gregory","email":"iurygregory@gmail.com","username":"iurygregory"},"change_message_id":"01b3065c1b0b94cf7ee97de4162eabff34ca019e","unresolved":true,"context_lines":[{"line_number":448,"context_line":"    @mock.patch.object(utils, \u0027is_journalctl_present\u0027, autospec\u003dTrue)"},{"line_number":449,"context_line":"    @mock.patch.object(utils, \u0027get_command_output\u0027, autospec\u003dTrue)"},{"line_number":450,"context_line":"    @mock.patch.object(utils, \u0027get_journalctl_output\u0027, autospec\u003dTrue)"},{"line_number":451,"context_line":"    def test_collect_system_logs_journald_with_logfile("},{"line_number":452,"context_line":"            self, mock_logs, mock_outputs, mock_journalctl, mock_gzip_b64):"},{"line_number":453,"context_line":"        tmp \u003d tempfile.NamedTemporaryFile()"},{"line_number":454,"context_line":"        self.addCleanup(lambda: tmp.close())"}],"source_content_type":"text/x-python","patch_set":3,"id":"6d4af454_e6d621ab","line":451,"range":{"start_line":451,"start_character":8,"end_line":451,"end_character":54},"updated":"2022-06-07 12:14:57.000000000","message":"testtools.TestCase doesn\u0027t have self.config so I\u0027m wondering how we should handle","commit_id":"05d5151080d3cef0d8f2d9694b1fd38d2c6077ce"},{"author":{"_account_id":23851,"name":"Riccardo Pittau","email":"elfosardo@gmail.com","username":"elfosardo"},"change_message_id":"ad035e68cdd736394b1b1856ec5e3c49922d53d5","unresolved":false,"context_lines":[{"line_number":448,"context_line":"    @mock.patch.object(utils, \u0027is_journalctl_present\u0027, autospec\u003dTrue)"},{"line_number":449,"context_line":"    @mock.patch.object(utils, \u0027get_command_output\u0027, autospec\u003dTrue)"},{"line_number":450,"context_line":"    @mock.patch.object(utils, \u0027get_journalctl_output\u0027, autospec\u003dTrue)"},{"line_number":451,"context_line":"    def test_collect_system_logs_journald_with_logfile("},{"line_number":452,"context_line":"            self, mock_logs, mock_outputs, mock_journalctl, mock_gzip_b64):"},{"line_number":453,"context_line":"        tmp \u003d tempfile.NamedTemporaryFile()"},{"line_number":454,"context_line":"        self.addCleanup(lambda: tmp.close())"}],"source_content_type":"text/x-python","patch_set":3,"id":"483ad41d_1a8d652a","line":451,"range":{"start_line":451,"start_character":8,"end_line":451,"end_character":54},"in_reply_to":"51cfbd0d_424f9f43","updated":"2022-06-07 17:40:13.000000000","message":"Done","commit_id":"05d5151080d3cef0d8f2d9694b1fd38d2c6077ce"},{"author":{"_account_id":15519,"name":"Iury Gregory Melo Ferreira","display_name":"Iury Gregory","email":"iurygregory@gmail.com","username":"iurygregory"},"change_message_id":"c5101d350b5f68e9029b54576248fdf04f14056f","unresolved":true,"context_lines":[{"line_number":448,"context_line":"    @mock.patch.object(utils, \u0027is_journalctl_present\u0027, autospec\u003dTrue)"},{"line_number":449,"context_line":"    @mock.patch.object(utils, \u0027get_command_output\u0027, autospec\u003dTrue)"},{"line_number":450,"context_line":"    @mock.patch.object(utils, \u0027get_journalctl_output\u0027, autospec\u003dTrue)"},{"line_number":451,"context_line":"    def test_collect_system_logs_journald_with_logfile("},{"line_number":452,"context_line":"            self, mock_logs, mock_outputs, mock_journalctl, mock_gzip_b64):"},{"line_number":453,"context_line":"        tmp \u003d tempfile.NamedTemporaryFile()"},{"line_number":454,"context_line":"        self.addCleanup(lambda: tmp.close())"}],"source_content_type":"text/x-python","patch_set":3,"id":"de9ede6d_58c854eb","line":451,"range":{"start_line":451,"start_character":8,"end_line":451,"end_character":54},"in_reply_to":"6d4af454_e6d621ab","updated":"2022-06-07 12:30:24.000000000","message":"Maybe we can try switching to \n\nfrom oslo_config import cfg\n\nCONF \u003d cfg.CONF","commit_id":"05d5151080d3cef0d8f2d9694b1fd38d2c6077ce"},{"author":{"_account_id":23851,"name":"Riccardo Pittau","email":"elfosardo@gmail.com","username":"elfosardo"},"change_message_id":"8863414d2ad373a542409611a0a318bcb622deb7","unresolved":true,"context_lines":[{"line_number":448,"context_line":"    @mock.patch.object(utils, \u0027is_journalctl_present\u0027, autospec\u003dTrue)"},{"line_number":449,"context_line":"    @mock.patch.object(utils, \u0027get_command_output\u0027, autospec\u003dTrue)"},{"line_number":450,"context_line":"    @mock.patch.object(utils, \u0027get_journalctl_output\u0027, autospec\u003dTrue)"},{"line_number":451,"context_line":"    def test_collect_system_logs_journald_with_logfile("},{"line_number":452,"context_line":"            self, mock_logs, mock_outputs, mock_journalctl, mock_gzip_b64):"},{"line_number":453,"context_line":"        tmp \u003d tempfile.NamedTemporaryFile()"},{"line_number":454,"context_line":"        self.addCleanup(lambda: tmp.close())"}],"source_content_type":"text/x-python","patch_set":3,"id":"51cfbd0d_424f9f43","line":451,"range":{"start_line":451,"start_character":8,"end_line":451,"end_character":54},"in_reply_to":"de9ede6d_58c854eb","updated":"2022-06-07 13:06:21.000000000","message":"I think we should not add those 2 tests\nI\u0027ll remove them","commit_id":"05d5151080d3cef0d8f2d9694b1fd38d2c6077ce"}]}