)]}'
{"id":"openstack%2Fproject-config~573315","triplet_id":"openstack%2Fproject-config~master~Ibd012c30ca4fdb71d760eedd822b23f395e597e9","project":"openstack/project-config","branch":"master","hashtags":[],"change_id":"Ibd012c30ca4fdb71d760eedd822b23f395e597e9","subject":"(airship) Disable verify permission for CI","status":"ABANDONED","created":"2018-06-07 16:33:11.000000000","updated":"2018-06-07 17:48:59.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"27b7fae27f248bf0311deec4f797d95477147236","_number":573315,"virtual_id_number":573315,"owner":{"_account_id":26449,"name":"Scott Hussey","email":"sthussey@att.com","username":"sh8121"},"actions":{},"labels":{"Verified":{"all":[{"date":"2018-06-07 17:35:40.000000000","_account_id":26449,"name":"Scott Hussey","email":"sthussey@att.com","username":"sh8121"},{"date":"2018-06-07 16:50:42.000000000","_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},{"date":"2018-06-07 16:34:11.000000000","_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},{"_account_id":26365,"name":"Bryan Strassner","email":"strassner.bryan@gmail.com","username":"bryan-strassner"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":26449,"name":"Scott Hussey","email":"sthussey@att.com","username":"sh8121"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},{"value":0,"permitted_voting_range":{"min":-2,"max":2},"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},{"value":2,"date":"2018-06-07 16:52:18.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},{"value":1,"date":"2018-06-07 16:34:27.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":26365,"name":"Bryan Strassner","email":"strassner.bryan@gmail.com","username":"bryan-strassner"}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","default_value":0,"optional":true},"Workflow":{"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":0},"_account_id":26449,"name":"Scott Hussey","email":"sthussey@att.com","username":"sh8121"},{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},{"_account_id":26365,"name":"Bryan Strassner","email":"strassner.bryan@gmail.com","username":"bryan-strassner"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},{"_account_id":26365,"name":"Bryan Strassner","email":"strassner.bryan@gmail.com","username":"bryan-strassner"},{"_account_id":26449,"name":"Scott Hussey","email":"sthussey@att.com","username":"sh8121"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2018-06-07 16:34:11.000000000","updated_by":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"reviewer":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"state":"REVIEWER"},{"updated":"2018-06-07 16:34:27.000000000","updated_by":{"_account_id":26365,"name":"Bryan Strassner","email":"strassner.bryan@gmail.com","username":"bryan-strassner"},"reviewer":{"_account_id":26365,"name":"Bryan Strassner","email":"strassner.bryan@gmail.com","username":"bryan-strassner"},"state":"REVIEWER"},{"updated":"2018-06-07 16:50:42.000000000","updated_by":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"reviewer":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"state":"REVIEWER"},{"updated":"2018-06-07 16:52:18.000000000","updated_by":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"reviewer":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"state":"REVIEWER"}],"messages":[{"id":"14625b03c3e63ca1cc851b585aedf8eab3c5594c","author":{"_account_id":26449,"name":"Scott Hussey","email":"sthussey@att.com","username":"sh8121"},"date":"2018-06-07 16:33:11.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"8d72e00b90f57a8e86592f8e15bd93131d0af182","author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"date":"2018-06-07 16:34:11.000000000","message":"Patch Set 1:\n\nCan you explain the motivation behind this change? Code-Review is expected to be review done by humans and this breaks that expectation.","accounts_in_message":[],"_revision_number":1},{"id":"23b754c36b84657e258c11990de7d9f0351f7228","author":{"_account_id":26365,"name":"Bryan Strassner","email":"strassner.bryan@gmail.com","username":"bryan-strassner"},"date":"2018-06-07 16:34:27.000000000","message":"Patch Set 1: Code-Review+1","accounts_in_message":[],"_revision_number":1},{"id":"b3e8daa88b20c57de89d0bcf42a6cac294d7849e","author":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"date":"2018-06-07 16:50:42.000000000","message":"Patch Set 1:\n\nI agree with clark - I\u0027d like to understand the motivation.\n\nThe overall model we tend to use is that Zuul has Verified +/- 2 (and is the only one that has that) that third-party CI has Verified +/- 1 (which indicates that a computer has done verification tasks, but that computer is informative and not binding)\n\nWhen we roll out gerrit 2.15 there is a specific new data structure that will be for 3rd-party CI\u0027s registering their votes so we can display them as an understandable matrix/table.\n\nAnyway - what are you looking to accomplish/solve? I\u0027d love to understand the issue more.","accounts_in_message":[],"_revision_number":1},{"id":"8428175d327ae1534bde8bc7cbe9d697d467f9c9","author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"date":"2018-06-07 16:52:18.000000000","message":"Patch Set 1: Code-Review+2","accounts_in_message":[],"_revision_number":1},{"id":"3d01ea16d8c1d2c84e50ec4d796e3af2de6824b4","author":{"_account_id":26449,"name":"Scott Hussey","email":"sthussey@att.com","username":"sh8121"},"date":"2018-06-07 17:33:52.000000000","message":"Patch Set 1:\n\n\u003e I agree with clark - I\u0027d like to understand the motivation.\n \u003e \n \u003e The overall model we tend to use is that Zuul has Verified +/- 2\n \u003e (and is the only one that has that) that third-party CI has\n \u003e Verified +/- 1 (which indicates that a computer has done\n \u003e verification tasks, but that computer is informative and not\n \u003e binding)\n \u003e \n \u003e When we roll out gerrit 2.15 there is a specific new data structure\n \u003e that will be for 3rd-party CI\u0027s registering their votes so we can\n \u003e display them as an understandable matrix/table.\n \u003e \n \u003e Anyway - what are you looking to accomplish/solve? I\u0027d love to\n \u003e understand the issue more.\n\nOur 3rd party gates are broken and I was directed by our CICD team it was our mistake to enable them to vote on the Verify label and block the merge of PS by Zuul. So this patchset is an attempt to correct that. I\u0027m new to project config stuff, so this may be the wrong avenue to correct this issue. The target state (for now, until our 3rd party gates are fixed) is to allow them to vote CR +/-1 so reviewers can see the results, but it doesn\u0027t block Zuul\u0027s gate pipeline.","accounts_in_message":[],"_revision_number":1},{"id":"09645eb1198f90f5dffdf434d5bdd9cdd0aa132c","author":{"_account_id":26449,"name":"Scott Hussey","email":"sthussey@att.com","username":"sh8121"},"date":"2018-06-07 17:35:40.000000000","message":"Patch Set 1:\n\n\u003e \u003e I agree with clark - I\u0027d like to understand the motivation.\n \u003e \u003e\n \u003e \u003e The overall model we tend to use is that Zuul has Verified +/- 2\n \u003e \u003e (and is the only one that has that) that third-party CI has\n \u003e \u003e Verified +/- 1 (which indicates that a computer has done\n \u003e \u003e verification tasks, but that computer is informative and not\n \u003e \u003e binding)\n \u003e \u003e\n \u003e \u003e When we roll out gerrit 2.15 there is a specific new data\n \u003e structure\n \u003e \u003e that will be for 3rd-party CI\u0027s registering their votes so we can\n \u003e \u003e display them as an understandable matrix/table.\n \u003e \u003e\n \u003e \u003e Anyway - what are you looking to accomplish/solve? I\u0027d love to\n \u003e \u003e understand the issue more.\n \u003e \n \u003e Our 3rd party gates are broken and I was directed by our CICD team\n \u003e it was our mistake to enable them to vote on the Verify label and\n \u003e block the merge of PS by Zuul. So this patchset is an attempt to\n \u003e correct that. I\u0027m new to project config stuff, so this may be the\n \u003e wrong avenue to correct this issue. The target state (for now,\n \u003e until our 3rd party gates are fixed) is to allow them to vote CR\n \u003e +/-1 so reviewers can see the results, but it doesn\u0027t block Zuul\u0027s\n \u003e gate pipeline.\n\nI should correct the above to say target state is allowing them to vote in some way that indicates results, but is not gating. Code-Review may not be the correct label for that.","accounts_in_message":[],"_revision_number":1},{"id":"27b7fae27f248bf0311deec4f797d95477147236","author":{"_account_id":26449,"name":"Scott Hussey","email":"sthussey@att.com","username":"sh8121"},"date":"2018-06-07 17:48:59.000000000","message":"Abandoned\n\nThanks to Clark\u0027s advice, I\u0027ll abandon this PS with the plan to remove our CICD robot account from airship-ci.","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"0fbcd29ffa7a16f0d20f52fe1f05dcb993e72e18","revisions":{"0fbcd29ffa7a16f0d20f52fe1f05dcb993e72e18":{"kind":"REWORK","_number":1,"created":"2018-06-07 16:33:11.000000000","uploader":{"_account_id":26449,"name":"Scott Hussey","email":"sthussey@att.com","username":"sh8121"},"ref":"refs/changes/15/573315/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/project-config","ref":"refs/changes/15/573315/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/project-config refs/changes/15/573315/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/project-config refs/changes/15/573315/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/project-config refs/changes/15/573315/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/project-config refs/changes/15/573315/1"}}},"commit":{"parents":[{"commit":"4fa910b574b1aa940b58154ffbbb65e3c5b57324","subject":"Merge \"Add zuul jobs for StarlingX\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/project-config/commit/4fa910b574b1aa940b58154ffbbb65e3c5b57324"}]}],"author":{"name":"Scott Hussey","email":"sh8121@att.com","date":"2018-06-07 16:30:32.000000000","tz":-300},"committer":{"name":"Scott Hussey","email":"sh8121@att.com","date":"2018-06-07 16:31:58.000000000","tz":-300},"subject":"(airship) Disable verify permission for CI","message":"(airship) Disable verify permission for CI\n\n- Disable the verify voting permissions for 3rd party\n  CI\n- Enable Code-Review +1,-1 voting for 3rd party\n  CI\n\nChange-Id: Ibd012c30ca4fdb71d760eedd822b23f395e597e9\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/project-config/commit/0fbcd29ffa7a16f0d20f52fe1f05dcb993e72e18"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/project-config/commit/0fbcd29ffa7a16f0d20f52fe1f05dcb993e72e18"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}