)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":22623,"name":"Erik Olof Gunnar Andersson","email":"eandersson@blizzard.com","username":"eoandersson"},"change_message_id":"49079c8ef68ffaf7207b783056e5004f0ac4e481","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"61753ef1_e53e4286","updated":"2026-03-16 08:52:20.000000000","message":"Looks good, but could you add some sort of regression test?","commit_id":"834728e898567e339187893bd0f122cf754ad45e"},{"author":{"_account_id":26319,"name":"Michel Raabe","display_name":"Michel Raabe (rmichel)","email":"rmichel@devnu11.net","username":"rmichel","status":"openstack 💚\nceph 💚\n\nworking for B1 Systems GmbH"},"change_message_id":"15b32fb9a6ee67e594f9412fe4f9c90643c8a2d0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"97ad6ce0_aa5f83d6","in_reply_to":"61753ef1_e53e4286","updated":"2026-03-17 21:58:01.000000000","message":"This is pretty much my first regression test — please bear with me","commit_id":"834728e898567e339187893bd0f122cf754ad45e"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"1f5d3e37c39442806f29b6dc76359d2d92fbbd56","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"f840961b_ab67b8a4","updated":"2026-03-31 08:57:43.000000000","message":"the new test fails without the fix applied, so I\u0027m fine with this fix as it stands, no need to hold up fixing a bug only due to testing shenanigans if no core is willing to work on that","commit_id":"0ab1d2126db3825a99935402f877d3f286da6657"},{"author":{"_account_id":31664,"name":"Omer Schwartz","email":"oschwart@redhat.com","username":"oschwart"},"change_message_id":"6c35f120f35a4d5e2e8e28ba837a9098b2c5e21f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"35af9869_83da27a8","updated":"2026-04-17 12:54:04.000000000","message":"LGTM","commit_id":"b6546cb1b47131d8b1d7e12150f8861f948b58c8"},{"author":{"_account_id":31664,"name":"Omer Schwartz","email":"oschwart@redhat.com","username":"oschwart"},"change_message_id":"fec7d47225765063fa8e916a5b5157ca2efe0f79","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"8c9abb85_6291ed45","updated":"2026-04-17 15:23:12.000000000","message":"recheck failure is not related to the patch","commit_id":"b6546cb1b47131d8b1d7e12150f8861f948b58c8"}],"designate/tests/functional/api/v2/test_service_status.py":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"1f5d3e37c39442806f29b6dc76359d2d92fbbd56","unresolved":true,"context_lines":[{"line_number":47,"context_line":"        data \u003d [self.update_service_status("},{"line_number":48,"context_line":"            hostname\u003d\"foo%s\" % i, service_name\u003d\"bar\") for i in range(0, 10)]"},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"        self._assert_paging(data, \u0027/service_statuses\u0027, key\u003d\u0027service_statuses\u0027)"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"    def test_legacy_list_service_status(self):"},{"line_number":53,"context_line":"        \"\"\"Test the legacy list service status path."}],"source_content_type":"text/x-python","patch_set":3,"id":"ac1ce3cf_1f01b96d","line":50,"updated":"2026-03-31 08:57:43.000000000","message":"if someone wants to invest a bit more time, it would be interesting to find out why this test didn\u0027t find the issue. one obvious thing is it either needs more services or decreasing the limit as done in the new test, but doing that still didn\u0027t trigger a failure for me","commit_id":"0ab1d2126db3825a99935402f877d3f286da6657"},{"author":{"_account_id":26319,"name":"Michel Raabe","display_name":"Michel Raabe (rmichel)","email":"rmichel@devnu11.net","username":"rmichel","status":"openstack 💚\nceph 💚\n\nworking for B1 Systems GmbH"},"change_message_id":"131bbce1b2a4a79a06302ded30daf2acab74d1f7","unresolved":false,"context_lines":[{"line_number":47,"context_line":"        data \u003d [self.update_service_status("},{"line_number":48,"context_line":"            hostname\u003d\"foo%s\" % i, service_name\u003d\"bar\") for i in range(0, 10)]"},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"        self._assert_paging(data, \u0027/service_statuses\u0027, key\u003d\u0027service_statuses\u0027)"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"    def test_legacy_list_service_status(self):"},{"line_number":53,"context_line":"        \"\"\"Test the legacy list service status path."}],"source_content_type":"text/x-python","patch_set":3,"id":"b972b7af_5ac78cf4","line":50,"in_reply_to":"ac1ce3cf_1f01b96d","updated":"2026-04-13 12:28:13.000000000","message":"Acknowledged","commit_id":"0ab1d2126db3825a99935402f877d3f286da6657"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"1f5d3e37c39442806f29b6dc76359d2d92fbbd56","unresolved":true,"context_lines":[{"line_number":122,"context_line":"        self.policy({\u0027find_service_status\u0027: \u0027@\u0027})"},{"line_number":123,"context_line":"        self._assert_invalid_uuid(self.client.get, \u0027/service_statuses/%s\u0027)"},{"line_number":124,"context_line":""},{"line_number":125,"context_line":"    def test_get_service_statuses_pagination_default_limit(self):"},{"line_number":126,"context_line":"        self.policy({\u0027find_service_statuses\u0027: \u0027@\u0027})"},{"line_number":127,"context_line":""},{"line_number":128,"context_line":"        # Set default_limit_v2 to a small value and create exactly that"}],"source_content_type":"text/x-python","patch_set":3,"id":"4d410059_cced96a1","line":125,"updated":"2026-03-31 08:57:43.000000000","message":"in case you need to respin, please move this to line 51 directly after test_get_service_statuses(), so that the two related tests are next to each other in this file, but I wouldn\u0027t consider this a blocker","commit_id":"0ab1d2126db3825a99935402f877d3f286da6657"},{"author":{"_account_id":26319,"name":"Michel Raabe","display_name":"Michel Raabe (rmichel)","email":"rmichel@devnu11.net","username":"rmichel","status":"openstack 💚\nceph 💚\n\nworking for B1 Systems GmbH"},"change_message_id":"131bbce1b2a4a79a06302ded30daf2acab74d1f7","unresolved":false,"context_lines":[{"line_number":122,"context_line":"        self.policy({\u0027find_service_status\u0027: \u0027@\u0027})"},{"line_number":123,"context_line":"        self._assert_invalid_uuid(self.client.get, \u0027/service_statuses/%s\u0027)"},{"line_number":124,"context_line":""},{"line_number":125,"context_line":"    def test_get_service_statuses_pagination_default_limit(self):"},{"line_number":126,"context_line":"        self.policy({\u0027find_service_statuses\u0027: \u0027@\u0027})"},{"line_number":127,"context_line":""},{"line_number":128,"context_line":"        # Set default_limit_v2 to a small value and create exactly that"}],"source_content_type":"text/x-python","patch_set":3,"id":"34443bd0_3ee71106","line":125,"in_reply_to":"4d410059_cced96a1","updated":"2026-04-13 12:28:13.000000000","message":"I moved it over there.","commit_id":"0ab1d2126db3825a99935402f877d3f286da6657"}]}