)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"2eb93955d9f38670f1ac5d56258bd086793a86d2","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     Joel Capitao \u003cjcapitao@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2024-12-16 17:17:27 +0100"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Authorize packstack-core to force push to remove branch"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"We need this authorization to be able to remove old branches."},{"line_number":10,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"5ebcf2cd_9899a3d8","line":7,"updated":"2024-12-16 16:22:13.000000000","message":"Note the change itself does not match this statement. We are not allowing force push simply allowing API/UI driven branch deletions.","commit_id":"f4d084cbc08310333c0f84998761553d43c5a989"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"4481a1f1e0eb09ab43f3be02d580a7fac98e5f2c","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Joel Capitao \u003cjcapitao@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2024-12-16 17:17:27 +0100"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Authorize packstack-core to force push to remove branch"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"We need this authorization to be able to remove old branches."},{"line_number":10,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"fa92c3d2_806d1284","line":7,"in_reply_to":"5ebcf2cd_9899a3d8","updated":"2024-12-17 09:17:09.000000000","message":"Done","commit_id":"f4d084cbc08310333c0f84998761553d43c5a989"},{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"1a5c1730140e58da5c1b789bd7ea8746218cdd9b","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Joel Capitao \u003cjcapitao@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2024-12-16 18:11:50 +0100"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Authorize packstack-core to force push to remove branch"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"We need this authorization to be able to remove old branches."},{"line_number":10,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"f3b4469c_b23aec22","line":7,"updated":"2024-12-16 17:57:04.000000000","message":"Nit: Title is inaccurate after recent revisions.","commit_id":"cece80ba49b77b5054fd9fcf6eddb6c692c438bb"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":31068,"name":"Joel Capitao","email":"jcapitao@redhat.com","username":"jcapiitao"},"change_message_id":"2736e9cfb23aebdba9e7d630ea8fbbe361c22ff6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"24462bea_e79d40fc","updated":"2024-12-16 16:09:40.000000000","message":"this is needed to clean up https://zuul.opendev.org/t/openstack/config-errors?project\u003dx%2Fpackstack\u0026severity\u003derror\u0026skip\u003d0\u0026limit\u003d50","commit_id":"681b75358bc5ce6d30944cb8c7dad0454109271a"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"4481a1f1e0eb09ab43f3be02d580a7fac98e5f2c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"4e5a3a96_d1288cb1","updated":"2024-12-17 09:17:09.000000000","message":"thx for the update","commit_id":"6553f6c50ed34e6fb57e699a3920f4f67e9a9467"}],"gerrit/acls/x/packstack.config":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"7edd0adaa99d5b23d275afd36f22720d7b15b4ed","unresolved":true,"context_lines":[{"line_number":1,"context_line":"[access \"refs/heads/*\"]"},{"line_number":2,"context_line":"\tabandon \u003d group packstack-core"},{"line_number":3,"context_line":"\tcreate \u003d group packstack-release"},{"line_number":4,"context_line":"\tpush \u003d +force group packstack-core"},{"line_number":5,"context_line":"\tlabel-Code-Review \u003d -2..+2 group packstack-core"},{"line_number":6,"context_line":"\tlabel-Verified \u003d -1..+1 group tripleo-ci"},{"line_number":7,"context_line":"\tlabel-Workflow \u003d -1..+1 group packstack-core"}],"source_content_type":"text/x-ini","patch_set":1,"id":"2a7b5ea6_33dd8d7a","line":4,"updated":"2024-12-16 16:10:08.000000000","message":"what we usually would have to delete branches is:\n\n```\n[access \"refs/*\"]\n        delete \u003d packstack-core\n```\n(with a tab instead of 8 spaces)","commit_id":"681b75358bc5ce6d30944cb8c7dad0454109271a"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"2eb93955d9f38670f1ac5d56258bd086793a86d2","unresolved":false,"context_lines":[{"line_number":1,"context_line":"[access \"refs/heads/*\"]"},{"line_number":2,"context_line":"\tabandon \u003d group packstack-core"},{"line_number":3,"context_line":"\tcreate \u003d group packstack-release"},{"line_number":4,"context_line":"\tpush \u003d +force group packstack-core"},{"line_number":5,"context_line":"\tlabel-Code-Review \u003d -2..+2 group packstack-core"},{"line_number":6,"context_line":"\tlabel-Verified \u003d -1..+1 group tripleo-ci"},{"line_number":7,"context_line":"\tlabel-Workflow \u003d -1..+1 group packstack-core"}],"source_content_type":"text/x-ini","patch_set":1,"id":"9ad98af3_f03030eb","line":4,"in_reply_to":"011f97f3_fc0f8bfc","updated":"2024-12-16 16:22:13.000000000","message":"There are two methods of deleting branches/references in Gerrit. One is to allow force push to the reference which allows you to also merge code. We try to avoid this. The other is via the delete permission which doesn\u0027t let you use git operations to delete the reference instead you must use the gerrit API or web UI to do so.","commit_id":"681b75358bc5ce6d30944cb8c7dad0454109271a"},{"author":{"_account_id":31068,"name":"Joel Capitao","email":"jcapitao@redhat.com","username":"jcapiitao"},"change_message_id":"c60cd5350854f684f1948990a68af50f1d882f80","unresolved":false,"context_lines":[{"line_number":1,"context_line":"[access \"refs/heads/*\"]"},{"line_number":2,"context_line":"\tabandon \u003d group packstack-core"},{"line_number":3,"context_line":"\tcreate \u003d group packstack-release"},{"line_number":4,"context_line":"\tpush \u003d +force group packstack-core"},{"line_number":5,"context_line":"\tlabel-Code-Review \u003d -2..+2 group packstack-core"},{"line_number":6,"context_line":"\tlabel-Verified \u003d -1..+1 group tripleo-ci"},{"line_number":7,"context_line":"\tlabel-Workflow \u003d -1..+1 group packstack-core"}],"source_content_type":"text/x-ini","patch_set":1,"id":"011f97f3_fc0f8bfc","line":4,"in_reply_to":"2a7b5ea6_33dd8d7a","updated":"2024-12-16 16:17:34.000000000","message":"Thank you for the feedback\nI made an assumption based on the error message \n`remote: You need \u0027Delete Reference\u0027 rights or \u0027Push\u0027 rights with the\nremote: \u0027Force Push\u0027 flag set to delete references.`","commit_id":"681b75358bc5ce6d30944cb8c7dad0454109271a"},{"author":{"_account_id":31068,"name":"Joel Capitao","email":"jcapitao@redhat.com","username":"jcapiitao"},"change_message_id":"e75e47a7c59c8bf70be81d9f543af0c4fdc1e026","unresolved":false,"context_lines":[{"line_number":1,"context_line":"[access \"refs/heads/*\"]"},{"line_number":2,"context_line":"\tabandon \u003d group packstack-core"},{"line_number":3,"context_line":"\tcreate \u003d group packstack-release"},{"line_number":4,"context_line":"\tpush \u003d +force group packstack-core"},{"line_number":5,"context_line":"\tlabel-Code-Review \u003d -2..+2 group packstack-core"},{"line_number":6,"context_line":"\tlabel-Verified \u003d -1..+1 group tripleo-ci"},{"line_number":7,"context_line":"\tlabel-Workflow \u003d -1..+1 group packstack-core"}],"source_content_type":"text/x-ini","patch_set":1,"id":"8ae9c53e_58a06cb0","line":4,"in_reply_to":"9ad98af3_f03030eb","updated":"2024-12-16 17:15:49.000000000","message":"That sounds better than \"force push\"","commit_id":"681b75358bc5ce6d30944cb8c7dad0454109271a"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"2eb93955d9f38670f1ac5d56258bd086793a86d2","unresolved":true,"context_lines":[{"line_number":1,"context_line":"[access \"refs/heads/*\"]"},{"line_number":2,"context_line":"\tabandon \u003d group packstack-core"},{"line_number":3,"context_line":"\tcreate \u003d group packstack-release"},{"line_number":4,"context_line":"\tdelete \u003d packstack-core"},{"line_number":5,"context_line":"\tlabel-Code-Review \u003d -2..+2 group packstack-core"},{"line_number":6,"context_line":"\tlabel-Verified \u003d -1..+1 group tripleo-ci"},{"line_number":7,"context_line":"\tlabel-Workflow \u003d -1..+1 group packstack-core"}],"source_content_type":"text/x-ini","patch_set":2,"id":"3185c77a_021d12db","line":4,"updated":"2024-12-16 16:22:13.000000000","message":"Consider giving this permission to just the packstack-release team to avoid unnecessary permissions.","commit_id":"f4d084cbc08310333c0f84998761553d43c5a989"},{"author":{"_account_id":31068,"name":"Joel Capitao","email":"jcapitao@redhat.com","username":"jcapiitao"},"change_message_id":"f2db5aa8cdb585fa240e821738d4e36ff82a085c","unresolved":false,"context_lines":[{"line_number":1,"context_line":"[access \"refs/heads/*\"]"},{"line_number":2,"context_line":"\tabandon \u003d group packstack-core"},{"line_number":3,"context_line":"\tcreate \u003d group packstack-release"},{"line_number":4,"context_line":"\tdelete \u003d packstack-core"},{"line_number":5,"context_line":"\tlabel-Code-Review \u003d -2..+2 group packstack-core"},{"line_number":6,"context_line":"\tlabel-Verified \u003d -1..+1 group tripleo-ci"},{"line_number":7,"context_line":"\tlabel-Workflow \u003d -1..+1 group packstack-core"}],"source_content_type":"text/x-ini","patch_set":2,"id":"aa987775_b75ce813","line":4,"in_reply_to":"2d2751cd_ac59945a","updated":"2024-12-17 08:28:07.000000000","message":"Good point, fixed in PS4","commit_id":"f4d084cbc08310333c0f84998761553d43c5a989"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"74753c1adff7ac4dc9a14e27ad06ef8da8660dd1","unresolved":true,"context_lines":[{"line_number":1,"context_line":"[access \"refs/heads/*\"]"},{"line_number":2,"context_line":"\tabandon \u003d group packstack-core"},{"line_number":3,"context_line":"\tcreate \u003d group packstack-release"},{"line_number":4,"context_line":"\tdelete \u003d packstack-core"},{"line_number":5,"context_line":"\tlabel-Code-Review \u003d -2..+2 group packstack-core"},{"line_number":6,"context_line":"\tlabel-Verified \u003d -1..+1 group tripleo-ci"},{"line_number":7,"context_line":"\tlabel-Workflow \u003d -1..+1 group packstack-core"}],"source_content_type":"text/x-ini","patch_set":2,"id":"2d2751cd_ac59945a","line":4,"in_reply_to":"3185c77a_021d12db","updated":"2024-12-17 06:50:39.000000000","message":"Ah, yes, I missed in my previous comment that there is also the -release group, so matching create and delete permissions seems better","commit_id":"f4d084cbc08310333c0f84998761553d43c5a989"}]}