)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":34975,"name":"Jaromír Wysoglad","email":"jwysogla@redhat.com","username":"jwysogla"},"change_message_id":"4828efefc5f3d6c2928d93c6569288999e112a11","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"cc7b631b_e8350dc8","updated":"2025-04-03 15:32:21.000000000","message":"So the dsvm jobs fail likely because this needs to merge first https://review.opendev.org/c/openstack/telemetry-tempest-plugin/+/945108 . I\u0027m trying to figure out why the pep8 is failing here, while it\u0027s passing for me locally.","commit_id":"0f94a13e6e1e87374101e75a574fd4f482e684c5"},{"author":{"_account_id":34975,"name":"Jaromír Wysoglad","email":"jwysogla@redhat.com","username":"jwysogla"},"change_message_id":"46d6a1ea06ab3fcf85ea5cf664d444f844819991","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"1a56e8fc_4d2b4786","in_reply_to":"cc7b631b_e8350dc8","updated":"2025-04-03 16:04:04.000000000","message":"Seems like update to hacking could help. I\u0027ll try that later. https://github.com/microsoft/vscode-flake8/issues/233","commit_id":"0f94a13e6e1e87374101e75a574fd4f482e684c5"},{"author":{"_account_id":34975,"name":"Jaromír Wysoglad","email":"jwysogla@redhat.com","username":"jwysogla"},"change_message_id":"28858c9f956801d01fa4ea435be7751f980a23d9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"b4527e60_a62d4e30","updated":"2025-04-09 10:21:49.000000000","message":"Hmm. Looking at this, zuul actually didn\u0027t run the observabilityclient functional tests. I think I know why. I\u0027ll propose a change to hopefuly fix it https://opendev.org/openstack/python-observabilityclient/src/branch/master/.zuul.yaml#L4","commit_id":"383672f967ea38ab6501b61d6615a5d0fc5a7823"},{"author":{"_account_id":32968,"name":"Juan Larriba","email":"jlarriba@redhat.com","username":"jlarriba"},"change_message_id":"97298ae8a590b290074bc1f778cb83a588709c0e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"daed456d_77ea56eb","updated":"2025-04-07 09:00:09.000000000","message":"Very nice set of tests, thank you!","commit_id":"383672f967ea38ab6501b61d6615a5d0fc5a7823"},{"author":{"_account_id":34975,"name":"Jaromír Wysoglad","email":"jwysogla@redhat.com","username":"jwysogla"},"change_message_id":"7a50dd3f75d2d3ea30fc1c12d70b8d9e9c796fc6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"3ef55b37_0ff72afd","updated":"2025-04-11 12:48:12.000000000","message":"I rebased this on top of current master. The functional job runs, so imo this is good for reviews again.","commit_id":"eab78d03a4805462a81f562ef4d2f2146174e1b9"},{"author":{"_account_id":34975,"name":"Jaromír Wysoglad","email":"jwysogla@redhat.com","username":"jwysogla"},"change_message_id":"79b3e336002d173dd0548cb3bee470b3a0b5cbb7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"557e6132_bf753c91","updated":"2025-04-22 09:06:47.000000000","message":"@jlarriba@redhat.com this needs +2 once again, because there was a change.","commit_id":"d69cb08a77243dad1c205a524c4bfacbe0866d54"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"5c07a6393d1539a9dc55469eacea544f776d58b0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"408d5933_28ed98b3","updated":"2025-04-14 13:25:28.000000000","message":"Made a minor modification to reduce interface change as well as direct access by indexes (to make the parsing more robust for illegal input). I\u0027ve double checked the unit tests pass but please feel free to revert the change in case I made something wrong ...","commit_id":"d69cb08a77243dad1c205a524c4bfacbe0866d54"},{"author":{"_account_id":4264,"name":"Matthias Runge","email":"mrunge@redhat.com","username":"mrunge"},"change_message_id":"045ab0cfc7ebafd0568618c1bc3f4be0253088fd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"96093763_647aa247","updated":"2025-04-23 19:26:15.000000000","message":"looks good to me, thanks!","commit_id":"d69cb08a77243dad1c205a524c4bfacbe0866d54"},{"author":{"_account_id":34975,"name":"Jaromír Wysoglad","email":"jwysogla@redhat.com","username":"jwysogla"},"change_message_id":"babc801ea909ba7a54250ea54147b87d2dc99885","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"a6b60b87_3f8eb11e","in_reply_to":"408d5933_28ed98b3","updated":"2025-04-14 15:02:39.000000000","message":"Clever. Thanks.","commit_id":"d69cb08a77243dad1c205a524c4bfacbe0866d54"}],"observabilityclient/v1/rbac.py":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"d0e1986f9a55775b5fe701d514481c7ba4410a0f","unresolved":true,"context_lines":[{"line_number":128,"context_line":"                             f\"{query[name_end_location + 1:]}\")"},{"line_number":129,"context_line":"                else:"},{"line_number":130,"context_line":"                    # There already are some labels"},{"line_number":131,"context_line":"                    labels_end \u003d self._find_label_section_end("},{"line_number":132,"context_line":"                        query,"},{"line_number":133,"context_line":"                        name_end_location"},{"line_number":134,"context_line":"                    )"}],"source_content_type":"text/x-python","patch_set":3,"id":"66110de1_c4c822aa","line":131,"range":{"start_line":131,"start_character":38,"end_line":131,"end_character":61},"updated":"2025-04-12 13:02:15.000000000","message":"I wonder if the actual issue is misbehavior of _find_label_section_end which does not handle empty labels and we should fix it instead of adding the workaroud externally ?","commit_id":"eab78d03a4805462a81f562ef4d2f2146174e1b9"}]}