)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":32968,"name":"Juan Larriba","email":"jlarriba@redhat.com","username":"jlarriba"},"change_message_id":"315c254e3dfdf90f5a59da40029c82c2085c266a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"9d245532_1c377bf4","updated":"2026-03-11 15:02:54.000000000","message":"Hi, thanks for your contribution.\n\nWe usually do not start reviews until the CI is passing and Zuul gives a +1. Do you intend to fix the tox errors and are still interested in getting this contributions reviewed and merged?","commit_id":"e04c0ae8bcff04e866804be3e49ffad8fb271455"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"a1e1745893ae0eacff191274b3ea49f61f2bccd7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"a1757f4f_5e0a4a7f","updated":"2026-05-07 13:25:08.000000000","message":"I agree with the comments left by Jaromír . I\u0027m also adding one inline comment.","commit_id":"3aa89b9b5f0ac16d7c0badf4221d2bf50622b513"},{"author":{"_account_id":34975,"name":"Jaromír Wysoglad","email":"jwysogla@redhat.com","username":"jwysogla"},"change_message_id":"a3f5f886662da9c3854c3df5f6d2d0df1f88e022","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"e854cd27_6e8ad83e","updated":"2026-05-04 14:45:43.000000000","message":"I think this is a good idea in general. But I have 3 things:\n- The hyphen sanitation is an unrelated fix. Ideally it\u0027d have it\u0027s own commit. But I see it\u0027d be a 1 line + a testcase, I\u0027m personally fine for you to just leave it here. But we should at least mention it in the commit message.\n- I think there should be a new testcase for the case when _has_grouping \u003d True\n- We should have a releasenote","commit_id":"3aa89b9b5f0ac16d7c0badf4221d2bf50622b513"}],"ceilometer/publisher/http.py":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"a1e1745893ae0eacff191274b3ea49f61f2bccd7","unresolved":true,"context_lines":[{"line_number":167,"context_line":"            return"},{"line_number":168,"context_line":"        self._do_post(json.dumps(data))"},{"line_number":169,"context_line":""},{"line_number":170,"context_line":"    def _do_post(self, data, target\u003dNone):"},{"line_number":171,"context_line":"        target \u003d target or self.target"},{"line_number":172,"context_line":"        LOG.trace(\u0027Message: %s\u0027, data)"},{"line_number":173,"context_line":"        try:"}],"source_content_type":"text/x-python","patch_set":3,"id":"8adea0fd_cfbe32ec","line":170,"range":{"start_line":170,"start_character":29,"end_line":170,"end_character":35},"updated":"2026-05-07 13:25:08.000000000","message":"Instead of replacing the whole path, I\u0027d suggest something like sub_path which is appended to the target.","commit_id":"3aa89b9b5f0ac16d7c0badf4221d2bf50622b513"}]}