)]}'
{"id":"openstack%2Fproject-config~749986","triplet_id":"openstack%2Fproject-config~master~Ib99bf4207c8c313b33bf161adbeea07f6f7d6cb1","project":"openstack/project-config","branch":"master","topic":"persist-octavia-backport-candidate","hashtags":[],"change_id":"Ib99bf4207c8c313b33bf161adbeea07f6f7d6cb1","subject":"Make the Backport-Candidate field in Octavia reviews persist","status":"ABANDONED","created":"2020-09-04 15:19:12.000000000","updated":"2020-09-04 17:42:27.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"e705ee3b33a8e29b7e49f683db4064ab9ea9ea43","_number":749986,"virtual_id_number":749986,"owner":{"_account_id":13995,"name":"Nate Johnston","email":"nate.johnston@redhat.com","username":"natejohnston"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},{"_account_id":13995,"name":"Nate Johnston","email":"nate.johnston@redhat.com","username":"natejohnston"},{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},{"tag":"autogenerated:zuul:check","value":1,"date":"2020-09-04 16:43:10.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"}],"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":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"all":[{"value":-1,"date":"2020-09-04 16:34:50.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":13995,"name":"Nate Johnston","email":"nate.johnston@redhat.com","username":"natejohnston"},{"value":-1,"date":"2020-09-04 16:55:05.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"]},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"}],"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":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},{"value":0,"permitted_voting_range":{"min":-1,"max":0},"_account_id":13995,"name":"Nate Johnston","email":"nate.johnston@redhat.com","username":"natejohnston"},{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},{"_account_id":13995,"name":"Nate Johnston","email":"nate.johnston@redhat.com","username":"natejohnston"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2020-09-04 15:19:36.000000000","updated_by":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"reviewer":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"state":"REVIEWER"},{"updated":"2020-09-04 16:29:32.000000000","updated_by":{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},"reviewer":{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},"state":"REVIEWER"},{"updated":"2020-09-04 16:43:10.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":"2020-09-04 16:55:05.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"}],"messages":[{"id":"9ff51a81cec5ca840f002077e5fc7ba454eda58a","author":{"_account_id":13995,"name":"Nate Johnston","email":"nate.johnston@redhat.com","username":"natejohnston"},"date":"2020-09-04 15:19:12.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"f4ddf31878855bc4f39fe93c5f72817ccdd9789d","author":{"_account_id":13995,"name":"Nate Johnston","email":"nate.johnston@redhat.com","username":"natejohnston"},"date":"2020-09-04 15:22:13.000000000","message":"Patch Set 1:\n\nThis came up in a discussion I was having with gthiemonge, so I thought I would put it out there and see if people like the idea.","accounts_in_message":[],"_revision_number":1},{"id":"8bebb9071cee9f63127af0e850d5682342e85a33","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-09-04 15:33:03.000000000","message":"Patch Set 1: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttps://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing\n\n\n- openstack-zuul-jobs-linters https://zuul.opendev.org/t/openstack/build/0891866cfced4788816aded3c3ae05fe : SUCCESS in 8m 25s\n- project-config-gerrit https://zuul.opendev.org/t/openstack/build/f2c9da77781444879d9902b24cdfa72a : FAILURE in 5m 05s","accounts_in_message":[],"_revision_number":1},{"id":"d2fb1da8948890cc1cfe2c99573a5d696640af57","author":{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},"date":"2020-09-04 16:29:32.000000000","message":"Patch Set 1: Code-Review-1\n\nCode in latest patch sets may change to the point it is no longer backportable or otherwise. Approver (last core reviewer) should be able to discern wether change is backportable or not.","accounts_in_message":[],"_revision_number":1},{"id":"d1ad711e30d84da57d29a33f454640625b34f209","author":{"_account_id":13995,"name":"Nate Johnston","email":"nate.johnston@redhat.com","username":"natejohnston"},"date":"2020-09-04 16:34:31.000000000","message":"Patch Set 1:\n\n\u003e Code in latest patch sets may change to the point it is no longer\n \u003e backportable or otherwise. Approver (last core reviewer) should be\n \u003e able to discern wether change is backportable or not.\n\nTrue, but the reviewer can make that judgement regardless of whether the backport candidate field had been set by the author or a previous reviewer.  I also think that may be an edge case, whereas having to reset the flag for every PS is a drag on everyone all the time.","accounts_in_message":[],"_revision_number":1},{"id":"c00fc84f3766e979fc5d884a2bdbce8aaaab46a2","author":{"_account_id":13995,"name":"Nate Johnston","email":"nate.johnston@redhat.com","username":"natejohnston"},"date":"2020-09-04 16:34:50.000000000","message":"Patch Set 2: Patch Set 1 was rebased","accounts_in_message":[],"_revision_number":2},{"id":"6f6c2b222b95d4b510a32cd289b7938c1be180f0","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-09-04 16:43:10.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-zuul-jobs-linters https://zuul.opendev.org/t/openstack/build/d7162bf05bf649179fd4ca0755a386a2 : SUCCESS in 6m 42s\n- project-config-gerrit https://zuul.opendev.org/t/openstack/build/0bc262a3212f45d5beb217608845862a : SUCCESS in 4m 00s","accounts_in_message":[],"_revision_number":2},{"id":"1ec74ca79070d9dd1b0825426ca2e87f6b57ce5b","author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"date":"2020-09-04 16:55:05.000000000","message":"Patch Set 2: Code-Review-1\n\nThis was an intentional decision the Octavia community made when I added the backport candidate field[1]. I think we should again discuss this at one of the Octavia weekly meetings if we want to change the behavior from what was previously decided.\n\n[1] https://review.opendev.org/#/c/657657/","accounts_in_message":[],"_revision_number":2},{"id":"3a56ff836ab8441f0d674ffa107349aba4868044","author":{"_account_id":13995,"name":"Nate Johnston","email":"nate.johnston@redhat.com","username":"natejohnston"},"date":"2020-09-04 17:42:27.000000000","message":"Abandoned\n\nIt turns out that it is not possible with the current version of gerrit.","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"529251f4c4de1ad9cbd5269ea442b25fc009f709","revisions":{"c8698c7cb2ae458b17c054a0afefe47a72156e64":{"kind":"REWORK","_number":1,"created":"2020-09-04 15:19:12.000000000","uploader":{"_account_id":13995,"name":"Nate Johnston","email":"nate.johnston@redhat.com","username":"natejohnston"},"ref":"refs/changes/86/749986/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/project-config","ref":"refs/changes/86/749986/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/project-config refs/changes/86/749986/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/project-config refs/changes/86/749986/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/project-config refs/changes/86/749986/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/project-config refs/changes/86/749986/1"}}},"commit":{"parents":[{"commit":"56d77522bf17165fa32e047b7c46d550fd498e2b","subject":"Merge \"Final step for networking-l2gw and networking-l2gw-tempest-plugin retirement\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/project-config/commit/56d77522bf17165fa32e047b7c46d550fd498e2b"}]}],"author":{"name":"Nate Johnston","email":"nate.johnston@redhat.com","date":"2020-09-04 15:16:55.000000000","tz":-240},"committer":{"name":"Nate Johnston","email":"nate.johnston@redhat.com","date":"2020-09-04 15:19:10.000000000","tz":-240},"subject":"Make the Backport-Candidate field in Octavia reviews persist","message":"Make the Backport-Candidate field in Octavia reviews persist\n\nThe status of backport candidate does not change across patchsets,\nso make it persistent for Octavia projects.\n\nChange-Id: Ib99bf4207c8c313b33bf161adbeea07f6f7d6cb1\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/project-config/commit/c8698c7cb2ae458b17c054a0afefe47a72156e64"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/project-config/commit/c8698c7cb2ae458b17c054a0afefe47a72156e64"}]},"branch":"refs/heads/master"},"529251f4c4de1ad9cbd5269ea442b25fc009f709":{"kind":"TRIVIAL_REBASE","_number":2,"created":"2020-09-04 16:34:50.000000000","uploader":{"_account_id":13995,"name":"Nate Johnston","email":"nate.johnston@redhat.com","username":"natejohnston"},"ref":"refs/changes/86/749986/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/project-config","ref":"refs/changes/86/749986/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/project-config refs/changes/86/749986/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/project-config refs/changes/86/749986/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/project-config refs/changes/86/749986/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/project-config refs/changes/86/749986/2"}}},"commit":{"parents":[{"commit":"06ef650337bb800d3dcd777728e6caab927e8061","subject":"Allow copyAnyScore in gerrit ACLs","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/project-config/commit/06ef650337bb800d3dcd777728e6caab927e8061"}]}],"author":{"name":"Nate Johnston","email":"nate.johnston@redhat.com","date":"2020-09-04 15:16:55.000000000","tz":-240},"committer":{"name":"Nate Johnston","email":"nate.johnston@redhat.com","date":"2020-09-04 16:34:50.000000000","tz":0},"subject":"Make the Backport-Candidate field in Octavia reviews persist","message":"Make the Backport-Candidate field in Octavia reviews persist\n\nThe status of backport candidate does not change across patchsets,\nso make it persistent for Octavia projects.\n\nChange-Id: Ib99bf4207c8c313b33bf161adbeea07f6f7d6cb1\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/project-config/commit/529251f4c4de1ad9cbd5269ea442b25fc009f709"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/project-config/commit/529251f4c4de1ad9cbd5269ea442b25fc009f709"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}
