)]}'
{"gerrit/acls/openstack/openstacksdk.config":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"50fad11faddbe87c39c97fafce4d1bf0ba605a09","unresolved":true,"context_lines":[{"line_number":7,"context_line":"[access \"refs/heads/*\"]"},{"line_number":8,"context_line":"\tabandon \u003d group openstacksdk-core"},{"line_number":9,"context_line":"\tlabel-Allow-Post-Review \u003d +0..+1 group openstacksdk-core"},{"line_number":10,"context_line":"\tlabel-Code-Review \u003d -2..+2 group cinder-core"},{"line_number":11,"context_line":"\tlabel-Code-Review \u003d -2..+2 group cyborg-core"},{"line_number":12,"context_line":"\tlabel-Code-Review \u003d -2..+2 group designate-core"},{"line_number":13,"context_line":"\tlabel-Code-Review \u003d -2..+2 group ironic-reviewers"},{"line_number":14,"context_line":"\tlabel-Code-Review \u003d -2..+2 group manila-core"},{"line_number":15,"context_line":"\tlabel-Code-Review \u003d -2..+2 group masakari-core"},{"line_number":16,"context_line":"\tlabel-Code-Review \u003d -2..+2 group neutron-core"},{"line_number":17,"context_line":"\tlabel-Code-Review \u003d -2..+2 group nova-core"},{"line_number":18,"context_line":"\tlabel-Code-Review \u003d -2..+2 group octavia-core"},{"line_number":19,"context_line":"\tlabel-Code-Review \u003d -2..+2 group openstacksdk-core"},{"line_number":20,"context_line":"\tlabel-Code-Review \u003d -2..+2 group openstacksdk-service-core"},{"line_number":21,"context_line":"\tlabel-Workflow \u003d -1..+1 group openstacksdk-core"}],"source_content_type":"text/x-ini","patch_set":2,"id":"121b75a0_5ab50395","line":18,"range":{"start_line":10,"start_character":1,"end_line":18,"end_character":46},"updated":"2026-01-22 17:19:54.000000000","message":"```suggestion\n# delete all these\n```","commit_id":"a44359cf982be20b7e95311c1fe829a8acd408a1"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"bb315caa22c826cb108a4f3c7711531583ccfc8d","unresolved":true,"context_lines":[{"line_number":17,"context_line":"\tlabel-Code-Review \u003d -2..+2 group nova-core"},{"line_number":18,"context_line":"\tlabel-Code-Review \u003d -2..+2 group octavia-core"},{"line_number":19,"context_line":"\tlabel-Code-Review \u003d -2..+2 group openstacksdk-core"},{"line_number":20,"context_line":"\tlabel-Code-Review \u003d -2..+2 group openstacksdk-service-core"},{"line_number":21,"context_line":"\tlabel-Workflow \u003d -1..+1 group openstacksdk-core"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"[access \"refs/heads/stable/*\"]"}],"source_content_type":"text/x-ini","patch_set":2,"id":"b51ed6f6_e15e0ade","line":20,"updated":"2026-01-22 07:01:54.000000000","message":"it seems this setup is currently duplicated, all of the groups above are also members of openstacksdk-service-core, so this ACL is kind of redundant. can we unify this into one direction? from an infra pov having the access managed via this group would be simpler, as is can be managed without repo changes, so I\u0027d have a slight preference for that","commit_id":"a44359cf982be20b7e95311c1fe829a8acd408a1"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"50fad11faddbe87c39c97fafce4d1bf0ba605a09","unresolved":true,"context_lines":[{"line_number":17,"context_line":"\tlabel-Code-Review \u003d -2..+2 group nova-core"},{"line_number":18,"context_line":"\tlabel-Code-Review \u003d -2..+2 group octavia-core"},{"line_number":19,"context_line":"\tlabel-Code-Review \u003d -2..+2 group openstacksdk-core"},{"line_number":20,"context_line":"\tlabel-Code-Review \u003d -2..+2 group openstacksdk-service-core"},{"line_number":21,"context_line":"\tlabel-Workflow \u003d -1..+1 group openstacksdk-core"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"[access \"refs/heads/stable/*\"]"}],"source_content_type":"text/x-ini","patch_set":2,"id":"7c82e60e_a11c9cb5","line":20,"in_reply_to":"44d6dff2_9a48923d","updated":"2026-01-22 17:19:54.000000000","message":"I have updated the group now, so this change here should no longer be needed. my suggestion would be to instead delete all the explicit listings above","commit_id":"a44359cf982be20b7e95311c1fe829a8acd408a1"},{"author":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"change_message_id":"1dde001290cc63b148e5c08c363cbb1a3be7daef","unresolved":true,"context_lines":[{"line_number":17,"context_line":"\tlabel-Code-Review \u003d -2..+2 group nova-core"},{"line_number":18,"context_line":"\tlabel-Code-Review \u003d -2..+2 group octavia-core"},{"line_number":19,"context_line":"\tlabel-Code-Review \u003d -2..+2 group openstacksdk-core"},{"line_number":20,"context_line":"\tlabel-Code-Review \u003d -2..+2 group openstacksdk-service-core"},{"line_number":21,"context_line":"\tlabel-Workflow \u003d -1..+1 group openstacksdk-core"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"[access \"refs/heads/stable/*\"]"}],"source_content_type":"text/x-ini","patch_set":2,"id":"44d6dff2_9a48923d","line":20,"in_reply_to":"b51ed6f6_e15e0ade","updated":"2026-01-22 17:06:35.000000000","message":"Where? And how do I do that? I\u0027m part of ironic-approvers but not part of ironic-core and I don\u0027t have +2 so the inclusion currently isn\u0027t working for me.","commit_id":"a44359cf982be20b7e95311c1fe829a8acd408a1"}],"gerrit/acls/openstack/python-openstackclient.config":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"bb315caa22c826cb108a4f3c7711531583ccfc8d","unresolved":true,"context_lines":[{"line_number":16,"context_line":"\tlabel-Code-Review \u003d -2..+2 group nova-core"},{"line_number":17,"context_line":"\tlabel-Code-Review \u003d -2..+2 group octavia-core"},{"line_number":18,"context_line":"\tlabel-Code-Review \u003d -2..+2 group python-openstackclient-core"},{"line_number":19,"context_line":"\tlabel-Code-Review \u003d -2..+2 group python-openstackclient-service-core"},{"line_number":20,"context_line":"\tlabel-Workflow \u003d -1..+1 group python-openstackclient-core"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"[access \"refs/heads/stable/*\"]"}],"source_content_type":"text/x-ini","patch_set":2,"id":"cc5392cf_b24f7192","line":19,"updated":"2026-01-22 07:01:54.000000000","message":"same holds here, as this group simply references openstacksdk-service-core","commit_id":"a44359cf982be20b7e95311c1fe829a8acd408a1"}]}
