)]}'
{"gerrit/acls/openstack/kayobe.config":[{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"828d1f1787d7b3692b619722dceb9ac1e41c2d54","unresolved":false,"context_lines":[{"line_number":20,"context_line":"[label \"Review-Priority\"]"},{"line_number":21,"context_line":"copyAllScoresIfNoCodeChange \u003d true"},{"line_number":22,"context_line":"copyAllScoresOnTrivialRebase \u003d true"},{"line_number":23,"context_line":"copyMaxScore \u003d true"},{"line_number":24,"context_line":"copyMinScore \u003d true"},{"line_number":25,"context_line":"defaultValue \u003d 0"},{"line_number":26,"context_line":"function \u003d AnyWithBlock"},{"line_number":27,"context_line":"value \u003d -1 Branch Freeze"}],"source_content_type":"text/x-ini","patch_set":2,"id":"ff570b3c_89c31738","line":24,"range":{"start_line":23,"start_character":0,"end_line":24,"end_character":19},"updated":"2020-06-04 08:07:46.000000000","message":"Should we add these, such that -2 and +2 aren\u0027t removed on patch updates? Might even make sense to keep -1 and +1, if that\u0027s possible. That would be copyAnyScore.\n\nPossibly we also want allowPostSubmit\n\nReference: https://gerrit-review.googlesource.com/Documentation/config-labels.html","commit_id":"f5f15a00b83fed4af7769299856fb36302b6d2f9"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"b583a382cc39ac26ce15808486a1a8f2ee8ed9cc","unresolved":false,"context_lines":[{"line_number":20,"context_line":"[label \"Review-Priority\"]"},{"line_number":21,"context_line":"copyAllScoresIfNoCodeChange \u003d true"},{"line_number":22,"context_line":"copyAllScoresOnTrivialRebase \u003d true"},{"line_number":23,"context_line":"copyMaxScore \u003d true"},{"line_number":24,"context_line":"copyMinScore \u003d true"},{"line_number":25,"context_line":"defaultValue \u003d 0"},{"line_number":26,"context_line":"function \u003d AnyWithBlock"},{"line_number":27,"context_line":"value \u003d -1 Branch Freeze"}],"source_content_type":"text/x-ini","patch_set":2,"id":"ff570b3c_74e49a3c","line":24,"range":{"start_line":23,"start_character":0,"end_line":24,"end_character":19},"in_reply_to":"ff570b3c_89c31738","updated":"2020-06-04 08:43:51.000000000","message":"All good suggestions, thanks Mark. I pushed an updated change.","commit_id":"f5f15a00b83fed4af7769299856fb36302b6d2f9"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"cdaefd1f7986ad14d0200c206dab525197615c5e","unresolved":false,"context_lines":[{"line_number":7,"context_line":"label-Workflow \u003d -1..+1 group kayobe-core"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"[label \"Backport-Candidate\"]"},{"line_number":10,"context_line":"allowPostSubmit \u003d true"},{"line_number":11,"context_line":"copyAnyScore \u003d true"},{"line_number":12,"context_line":"defaultValue \u003d 0"},{"line_number":13,"context_line":"function \u003d NoBlock"}],"source_content_type":"text/x-ini","patch_set":4,"id":"ff570b3c_71bc9eba","line":10,"range":{"start_line":10,"start_character":0,"end_line":10,"end_character":15},"updated":"2020-06-04 18:57:01.000000000","message":"I don\u0027t think this is supported by our current version of gerrit: https://review.opendev.org/Documentation/config-labels.html. It should be removed to avoid configuration problems.\n\nNote it also default set to true in newer Gerrit so you\u0027ll get this for free once we\u0027ve upgraded.","commit_id":"e49094fdebe0085409de4ed78edfd0420d360c33"}],"gerrit/acls/openstack/kolla-ansible.config":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"cdaefd1f7986ad14d0200c206dab525197615c5e","unresolved":false,"context_lines":[{"line_number":7,"context_line":"label-Workflow \u003d -1..+1 group kolla-ansible-core"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"[label \"Backport-Candidate\"]"},{"line_number":10,"context_line":"allowPostSubmit \u003d true"},{"line_number":11,"context_line":"copyAnyScore \u003d true"},{"line_number":12,"context_line":"defaultValue \u003d 0"},{"line_number":13,"context_line":"function \u003d NoBlock"}],"source_content_type":"text/x-ini","patch_set":4,"id":"ff570b3c_b1b2b6e3","line":10,"range":{"start_line":10,"start_character":0,"end_line":10,"end_character":15},"updated":"2020-06-04 18:57:01.000000000","message":"I don\u0027t think this is supported by our current version of gerrit: https://review.opendev.org/Documentation/config-labels.html. It should be removed to avoid configuration problems.\n\nNote it also default set to true in newer Gerrit so you\u0027ll get this for free once we\u0027ve upgraded.","commit_id":"e49094fdebe0085409de4ed78edfd0420d360c33"}],"gerrit/acls/openstack/kolla.config":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"cdaefd1f7986ad14d0200c206dab525197615c5e","unresolved":false,"context_lines":[{"line_number":7,"context_line":"label-Workflow \u003d -1..+1 group kolla-core"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"[label \"Backport-Candidate\"]"},{"line_number":10,"context_line":"allowPostSubmit \u003d true"},{"line_number":11,"context_line":"copyAnyScore \u003d true"},{"line_number":12,"context_line":"defaultValue \u003d 0"},{"line_number":13,"context_line":"function \u003d NoBlock"}],"source_content_type":"text/x-ini","patch_set":4,"id":"ff570b3c_11cd6265","line":10,"range":{"start_line":10,"start_character":0,"end_line":10,"end_character":15},"updated":"2020-06-04 18:57:01.000000000","message":"I don\u0027t think this is supported by our current version of gerrit: https://review.opendev.org/Documentation/config-labels.html. It should be removed to avoid configuration problems.\n\nNote it also default set to true in newer Gerrit so you\u0027ll get this for free once we\u0027ve upgraded.","commit_id":"e49094fdebe0085409de4ed78edfd0420d360c33"}],"tools/normalize_acl.py":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"cdaefd1f7986ad14d0200c206dab525197615c5e","unresolved":false,"context_lines":[{"line_number":58,"context_line":""},{"line_number":59,"context_line":"valid_keys \u003d {\u0027abandon\u0027,"},{"line_number":60,"context_line":"              \u0027access\u0027,"},{"line_number":61,"context_line":"              \u0027allowPostSubmit\u0027,"},{"line_number":62,"context_line":"              \u0027copyAllScoresIfNoCodeChange\u0027,"},{"line_number":63,"context_line":"              \u0027copyAllScoresOnTrivialRebase\u0027,"},{"line_number":64,"context_line":"              \u0027copyAnyScore\u0027,"}],"source_content_type":"text/x-python","patch_set":4,"id":"ff570b3c_f1c70e44","line":61,"range":{"start_line":61,"start_character":15,"end_line":61,"end_character":30},"updated":"2020-06-04 18:57:01.000000000","message":"I don\u0027t think this is supported by our current version of gerrit: https://review.opendev.org/Documentation/config-labels.html. It should be removed to avoid configuration problems.\n\nNote it also default set to true in newer Gerrit so you\u0027ll get this for free once we\u0027ve upgraded.","commit_id":"e49094fdebe0085409de4ed78edfd0420d360c33"}]}