)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"37e63068952743ed295f906b6168ffc688f94c9b","unresolved":true,"context_lines":[{"line_number":19,"context_line":"compatibility."},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"Closes-Bug: #2154609"},{"line_number":22,"context_line":"Generated-By: claude-opus-4-6 (OpenCode)"},{"line_number":23,"context_line":"Change-Id: If3c7ee9ea16e13047e5ec8b8463d022abca745b5"},{"line_number":24,"context_line":"Signed-off-by: flux.adam@gmail.com"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"15976a76_c26c317d","line":22,"range":{"start_line":22,"start_character":0,"end_line":22,"end_character":12},"updated":"2026-06-01 08:15:06.000000000","message":"Please check https://openinfra.org/legal/ai-policy\n```\nAssistive AI tools are permitted, as long as contributions are marked with an “Assisted-By:” label in commit messages.\n```\n\nIn other words: you are still responsible of the code submitted, not the AI.","commit_id":"91270e2e44101d15e4d782227dcba05c5ae319fa"},{"author":{"_account_id":10273,"name":"Adam Harwell","email":"flux.adam@gmail.com","username":"rm_you"},"change_message_id":"7d7fbd8bc28881e0628a27d2bebac68104f9d587","unresolved":true,"context_lines":[{"line_number":19,"context_line":"compatibility."},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"Closes-Bug: #2154609"},{"line_number":22,"context_line":"Generated-By: claude-opus-4-6 (OpenCode)"},{"line_number":23,"context_line":"Change-Id: If3c7ee9ea16e13047e5ec8b8463d022abca745b5"},{"line_number":24,"context_line":"Signed-off-by: flux.adam@gmail.com"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"df4d4633_d8cab040","line":22,"range":{"start_line":22,"start_character":0,"end_line":22,"end_character":12},"in_reply_to":"15976a76_c26c317d","updated":"2026-06-01 09:30:50.000000000","message":"Yep, I have Generated-By and have signed off, personally reviewed by hand, and tested this code.\n\nPer that page:\n\n\u003e If a substantial portion of a patch is generated in this way, it must be marked with a “Generated-By:” label.","commit_id":"91270e2e44101d15e4d782227dcba05c5ae319fa"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"5c9692391e4080c0379a0c8bc2bf3820b1647679","unresolved":false,"context_lines":[{"line_number":19,"context_line":"compatibility."},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"Closes-Bug: #2154609"},{"line_number":22,"context_line":"Generated-By: claude-opus-4-6 (OpenCode)"},{"line_number":23,"context_line":"Change-Id: If3c7ee9ea16e13047e5ec8b8463d022abca745b5"},{"line_number":24,"context_line":"Signed-off-by: flux.adam@gmail.com"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"4cad9cd1_4612df2a","line":22,"range":{"start_line":22,"start_character":0,"end_line":22,"end_character":12},"in_reply_to":"df4d4633_d8cab040","updated":"2026-06-01 10:09:16.000000000","message":"ok","commit_id":"91270e2e44101d15e4d782227dcba05c5ae319fa"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":10273,"name":"Adam Harwell","email":"flux.adam@gmail.com","username":"rm_you"},"change_message_id":"0f530f75dfd0596fd2df04f1350bdb689a7a927f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"2f4f49f4_fb9f3e4e","updated":"2026-05-29 21:19:57.000000000","message":"recheck unrelated multinode test failure","commit_id":"59bb3c32d758689bfa831a354f4a24365f4de1fb"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"37e63068952743ed295f906b6168ffc688f94c9b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"516950cb_b7d44609","updated":"2026-06-01 08:15:06.000000000","message":"I think we had a spec/RFE to use a template as metadata configuration. But I don\u0027t remember where this proposal was.\n\nThe idea of having a static template and provide independent configuration options is safer, for sure.\n\nI\u0027m ok with this implementation that is also extensible.\n\n-1 just for the comments.","commit_id":"91270e2e44101d15e4d782227dcba05c5ae319fa"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"8685e6e9a958a13591b2b406b09a90e3f67cc9f8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"47ef17ac_8ed4acb5","updated":"2026-06-02 12:37:30.000000000","message":"generally good idea, thanks.","commit_id":"91270e2e44101d15e4d782227dcba05c5ae319fa"}],"neutron/conf/agent/ovn/metadata/config.py":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"37e63068952743ed295f906b6168ffc688f94c9b","unresolved":true,"context_lines":[{"line_number":51,"context_line":"        (meta_conf.RATE_LIMITING_GROUP,"},{"line_number":52,"context_line":"         meta_conf.METADATA_RATE_LIMITING_OPTS),"},{"line_number":53,"context_line":"        (meta_conf.HAPROXY_GROUP,"},{"line_number":54,"context_line":"         meta_conf.METADATA_HAPROXY_OPTS)"},{"line_number":55,"context_line":"    ]"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"4150616a_02fef3db","line":54,"range":{"start_line":54,"start_character":38,"end_line":54,"end_character":41},"updated":"2026-06-01 08:15:06.000000000","message":"better to add a comma at the end, in order to preserve the line git history the next time","commit_id":"91270e2e44101d15e4d782227dcba05c5ae319fa"},{"author":{"_account_id":10273,"name":"Adam Harwell","email":"flux.adam@gmail.com","username":"rm_you"},"change_message_id":"7d7fbd8bc28881e0628a27d2bebac68104f9d587","unresolved":true,"context_lines":[{"line_number":51,"context_line":"        (meta_conf.RATE_LIMITING_GROUP,"},{"line_number":52,"context_line":"         meta_conf.METADATA_RATE_LIMITING_OPTS),"},{"line_number":53,"context_line":"        (meta_conf.HAPROXY_GROUP,"},{"line_number":54,"context_line":"         meta_conf.METADATA_HAPROXY_OPTS)"},{"line_number":55,"context_line":"    ]"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"80ac40fb_7a315a8f","line":54,"range":{"start_line":54,"start_character":38,"end_line":54,"end_character":41},"in_reply_to":"4150616a_02fef3db","updated":"2026-06-01 09:30:50.000000000","message":"I’m not sure I necessarily see the point of premature optimization of a few commas here, but I’ll do it if you feel strongly.","commit_id":"91270e2e44101d15e4d782227dcba05c5ae319fa"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"223e7067f019650b75a43dba14d4ad0fb50c8ba2","unresolved":true,"context_lines":[{"line_number":51,"context_line":"        (meta_conf.RATE_LIMITING_GROUP,"},{"line_number":52,"context_line":"         meta_conf.METADATA_RATE_LIMITING_OPTS),"},{"line_number":53,"context_line":"        (meta_conf.HAPROXY_GROUP,"},{"line_number":54,"context_line":"         meta_conf.METADATA_HAPROXY_OPTS)"},{"line_number":55,"context_line":"    ]"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"21370ac0_81047a85","line":54,"range":{"start_line":54,"start_character":38,"end_line":54,"end_character":41},"in_reply_to":"80ac40fb_7a315a8f","updated":"2026-06-01 10:04:59.000000000","message":"As you can see in the current file (and the others modify by this patch), the L52 is loosing the `git blame` owner due to this change. If any one is adding an extra configuration here, we\u0027ll loose the information of L54.\n\nIn Neutron, we recommend in these cases to add additional commas at the end the lines of a list or tuple just to keep the `git blame` reference.\n\nAs an example: https://github.com/openstack/neutron/blob/e9abc267d0911c27b2f9772e7cb08f3af3fc3a6d/neutron/plugins/ml2/plugin.py#L216-L263","commit_id":"91270e2e44101d15e4d782227dcba05c5ae319fa"}],"neutron/conf/agent/ovn/ovn_neutron_agent/config.py":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"37e63068952743ed295f906b6168ffc688f94c9b","unresolved":true,"context_lines":[{"line_number":47,"context_line":"                                )"},{"line_number":48,"context_line":"         ),"},{"line_number":49,"context_line":"        (meta_conf.RATE_LIMITING_GROUP, meta_conf.METADATA_RATE_LIMITING_OPTS),"},{"line_number":50,"context_line":"        (meta_conf.HAPROXY_GROUP, meta_conf.METADATA_HAPROXY_OPTS)"},{"line_number":51,"context_line":"    ]"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"f88ebc2e_ca056613","line":50,"range":{"start_line":50,"start_character":7,"end_line":50,"end_character":66},"updated":"2026-06-01 08:15:06.000000000","message":"better to add a comma at the end, in order to preserve the line git history the next time","commit_id":"91270e2e44101d15e4d782227dcba05c5ae319fa"}],"neutron/opts.py":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"37e63068952743ed295f906b6168ffc688f94c9b","unresolved":true,"context_lines":[{"line_number":241,"context_line":"         ),"},{"line_number":242,"context_line":"        (meta_conf.RATE_LIMITING_GROUP,"},{"line_number":243,"context_line":"         meta_conf.METADATA_RATE_LIMITING_OPTS),"},{"line_number":244,"context_line":"        (meta_conf.HAPROXY_GROUP,"},{"line_number":245,"context_line":"         meta_conf.METADATA_HAPROXY_OPTS)"},{"line_number":246,"context_line":"    ]"},{"line_number":247,"context_line":""},{"line_number":248,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"abaa59a0_d2e9d888","line":245,"range":{"start_line":244,"start_character":7,"end_line":245,"end_character":41},"updated":"2026-06-01 08:15:06.000000000","message":"better to add a comma at the end, in order to preserve the line git history the next time","commit_id":"91270e2e44101d15e4d782227dcba05c5ae319fa"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"37e63068952743ed295f906b6168ffc688f94c9b","unresolved":true,"context_lines":[{"line_number":261,"context_line":"         neutron.conf.services.logging.log_driver_opts),"},{"line_number":262,"context_line":"        (meta_conf.RATE_LIMITING_GROUP,"},{"line_number":263,"context_line":"         meta_conf.METADATA_RATE_LIMITING_OPTS),"},{"line_number":264,"context_line":"        (meta_conf.HAPROXY_GROUP,"},{"line_number":265,"context_line":"         meta_conf.METADATA_HAPROXY_OPTS)"},{"line_number":266,"context_line":"    ]"},{"line_number":267,"context_line":""},{"line_number":268,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"9beb1301_82f01214","line":265,"range":{"start_line":264,"start_character":7,"end_line":265,"end_character":41},"updated":"2026-06-01 08:15:06.000000000","message":"ditto","commit_id":"91270e2e44101d15e4d782227dcba05c5ae319fa"}]}