)]}'
{"id":"openstack%2Fproject-config~774300","triplet_id":"openstack%2Fproject-config~master~I3f259c3f7d346b37269eba1c57c864c653cf2103","project":"openstack/project-config","branch":"master","attention_set":{},"removed_from_attention_set":{"1":{"account":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"last_update":"2023-11-08 07:37:05.000000000","reason":"Change was abandoned"}},"hashtags":[],"change_id":"I3f259c3f7d346b37269eba1c57c864c653cf2103","subject":"Disallow fail-fast in Zuul","status":"ABANDONED","created":"2021-02-05 19:31:54.000000000","updated":"2023-11-08 07:37:05.000000000","total_comment_count":1,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"8202a9fe8550ce91333fb2cfe7ebef5cff17cd6a","_number":774300,"virtual_id_number":774300,"owner":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},{"tag":"autogenerated:zuul:check","value":1,"date":"2021-02-05 19:42:06.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","value":1,"default_value":0,"optional":true},"Code-Review":{"disliked":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"all":[{"value":-1,"date":"2021-02-10 16:19:43.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},{"value":-1,"date":"2021-02-05 20:08:13.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"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":"","value":-1,"default_value":0,"optional":true},"Workflow":{"all":[{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"CC":[{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2021-02-05 19:42:06.000000000","updated_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"reviewer":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2021-02-05 20:08:13.000000000","updated_by":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"reviewer":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"state":"REVIEWER"},{"updated":"2021-02-09 23:10:01.000000000","updated_by":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"reviewer":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"state":"CC"},{"updated":"2021-02-10 16:19:43.000000000","updated_by":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"reviewer":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"state":"REVIEWER"},{"updated":"2023-10-05 03:00:00.000000000","updated_by":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"reviewer":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"state":"CC"}],"messages":[{"id":"c4c1eaa57ec9d31ed118b5afcfe83ae2d6f0b690","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"date":"2021-02-05 19:31:54.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"5196824cbc1da0f170da93119b36b3feb1aacc48","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-02-05 19:42:06.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-zuul-jobs-linters https://zuul.opendev.org/t/openstack/build/ed2aecb24ff5477899a7f4412d8ea8d0 : SUCCESS in 5m 40s","accounts_in_message":[],"_revision_number":1},{"id":"59099fc52be9edbba6c01249a4b7423221259c92","author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"date":"2021-02-05 20:08:13.000000000","message":"Patch Set 1: Code-Review-1\n\nFYI, the Octavia project has had this enabled on the gate pipeline for two years.\nWe think there is value in having this option on the gate pipeline with very long running jobs.","accounts_in_message":[],"_revision_number":1},{"id":"d5f5c6bfc02369fe5a28722497b0e849d505eb49","author":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"date":"2021-02-07 00:18:53.000000000","message":"Patch Set 1:\n\nI think there\u0027s a good argument to be made that enabling it in gate with clean-check is advantageous.  Perhaps even in gate without clean-check.\n\nIt would be nice if someone were to establish some parameters to optimize and perform a controlled experiment.  I\u0027m not in a position to do that, but it should be possible.","accounts_in_message":[],"_revision_number":1},{"id":"80cb4fbb390d507b0a224b8f5eeb1f56eff3926b","author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"date":"2021-02-09 23:10:01.000000000","message":"Patch Set 1:\n\nI think the counterargument to having it set for gating, even with clean-check, is that people working on the change won\u0027t know if it fails more than one job, only the first one it failed. While clean-check does at least require that the change passed all its configured jobs when it was submitted, that result can still be quite old (I\u0027m rather often reviewing changes with year+ old check results, embarrassing as that is to say). If reviewers are proactively rechecking older changes before approving them, then maybe it\u0027s mitigated? That might come down to reminding people to get in that habit though...","accounts_in_message":[],"_revision_number":1},{"id":"fce8e1ddbdf4b959c4671ac3dc428e4855036e77","author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"date":"2021-02-09 23:11:55.000000000","message":"Patch Set 1:\n\nI\u0027ll try to summarize for the ML tomorrow.","accounts_in_message":[],"_revision_number":1},{"id":"208754f144d637a6552b292b05e67f927e26ffb6","author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"date":"2021-02-10 16:19:43.000000000","message":"Patch Set 1: Code-Review-1\n\nAdding to Michael\u0027s comment, I also have this enabled in the ovn-octavia-provider gate pipeline and think it\u0027s useful, only remember it triggering on things like mirror failures, in which case we gave a couple of CPU hours back to the pool.","accounts_in_message":[],"_revision_number":1},{"id":"a2bce9f8c8568f4aa5fa9a034f08bcfb910a253f","author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"date":"2023-10-05 03:00:00.000000000","message":"Patch Set 1:\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"8202a9fe8550ce91333fb2cfe7ebef5cff17cd6a","tag":"autogenerated:gerrit:abandon","author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"date":"2023-11-08 07:37:05.000000000","message":"Abandoned","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"feef758e31e7ebad7d476c8a6158c0f01066629a","revisions":{"feef758e31e7ebad7d476c8a6158c0f01066629a":{"kind":"REWORK","_number":1,"created":"2021-02-05 19:31:54.000000000","uploader":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"ref":"refs/changes/00/774300/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/project-config","ref":"refs/changes/00/774300/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/project-config refs/changes/00/774300/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/project-config refs/changes/00/774300/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/project-config refs/changes/00/774300/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/project-config refs/changes/00/774300/1"}}},"commit":{"parents":[{"commit":"e707d93f2b38b5c1b0645fdc2f9e62951f6ded52","subject":"Merge \"zuul-worker: remove additional install of apt-transport-https\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/project-config/commit/e707d93f2b38b5c1b0645fdc2f9e62951f6ded52"}]}],"author":{"name":"James E. Blair","email":"jim@acmegating.com","date":"2021-02-05 19:29:10.000000000","tz":-480},"committer":{"name":"James E. Blair","email":"jim@acmegating.com","date":"2021-02-05 19:29:45.000000000","tz":-480},"subject":"Disallow fail-fast in Zuul","message":"Disallow fail-fast in Zuul\n\nTo avoid unecessary rechecks, we decided some time ago to disallow\nthe fail-fast feature.  This way developers receive information about\nall failures simultaneously rather than having to work through one\nfailing job at a time.\n\nIt appears we neglected to actually implement the control; this\nchange does so.\n\nChange-Id: I3f259c3f7d346b37269eba1c57c864c653cf2103\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/project-config/commit/feef758e31e7ebad7d476c8a6158c0f01066629a"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/project-config/commit/feef758e31e7ebad7d476c8a6158c0f01066629a"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[{"name":"Verified","description":"Verified in gate by CI","status":"UNSATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Verified\u003dMAX AND -label:Verified\u003dMIN","fulfilled":false,"status":"FAIL","passing_atoms":[],"failing_atoms":["label:Verified\u003dMAX","label:Verified\u003dMIN"],"atom_explanations":{}}},{"name":"Code-Review","description":"Code reviewed by core reviewer","status":"UNSATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Code-Review\u003dMAX AND -label:Code-Review\u003dMIN","fulfilled":false,"status":"FAIL","passing_atoms":[],"failing_atoms":["label:Code-Review\u003dMAX","label:Code-Review\u003dMIN"],"atom_explanations":{}}},{"name":"Workflow","description":"Approved for gate by core reviewer","status":"UNSATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Workflow\u003dMAX AND -label:Workflow\u003dMIN","fulfilled":false,"status":"FAIL","passing_atoms":[],"failing_atoms":["label:Workflow\u003dMAX","label:Workflow\u003dMIN"],"atom_explanations":{}}}]}
