)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":11915,"name":"Jan Klare","email":"klare@osism.tech","username":"jklare"},"change_message_id":"93f9a94c1a3e64b96c7c20c24bc872ab593db559","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"a9cf6982_3d7509d6","updated":"2026-02-13 07:33:35.000000000","message":"Hi, thanks for the PR. I think the upstream image is called redis_exporter, so i would not want to overwrite that image naming here (even though the exporter can collect metrics from both valkey and redis)","commit_id":"a7a2c726646c9a4e57fe852682fd862e70aa1e1d"},{"author":{"_account_id":37306,"name":"Piotr Milewski","display_name":"Piotr Milewski","email":"vurmil@gmail.com","username":"vurmil"},"change_message_id":"1445da8fe0d54906068cde6c205a2b200a26bc64","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"38ce9e1a_e78c68e9","in_reply_to":"844ab4d6_82d53dc4","updated":"2026-03-25 14:45:49.000000000","message":"Acknowledged","commit_id":"a7a2c726646c9a4e57fe852682fd862e70aa1e1d"},{"author":{"_account_id":37306,"name":"Piotr Milewski","display_name":"Piotr Milewski","email":"vurmil@gmail.com","username":"vurmil"},"change_message_id":"2b59d33804edca7fd7dbeef960469b480de1eb05","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"844ab4d6_82d53dc4","in_reply_to":"a9cf6982_3d7509d6","updated":"2026-02-13 09:19:08.000000000","message":"I believe switching to valkey naming improves code clarity and consistency following the migration from Redis, making variable management easier across the project. However, I leave the final decision on this convention to the core reviewers.","commit_id":"a7a2c726646c9a4e57fe852682fd862e70aa1e1d"},{"author":{"_account_id":37306,"name":"Piotr Milewski","display_name":"Piotr Milewski","email":"vurmil@gmail.com","username":"vurmil"},"change_message_id":"2252bec64916a4963d0a841d647daea436098aaa","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"0fdc43e5_9e40c00f","updated":"2026-03-16 16:21:25.000000000","message":"recheck","commit_id":"ad901e0d78488e8edf0648b9d256c3a4a068277c"},{"author":{"_account_id":37306,"name":"Piotr Milewski","display_name":"Piotr Milewski","email":"vurmil@gmail.com","username":"vurmil"},"change_message_id":"0301172459ef703d2fe69e31cccdded4fc77876a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"0fd3f840_a62df9c5","updated":"2026-03-17 06:42:42.000000000","message":"recheck","commit_id":"1257cf784af676b5fc830398e8e3217b7a23ed82"},{"author":{"_account_id":37306,"name":"Piotr Milewski","display_name":"Piotr Milewski","email":"vurmil@gmail.com","username":"vurmil"},"change_message_id":"16dfac324e355973c11794904c59a54abfc28f26","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"f90e5dbe_8eecd2de","updated":"2026-03-17 08:22:03.000000000","message":"recheck","commit_id":"1257cf784af676b5fc830398e8e3217b7a23ed82"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"78c02899c57484567d6a182bb855719d1687227f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"3816c2bf_55d136d4","updated":"2026-03-26 07:51:15.000000000","message":"recheck","commit_id":"d749a6a6772b798a73d535473d43a907ee1197e4"},{"author":{"_account_id":37306,"name":"Piotr Milewski","display_name":"Piotr Milewski","email":"vurmil@gmail.com","username":"vurmil"},"change_message_id":"f5c5c8c481ea8a149a4abba67600e16535a48e64","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"544e648e_3e8f23c2","updated":"2026-03-30 07:08:09.000000000","message":"recheck","commit_id":"d749a6a6772b798a73d535473d43a907ee1197e4"},{"author":{"_account_id":17669,"name":"Doug Szumski","email":"doug@stackhpc.com","username":"DougSzumski"},"change_message_id":"379f0f10663bf317cd5a281f9aa10f2f9ac5c5b1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"b69a5b32_d3ad4e61","updated":"2026-03-26 11:08:43.000000000","message":"recheck - fail looks unrelated","commit_id":"d749a6a6772b798a73d535473d43a907ee1197e4"}],"docker/prometheus/prometheus-valkey-exporter/Dockerfile.j2":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"2cc3bec2ff7079239535625abd604db3cd564600","unresolved":true,"context_lines":[{"line_number":13,"context_line":"ADD prometheus-valkey-exporter-archive /prometheus-valkey-exporter-source"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"RUN ln -s /prometheus-valkey-exporter-source/* prometheus-valkey-exporter \\"},{"line_number":16,"context_line":"    \u0026\u0026 cp /prometheus-valkey-exporter/redis_exporter /opt/redis-exporter"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"{% endblock %}"},{"line_number":19,"context_line":""}],"source_content_type":"text/x-jinja2","patch_set":1,"id":"c850d3ba_8b614177","line":16,"range":{"start_line":16,"start_character":58,"end_line":16,"end_character":63},"updated":"2026-02-17 20:40:49.000000000","message":"```suggestion\n    \u0026\u0026 cp /prometheus-valkey-exporter/redis_exporter /opt/valkey-exporter\n```\nif we do this, let\u0027s go all the way. see also my comment on https://review.opendev.org/c/openstack/kolla-ansible/+/976653","commit_id":"a7a2c726646c9a4e57fe852682fd862e70aa1e1d"},{"author":{"_account_id":37306,"name":"Piotr Milewski","display_name":"Piotr Milewski","email":"vurmil@gmail.com","username":"vurmil"},"change_message_id":"c56569b9261175f490c4f8eb193f94d0f7ebec42","unresolved":false,"context_lines":[{"line_number":13,"context_line":"ADD prometheus-valkey-exporter-archive /prometheus-valkey-exporter-source"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"RUN ln -s /prometheus-valkey-exporter-source/* prometheus-valkey-exporter \\"},{"line_number":16,"context_line":"    \u0026\u0026 cp /prometheus-valkey-exporter/redis_exporter /opt/redis-exporter"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"{% endblock %}"},{"line_number":19,"context_line":""}],"source_content_type":"text/x-jinja2","patch_set":1,"id":"8d336d67_71fea112","line":16,"range":{"start_line":16,"start_character":58,"end_line":16,"end_character":63},"in_reply_to":"c850d3ba_8b614177","updated":"2026-02-18 09:36:42.000000000","message":"Acknowledged","commit_id":"a7a2c726646c9a4e57fe852682fd862e70aa1e1d"}],"kolla/common/sources.py":[{"author":{"_account_id":17669,"name":"Doug Szumski","email":"doug@stackhpc.com","username":"DougSzumski"},"change_message_id":"ad808b4ee67514c369f0b6e6576e122b2756dc0f","unresolved":true,"context_lines":[{"line_number":392,"context_line":"                     \u0027prometheus\u0027"},{"line_number":393,"context_line":"                     \u0027-${version}.linux-${debian_arch}.tar.gz\u0027)},"},{"line_number":394,"context_line":"    \u0027prometheus-valkey-exporter\u0027: {"},{"line_number":395,"context_line":"            \u0027version\u0027: \u00271.81.0\u0027,"},{"line_number":396,"context_line":"            \u0027type\u0027: \u0027url\u0027,"},{"line_number":397,"context_line":"            \u0027sha256\u0027: {"},{"line_number":398,"context_line":"                \u0027amd64\u0027: \u0027d2f4740fa81e1a274ca99677783f3fe1544f6022fed997d4cd632d01a8eff1bb\u0027,  # noqa: E501"}],"source_content_type":"text/x-python","patch_set":3,"id":"6ad454c6_236b6468","line":395,"updated":"2026-03-25 14:34:45.000000000","message":"nit: Please update to latest release (1.82.0)","commit_id":"1257cf784af676b5fc830398e8e3217b7a23ed82"},{"author":{"_account_id":37306,"name":"Piotr Milewski","display_name":"Piotr Milewski","email":"vurmil@gmail.com","username":"vurmil"},"change_message_id":"1445da8fe0d54906068cde6c205a2b200a26bc64","unresolved":false,"context_lines":[{"line_number":392,"context_line":"                     \u0027prometheus\u0027"},{"line_number":393,"context_line":"                     \u0027-${version}.linux-${debian_arch}.tar.gz\u0027)},"},{"line_number":394,"context_line":"    \u0027prometheus-valkey-exporter\u0027: {"},{"line_number":395,"context_line":"            \u0027version\u0027: \u00271.81.0\u0027,"},{"line_number":396,"context_line":"            \u0027type\u0027: \u0027url\u0027,"},{"line_number":397,"context_line":"            \u0027sha256\u0027: {"},{"line_number":398,"context_line":"                \u0027amd64\u0027: \u0027d2f4740fa81e1a274ca99677783f3fe1544f6022fed997d4cd632d01a8eff1bb\u0027,  # noqa: E501"}],"source_content_type":"text/x-python","patch_set":3,"id":"ae0b9473_9c720ceb","line":395,"in_reply_to":"6ad454c6_236b6468","updated":"2026-03-25 14:45:49.000000000","message":"Acknowledged","commit_id":"1257cf784af676b5fc830398e8e3217b7a23ed82"},{"author":{"_account_id":17669,"name":"Doug Szumski","email":"doug@stackhpc.com","username":"DougSzumski"},"change_message_id":"ad808b4ee67514c369f0b6e6576e122b2756dc0f","unresolved":true,"context_lines":[{"line_number":398,"context_line":"                \u0027amd64\u0027: \u0027d2f4740fa81e1a274ca99677783f3fe1544f6022fed997d4cd632d01a8eff1bb\u0027,  # noqa: E501"},{"line_number":399,"context_line":"                \u0027arm64\u0027: \u0027038d0d2d8f044075cc97acd323c16a8c3628f08ef7d186601fedaa38b71aad8f\u0027},  # noqa: E501"},{"line_number":400,"context_line":"            \u0027location\u0027: (\u0027https://github.com/\u0027"},{"line_number":401,"context_line":"                         \u0027oliver006/redis_exporter/\u0027"},{"line_number":402,"context_line":"                         \u0027releases/download/v${version}/\u0027"},{"line_number":403,"context_line":"                         \u0027redis_exporter\u0027"},{"line_number":404,"context_line":"                         \u0027-v${version}.linux-${debian_arch}.tar.gz\u0027)},"}],"source_content_type":"text/x-python","patch_set":3,"id":"33c4b8d9_92fe9bfa","line":401,"updated":"2026-03-25 14:34:45.000000000","message":"I\u0027m nervous about adding another 3rd party exporter, but it looks like the best option. There is some activity from other contributors:\n\nhttps://github.com/oliver006/redis_exporter/graphs/contributors?from\u003d20%2F12%2F2025","commit_id":"1257cf784af676b5fc830398e8e3217b7a23ed82"},{"author":{"_account_id":37306,"name":"Piotr Milewski","display_name":"Piotr Milewski","email":"vurmil@gmail.com","username":"vurmil"},"change_message_id":"14d490b2f577c796c96de51e4e1160cf39a20c68","unresolved":true,"context_lines":[{"line_number":398,"context_line":"                \u0027amd64\u0027: \u0027d2f4740fa81e1a274ca99677783f3fe1544f6022fed997d4cd632d01a8eff1bb\u0027,  # noqa: E501"},{"line_number":399,"context_line":"                \u0027arm64\u0027: \u0027038d0d2d8f044075cc97acd323c16a8c3628f08ef7d186601fedaa38b71aad8f\u0027},  # noqa: E501"},{"line_number":400,"context_line":"            \u0027location\u0027: (\u0027https://github.com/\u0027"},{"line_number":401,"context_line":"                         \u0027oliver006/redis_exporter/\u0027"},{"line_number":402,"context_line":"                         \u0027releases/download/v${version}/\u0027"},{"line_number":403,"context_line":"                         \u0027redis_exporter\u0027"},{"line_number":404,"context_line":"                         \u0027-v${version}.linux-${debian_arch}.tar.gz\u0027)},"}],"source_content_type":"text/x-python","patch_set":3,"id":"8e25ddd8_c7ab17f2","line":401,"in_reply_to":"33c4b8d9_92fe9bfa","updated":"2026-03-25 14:45:15.000000000","message":"I also have concerns, but it works and displays what is needed. there is no better alternative at the moment","commit_id":"1257cf784af676b5fc830398e8e3217b7a23ed82"},{"author":{"_account_id":37306,"name":"Piotr Milewski","display_name":"Piotr Milewski","email":"vurmil@gmail.com","username":"vurmil"},"change_message_id":"1445da8fe0d54906068cde6c205a2b200a26bc64","unresolved":false,"context_lines":[{"line_number":398,"context_line":"                \u0027amd64\u0027: \u0027d2f4740fa81e1a274ca99677783f3fe1544f6022fed997d4cd632d01a8eff1bb\u0027,  # noqa: E501"},{"line_number":399,"context_line":"                \u0027arm64\u0027: \u0027038d0d2d8f044075cc97acd323c16a8c3628f08ef7d186601fedaa38b71aad8f\u0027},  # noqa: E501"},{"line_number":400,"context_line":"            \u0027location\u0027: (\u0027https://github.com/\u0027"},{"line_number":401,"context_line":"                         \u0027oliver006/redis_exporter/\u0027"},{"line_number":402,"context_line":"                         \u0027releases/download/v${version}/\u0027"},{"line_number":403,"context_line":"                         \u0027redis_exporter\u0027"},{"line_number":404,"context_line":"                         \u0027-v${version}.linux-${debian_arch}.tar.gz\u0027)},"}],"source_content_type":"text/x-python","patch_set":3,"id":"33d05c08_3e1709e5","line":401,"in_reply_to":"8e25ddd8_c7ab17f2","updated":"2026-03-25 14:45:49.000000000","message":"Acknowledged","commit_id":"1257cf784af676b5fc830398e8e3217b7a23ed82"}]}
