)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"9449afd96e2e2b70f04271a74483c0677a1b8f3b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"0566df12_447605a6","updated":"2022-12-18 22:37:34.000000000","message":"\u003e Patch Set 2:\n\u003e \n\u003e We\u0027re also going to need to do this to the All-Projects ACL (which we don\u0027t have under configuration management), right?\n\nYep, I have a change for system-config where we do document the initial settings.\n\nI think we should add it to the gate testing -- I\u0027m trying to get things a bit more into CI before I/we report anything upstream about the columns showing backport/review priority to see what they think about them disappearing on 3.6","commit_id":"6349f18d405642ec7cb113235b2b37035aeb14bf"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"b01465bc5e86fb14d729b55844d1d14dc84cb9fa","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"14dd3da4_f2bd295a","updated":"2023-01-24 22:34:41.000000000","message":"Not approving this because I think we should announce it before hand. Not necessarily with a ton of lead time, but before we land it so that people can call out changes of behavior if it occurs. Most of these repos are repos that we don\u0027t interact with every day but others do so they can help tell us if anything has gone wrong.","commit_id":"6349f18d405642ec7cb113235b2b37035aeb14bf"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"42d23eb2fa143446636e6b391e8f1d9b32c997c0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"ca1d71a0_3c8b06b5","updated":"2023-02-08 08:03:31.000000000","message":"Thanks, looks ok, I ask other Neutron folks also to check it","commit_id":"6349f18d405642ec7cb113235b2b37035aeb14bf"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"7cfa5c00a76e0b4dce2d99f2de93a0368f98d455","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"c0a59986_8df08454","updated":"2023-03-08 23:04:01.000000000","message":"we just moved all-projects to CopyConditions and nothing has exploded.  I will watch this, but I think we\u0027re gtg with it.","commit_id":"6349f18d405642ec7cb113235b2b37035aeb14bf"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"e8ebdfa392bdda939025dccef39d0b7bd38ad4cf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"06ea0169_383bb28a","in_reply_to":"ca1d71a0_3c8b06b5","updated":"2023-02-08 08:45:54.000000000","message":"I think so too.","commit_id":"6349f18d405642ec7cb113235b2b37035aeb14bf"}],"gerrit/acls/openstack/automaton.config":[{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"48161196cd1f51ad3f7adaafde5907e204f56be0","unresolved":false,"context_lines":[{"line_number":14,"context_line":"label-Workflow \u003d -1..+1 group stable-maint-core"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"[label \"Backport-Candidate\"]"},{"line_number":17,"context_line":"copyCondition \u003d changekind:NO_CODE_CHANGE OR changekind:TRIVIAL_REBASE OR is:MAX OR is:MIN"},{"line_number":18,"context_line":"defaultValue \u003d 0"},{"line_number":19,"context_line":"function \u003d NoBlock"},{"line_number":20,"context_line":"value \u003d -2 Do Not Backport"}],"source_content_type":"text/x-ini","patch_set":2,"id":"0c89e03c_77a83d3a","line":17,"updated":"2022-12-16 15:10:58.000000000","message":"Are you sure the first two conditions should be ORed with the last two? Wouldn\u0027t AND be the former behavior? (Copy the score if there\u0027s no code change or it\u0027s a trivial rebase, but only when the vote has the min or max score not other values)?","commit_id":"6349f18d405642ec7cb113235b2b37035aeb14bf"},{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"a9abd7c11e0c4b70a3749255b739a33ba990bdb9","unresolved":false,"context_lines":[{"line_number":14,"context_line":"label-Workflow \u003d -1..+1 group stable-maint-core"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"[label \"Backport-Candidate\"]"},{"line_number":17,"context_line":"copyCondition \u003d changekind:NO_CODE_CHANGE OR changekind:TRIVIAL_REBASE OR is:MAX OR is:MIN"},{"line_number":18,"context_line":"defaultValue \u003d 0"},{"line_number":19,"context_line":"function \u003d NoBlock"},{"line_number":20,"context_line":"value \u003d -2 Do Not Backport"}],"source_content_type":"text/x-ini","patch_set":2,"id":"6022759a_33a9f66b","line":17,"updated":"2022-12-16 23:16:55.000000000","message":"Oh, you\u0027re right. The intent was to copy the min and max values even if it was neither a trivial rebase nor a commit message edit, so your translation does seem to be correct after all. Thanks!","commit_id":"6349f18d405642ec7cb113235b2b37035aeb14bf"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"916b658d90d388a1fff06f051901dfb0b2d28581","unresolved":true,"context_lines":[{"line_number":14,"context_line":"label-Workflow \u003d -1..+1 group stable-maint-core"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"[label \"Backport-Candidate\"]"},{"line_number":17,"context_line":"copyCondition \u003d changekind:NO_CODE_CHANGE OR changekind:TRIVIAL_REBASE OR is:MAX OR is:MIN"},{"line_number":18,"context_line":"defaultValue \u003d 0"},{"line_number":19,"context_line":"function \u003d NoBlock"},{"line_number":20,"context_line":"value \u003d -2 Do Not Backport"}],"source_content_type":"text/x-ini","patch_set":2,"id":"37e358a7_4c2088ff","line":17,"in_reply_to":"0c89e03c_77a83d3a","updated":"2022-12-16 20:41:51.000000000","message":"I am not 100% certain, but afaik these previous commands all applied individually?  Old Documentation seems to suggest the rules are all applied individually? http://gerrit-documentation.storage.googleapis.com/Documentation/2.11/config-labels.html#label_copyAllScoresIfNoCodeChange\n\nSo a +1/-1 would survive no change/trivial rebase, but not a \"regular\" upload.  However, +2/-2 survive anything.  \n\nmaybe that\u0027s not exactly as intended ... in that case the copycondition here is much clearer in this respect, so we can fiddle with it.","commit_id":"6349f18d405642ec7cb113235b2b37035aeb14bf"}]}