)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"d8a436279c7967e57829a1a33d2d7c60d8edba09","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"9292f4c6_c77c5e67","updated":"2025-08-20 16:47:00.000000000","message":"Please see my comments on https://review.opendev.org/c/openstack/python-openstackclient/+/957586 and https://review.opendev.org/c/openstack/python-openstackclient/+/958062. Unfortunately we can\u0027t merge tests that modify things like services, since they can cause other tests running at the same time to fail.","commit_id":"27536920aef330c8ec9e4661492ff8a0030625b4"},{"author":{"_account_id":38265,"name":"seheon","display_name":"seheon","email":"ho78901@naver.com","username":"seheonnn"},"change_message_id":"3849d1f9a850ceb0ccd70046a64093ad05ad3794","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"471dc5ec_2ba0edca","updated":"2025-08-23 07:12:03.000000000","message":"recheck","commit_id":"32c11d854d02f638ec093479e6dfcfaf4ec8b657"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"3be6da7f1345d86b7fe5e28ab064d8b7b96e322b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"20643a55_9c22d4e7","updated":"2025-09-09 21:52:52.000000000","message":"k d","commit_id":"132fcee7cc2000f1905883b2d47f42fee144da48"}],"openstackclient/tests/functional/volume/v2/test_volume_host.py":[{"author":{"_account_id":35119,"name":"jihyun huh","email":"huhji.elha@gmail.com","username":"jhhuh"},"change_message_id":"91d03c31a191430f38c7044378f41b25f7e23e5f","unresolved":true,"context_lines":[{"line_number":26,"context_line":"    def test_volume_host(self):"},{"line_number":27,"context_line":"        \"\"\"One test function with sections: Help / List (read-only).\"\"\""},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"        # ----- Help -----"},{"line_number":30,"context_line":"        # Check that help output shows enable/disable options."},{"line_number":31,"context_line":"        out \u003d self.openstack(\u0027volume host set -h\u0027)"},{"line_number":32,"context_line":"        self.assertIn(\u0027--enable\u0027, out)"}],"source_content_type":"text/x-python","patch_set":5,"id":"8915e676_e57bff2d","line":29,"updated":"2025-08-30 07:06:08.000000000","message":"You can remove the upper comment because it has the same meaning as the underline. And it also needs to follow the comment style of other functional tests.","commit_id":"e6bbe644d8ab748fd7229ae51de3181822a8ae1a"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"3be6da7f1345d86b7fe5e28ab064d8b7b96e322b","unresolved":true,"context_lines":[{"line_number":25,"context_line":"    def test_volume_host(self):"},{"line_number":26,"context_line":"        \"\"\"One test function with sections: Help / List (read-only).\"\"\""},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"        out \u003d self.openstack(\u0027volume host set -h\u0027)"},{"line_number":29,"context_line":"        self.assertIn(\u0027--enable\u0027, out)"},{"line_number":30,"context_line":"        self.assertIn(\u0027--disable\u0027, out)"},{"line_number":31,"context_line":"        self.assertIn(\u0027\u003chost-name\u003e\u0027, out)"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"        if not self.cinder_hosts:"},{"line_number":34,"context_line":"            self.skipTest(\u0027No cinder-volume service found\u0027)"},{"line_number":35,"context_line":""}],"source_content_type":"text/x-python","patch_set":7,"id":"9d5381d7_5f5447d4","line":32,"range":{"start_line":28,"start_character":0,"end_line":32,"end_character":1},"updated":"2025-09-09 21:52:52.000000000","message":"I appreciate what you\u0027re trying to do, but I think you should abandon this patch and move onto something else. What you\u0027re testing here is common functionality that is not specific to volume hosts. We don\u0027t test those for every command because it\u0027s unnecessary.","commit_id":"132fcee7cc2000f1905883b2d47f42fee144da48"}]}