)]}'
{"gerrit/acls/openstack/bifrost.config":[{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"db5f56545682d50d576c7779fd2d9cbcce9671d2","unresolved":true,"context_lines":[{"line_number":45,"context_line":"label-Workflow \u003d -1..+1 group ironic-stable-maint"},{"line_number":46,"context_line":"label-Workflow \u003d -1..+1 group stable-maint-core"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"[label \"Backport-Candidate\"]"},{"line_number":49,"context_line":"copyAllScoresIfNoCodeChange \u003d true"},{"line_number":50,"context_line":"copyAllScoresOnTrivialRebase \u003d true"},{"line_number":51,"context_line":"copyMaxScore \u003d true"}],"source_content_type":"text/x-ini","patch_set":1,"id":"dff64059_73fc1306","line":48,"updated":"2021-02-16 15:58:46.000000000","message":"How is it that every project has created their own \"definition\" label for Backport-Candidate? I\u0027d think there would be some consistency in using this label... :-( Is this the suggested way to do this?\n\neg, we don\u0027t have [label \"Code-Review\"], [label \"Workflow\"]...","commit_id":"07eaeff9ee36fbf00fa9b4fc6fd9d5158931ffc0"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"eb97ccd4dd89f080e6a239740c79b707b9be0245","unresolved":true,"context_lines":[{"line_number":45,"context_line":"label-Workflow \u003d -1..+1 group ironic-stable-maint"},{"line_number":46,"context_line":"label-Workflow \u003d -1..+1 group stable-maint-core"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"[label \"Backport-Candidate\"]"},{"line_number":49,"context_line":"copyAllScoresIfNoCodeChange \u003d true"},{"line_number":50,"context_line":"copyAllScoresOnTrivialRebase \u003d true"},{"line_number":51,"context_line":"copyMaxScore \u003d true"}],"source_content_type":"text/x-ini","patch_set":1,"id":"07f2ad4f_4232742e","line":48,"in_reply_to":"50874cce_13f9c4ef","updated":"2021-02-25 20:42:45.000000000","message":"Backport candidate labels don\u0027t make sense on all hosted projects (not even all OpenStack projects). For this reason we don\u0027t centrally define them like Code-Review, Verified, and Workflow.","commit_id":"07eaeff9ee36fbf00fa9b4fc6fd9d5158931ffc0"},{"author":{"_account_id":15519,"name":"Iury Gregory Melo Ferreira","display_name":"Iury Gregory","email":"iurygregory@gmail.com","username":"iurygregory"},"change_message_id":"7e68483b4b4ffb0a09b0c1b4c29f5d81b82866f4","unresolved":true,"context_lines":[{"line_number":45,"context_line":"label-Workflow \u003d -1..+1 group ironic-stable-maint"},{"line_number":46,"context_line":"label-Workflow \u003d -1..+1 group stable-maint-core"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"[label \"Backport-Candidate\"]"},{"line_number":49,"context_line":"copyAllScoresIfNoCodeChange \u003d true"},{"line_number":50,"context_line":"copyAllScoresOnTrivialRebase \u003d true"},{"line_number":51,"context_line":"copyMaxScore \u003d true"}],"source_content_type":"text/x-ini","patch_set":1,"id":"50874cce_13f9c4ef","line":48,"in_reply_to":"dff64059_73fc1306","updated":"2021-02-16 16:10:37.000000000","message":"I took the code from oslo\u0027s approach https://opendev.org/openstack/project-config/commit/f0dbc7d5a886b40db4d85ef9d1cce56d02d5cc04","commit_id":"07eaeff9ee36fbf00fa9b4fc6fd9d5158931ffc0"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"db5f56545682d50d576c7779fd2d9cbcce9671d2","unresolved":true,"context_lines":[{"line_number":54,"context_line":"function \u003d NoBlock"},{"line_number":55,"context_line":"value \u003d -1 Do Not Backport"},{"line_number":56,"context_line":"value \u003d 0 Backport Review Needed"},{"line_number":57,"context_line":"value \u003d +1 Should Backport"},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"[receive]"},{"line_number":60,"context_line":"requireChangeId \u003d true"}],"source_content_type":"text/x-ini","patch_set":1,"id":"45423e9f_5b202b72","line":57,"updated":"2021-02-16 15:58:46.000000000","message":"I randomly looked at another project. They use -2 \u0026 +2 too. Thoughts? It would be nice to be consistent across openstack projects. This is oslo.config: https://opendev.org/openstack/project-config/src/commit/88a309642f7fc1cbdfcc4cbcff25a2f1608c3e0a/gerrit/acls/openstack/oslo.config#L13-L24","commit_id":"07eaeff9ee36fbf00fa9b4fc6fd9d5158931ffc0"},{"author":{"_account_id":15519,"name":"Iury Gregory Melo Ferreira","display_name":"Iury Gregory","email":"iurygregory@gmail.com","username":"iurygregory"},"change_message_id":"9476792fab41f7831c3b309bade67d3e1de02c06","unresolved":true,"context_lines":[{"line_number":54,"context_line":"function \u003d NoBlock"},{"line_number":55,"context_line":"value \u003d -1 Do Not Backport"},{"line_number":56,"context_line":"value \u003d 0 Backport Review Needed"},{"line_number":57,"context_line":"value \u003d +1 Should Backport"},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"[receive]"},{"line_number":60,"context_line":"requireChangeId \u003d true"}],"source_content_type":"text/x-ini","patch_set":1,"id":"30c9d2ea_8418bcc5","line":57,"in_reply_to":"45423e9f_5b202b72","updated":"2021-02-16 16:09:22.000000000","message":"Hey Ruby, thank you for your feedback, I\u0027ve talked with oslo folks and according to them the +2/-2 is not worth, we can of course add +2/-2  like they did.\nWe can choose between cores or community to be able to give +1/-1 in the Backport Candidate Label, I\u0027ve pushed the patch with the restriction for only cores to see if that\u0027s the approach we want to take.","commit_id":"07eaeff9ee36fbf00fa9b4fc6fd9d5158931ffc0"}],"gerrit/acls/openstack/ironic-inspector.config":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"eb97ccd4dd89f080e6a239740c79b707b9be0245","unresolved":true,"context_lines":[{"line_number":2,"context_line":"abandon \u003d group ironic-inspector-core"},{"line_number":3,"context_line":"editHashtags \u003d group ironic-inspector-core"},{"line_number":4,"context_line":"label-Backport-Candidate \u003d -1..+1 group ironic-core"},{"line_number":5,"context_line":"label-Backport-Candidate \u003d -1..+1 ironic-inspector-core"},{"line_number":6,"context_line":"label-Code-Review \u003d -2..+2 group ironic-core"},{"line_number":7,"context_line":"label-Code-Review \u003d -2..+2 group ironic-inspector-core"},{"line_number":8,"context_line":"label-Workflow \u003d -1..+1 group ironic-core"}],"source_content_type":"text/x-ini","patch_set":1,"id":"827375c7_d3ce7d59","line":5,"range":{"start_line":5,"start_character":34,"end_line":5,"end_character":55},"updated":"2021-02-25 20:42:45.000000000","message":"This is missing its group prefix.","commit_id":"07eaeff9ee36fbf00fa9b4fc6fd9d5158931ffc0"}]}