)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"change_message_id":"490164c6aba11397fca2ede894b8f3b9fa19337b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"77d5b769_8cf853a2","updated":"2026-08-10 16:45:02.000000000","message":"Most likely not do anything. So far nobody using it in prod complained (long-running openstacksdk is anyway not a trivial thing), so let\u0027s leave it unless this really breaks somebody","commit_id":"299233afa7dde8843459b08e6b87fe81447dd799"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"a853b0e35fa67a1ada883edb336f7dd5919adf43","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"6a0d53f2_186a2138","updated":"2026-08-10 10:37:12.000000000","message":"This makes sense but I\u0027d like to Artem to take a look at this also","commit_id":"299233afa7dde8843459b08e6b87fe81447dd799"},{"author":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"change_message_id":"9e6fe986b04082fb2e12f8c7f6a5a22015720a6c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"d83fa136_bbeec466","updated":"2026-08-10 12:59:53.000000000","message":"While I agree with the reasoning (unsanitized url MUST never be used as a label), simply dropping it is worse as not having any metrics at all since you literally merge different latencies of fast operation with slow ones. Influxdb and statsd reporter normalize the url down to the resource specification (removing all unique identifiers out of it) so that you can differentiate create server from create project operations. Please use the same method as there (it\u0027s literally just some lines above in the report_stats_statsd handler. \nOther than that this is the breaking change on the metrics side with the data all of a sudden coming very differently - it can\u0027t be a simple bigfix.","commit_id":"299233afa7dde8843459b08e6b87fe81447dd799"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"0a881cb43f2a7b5a423f001b66bb10f50091d954","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"2622d26a_fde5d149","updated":"2026-08-10 13:08:15.000000000","message":"aaaand that\u0027s why I wanted Artem to look at this 😀","commit_id":"299233afa7dde8843459b08e6b87fe81447dd799"},{"author":{"_account_id":39263,"name":"Udayendu Kar","display_name":"Udayendu Kar","email":"udayendu.kar@gmail.com","username":"udayendu.kar"},"change_message_id":"30bb6a9a7fe85c6284edc46fdad3c1a93dcc431e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"00da42c2_50fd2127","updated":"2026-08-10 10:52:45.000000000","message":"recheck","commit_id":"299233afa7dde8843459b08e6b87fe81447dd799"},{"author":{"_account_id":39263,"name":"Udayendu Kar","display_name":"Udayendu Kar","email":"udayendu.kar@gmail.com","username":"udayendu.kar"},"change_message_id":"0edb50ca300c565122c6db1a0be63556974976f0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"c4fc555e_f6abfe76","updated":"2026-08-10 05:44:54.000000000","message":"recheck","commit_id":"299233afa7dde8843459b08e6b87fe81447dd799"},{"author":{"_account_id":39263,"name":"Udayendu Kar","display_name":"Udayendu Kar","email":"udayendu.kar@gmail.com","username":"udayendu.kar"},"change_message_id":"6dfb08cff9d70493cd8aa8388e25f4ee96463000","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"ff0c2672_3f8d9397","in_reply_to":"77d5b769_8cf853a2","updated":"2026-08-10 16:55:04.000000000","message":"Sure.","commit_id":"299233afa7dde8843459b08e6b87fe81447dd799"},{"author":{"_account_id":39263,"name":"Udayendu Kar","display_name":"Udayendu Kar","email":"udayendu.kar@gmail.com","username":"udayendu.kar"},"change_message_id":"ace38c34a51dbe7c2b24d582a7af489924b75d6f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"f5db1d3f_9238ac3d","in_reply_to":"d83fa136_bbeec466","updated":"2026-08-10 14:21:35.000000000","message":"Thanks Artem. So, should we do any work on it or leave it as it is for now and will take some time in future.","commit_id":"299233afa7dde8843459b08e6b87fe81447dd799"}],"openstack/tests/unit/test_proxy.py":[{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"a9f12457109170b99472867486985fa457380805","unresolved":false,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"import copy"},{"line_number":14,"context_line":"import queue"},{"line_number":15,"context_line":"import prometheus_client"},{"line_number":16,"context_line":"from requests import Response"},{"line_number":17,"context_line":"from unittest import mock"},{"line_number":18,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"9ed6f310_a6b52221","line":15,"updated":"2026-08-09 18:45:27.000000000","message":"pep8: H306: imports not in alphabetical order (queue, prometheus_client)","commit_id":"7a3a47147b15d2ee9f1daf0852139254f0c8809f"}]}