)]}'
{"id":"openstack%2Foctavia~798255","triplet_id":"openstack%2Foctavia~stable%2Fwallaby~I55136f570db33cfd22e919019c18e9028a6a9bf7","project":"openstack/octavia","branch":"stable/wallaby","hashtags":[],"change_id":"I55136f570db33cfd22e919019c18e9028a6a9bf7","subject":"Fix race conditions between API and worker DB calls","status":"MERGED","created":"2021-06-28 07:13:51.000000000","updated":"2021-06-30 16:32:10.000000000","submitted":"2021-06-30 16:29:43.000000000","submitter":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"submission_id":"798255","meta_rev_id":"cd727d93cbde4dc0ef05e939818c9bc29f914cf3","_number":798255,"virtual_id_number":798255,"owner":{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"value":0,"_account_id":7249,"name":"Ann Taraday","email":"akamyshnikova@mirantis.com","username":"AKamyshnikova"},{"value":0,"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},{"tag":"autogenerated:zuul:gate","value":2,"date":"2021-06-30 16:29:43.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":"","default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":7249,"name":"Ann Taraday","email":"akamyshnikova@mirantis.com","username":"AKamyshnikova"},"all":[{"value":2,"date":"2021-06-30 14:35:24.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":7249,"name":"Ann Taraday","email":"akamyshnikova@mirantis.com","username":"AKamyshnikova"},{"value":2,"date":"2021-06-29 08:27:31.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},{"value":0,"_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":"","default_value":0,"optional":true},"Workflow":{"approved":{"_account_id":7249,"name":"Ann Taraday","email":"akamyshnikova@mirantis.com","username":"AKamyshnikova"},"all":[{"value":1,"date":"2021-06-30 14:35:24.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":7249,"name":"Ann Taraday","email":"akamyshnikova@mirantis.com","username":"AKamyshnikova"},{"value":0,"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},{"value":0,"_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},"Backport-Candidate":{"all":[{"value":0,"_account_id":7249,"name":"Ann Taraday","email":"akamyshnikova@mirantis.com","username":"AKamyshnikova"},{"value":0,"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-2":"Do Not Backport","-1":"Not A Backport Candidate"," 0":"Backport Review Needed","+1":"Proposed Backport","+2":"Should Backport"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":7249,"name":"Ann Taraday","email":"akamyshnikova@mirantis.com","username":"AKamyshnikova"},{"_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":"2021-06-28 07:13:51.000000000","updated_by":{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},"reviewer":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"state":"REVIEWER"},{"updated":"2021-06-28 09:15:37.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-06-30 14:35:24.000000000","updated_by":{"_account_id":7249,"name":"Ann Taraday","email":"akamyshnikova@mirantis.com","username":"AKamyshnikova"},"reviewer":{"_account_id":7249,"name":"Ann Taraday","email":"akamyshnikova@mirantis.com","username":"AKamyshnikova"},"state":"REVIEWER"}],"messages":[{"id":"6388c1e3071bb4c65722812bbd863fca83df83f3","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},"date":"2021-06-28 07:13:51.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"fcdfc97e5a01ad8557d0dd684bc90c515a0dbc0e","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},"date":"2021-06-28 07:14:57.000000000","message":"Uploaded patch set 2: Commit message was updated.","accounts_in_message":[],"_revision_number":2},{"id":"230ae6db343e03b31a1a3d333074f9b00223e307","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-06-28 09:15:37.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/b175c39d22b54f5ba9a3ed2b47d37930 : SUCCESS in 11m 56s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/f791e81e598a4f7c81dff91165973e82 : SUCCESS in 10m 47s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/b5684ffad1f34c75839e2e7c50a8eee6 : SUCCESS in 8m 52s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/5ae797b907154d13aa44a8569aad8b42 : SUCCESS in 9m 26s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/c0155730b7fe443abf85232b82f15adb : SUCCESS in 8m 29s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/5a984b5784d94030bcee2d03b9038cfe : SUCCESS in 8m 14s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/4233fb3914fa427ab1ae52f047d96e4e : SUCCESS in 16m 26s\n- octavia-tox-py37-tips https://zuul.opendev.org/t/openstack/build/0a390f5b1f334923bf2c687d8ddd673a : SUCCESS in 9m 33s\n- octavia-tox-functional-py37-tips https://zuul.opendev.org/t/openstack/build/e2019630baa743e896c27e95b38b0fe0 : SUCCESS in 11m 24s\n- openstack-tox-pip-check-reqs https://zuul.opendev.org/t/openstack/build/8e3af891782841059f312bfc0308ff35 : SUCCESS in 7m 58s\n- openstack-tox-functional-py36 https://zuul.opendev.org/t/openstack/build/99162a188eb345c9b75eaab182792aa7 : SUCCESS in 10m 05s\n- octavia-v2-dsvm-noop-api https://zuul.opendev.org/t/openstack/build/e3ae1c0c9303401cb44ee4bdcf972ac0 : SUCCESS in 1h 34m 58s\n- octavia-v2-dsvm-scenario https://zuul.opendev.org/t/openstack/build/c17a700f12ec407b81102cf4d7291c2a : SUCCESS in 1h 20m 26s\n- octavia-v2-dsvm-tls-barbican https://zuul.opendev.org/t/openstack/build/800e8791920d4dd5ba0aca991349abd4 : SUCCESS in 40m 27s\n- octavia-v2-dsvm-spare-pool https://zuul.opendev.org/t/openstack/build/6791e91985d949ffb1b4acedcc32e631 : SUCCESS in 45m 04s\n- octavia-v2-act-stdby-dsvm-scenario https://zuul.opendev.org/t/openstack/build/467d90c1021e4186b5592200d71d9831 : SUCCESS in 39m 22s\n- octavia-grenade https://zuul.opendev.org/t/openstack/build/27be857736ba40a7bb6602a5f265d5fd : SUCCESS in 57m 11s\n- octavia-v2-dsvm-cinder-amphora https://zuul.opendev.org/t/openstack/build/7b3ae89148e04ef98a032e25d031eb05 : SUCCESS in 1h 36m 27s (non-voting)\n- octavia-v2-dsvm-scenario-two-node https://zuul.opendev.org/t/openstack/build/35b37445b55a4c159ccfc241d0091bff : FAILURE in 46m 21s (non-voting)\n- octavia-v2-dsvm-scenario-ipv6-only https://zuul.opendev.org/t/openstack/build/1eefe0079de34d85bf2bf8b89dd9b529 : SUCCESS in 1h 53m 37s (non-voting)\n- octavia-v2-dsvm-scenario-amphora-v2 https://zuul.opendev.org/t/openstack/build/df11797828fc4dc3bea15228b44a16a7 : SUCCESS in 1h 59m 58s (non-voting)\n- octavia-v2-dsvm-scenario-amphora-v2-no-jobboard https://zuul.opendev.org/t/openstack/build/6d73365ee2864ea88f21c162b884cbf6 : SUCCESS in 1h 40m 01s (non-voting)\n- octavia-v2-dsvm-scenario-centos-8 https://zuul.opendev.org/t/openstack/build/990ed7b536b74ea69846c722b7ac9f40 : FAILURE in 10m 10s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"34b4c5d57dbd4b89a2aa248b78e35d850499fbec","author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"date":"2021-06-29 08:27:31.000000000","message":"Patch Set 2: Code-Review+2","accounts_in_message":[],"_revision_number":2},{"id":"7b5426535a716462619434d5785096ced5194246","author":{"_account_id":7249,"name":"Ann Taraday","email":"akamyshnikova@mirantis.com","username":"AKamyshnikova"},"date":"2021-06-30 14:35:24.000000000","message":"Patch Set 2: Code-Review+2 Workflow+1","accounts_in_message":[],"_revision_number":2},{"id":"5abe9af50d619f474f1476856fe08380aaee3457","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-06-30 14:35:45.000000000","message":"Patch Set 2: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":2},{"id":"2360796220d643f58f034f9c504a67e584c5de11","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-06-30 16:29:43.000000000","message":"Patch Set 2: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/fc737838922749cfa673126a607c5d01 : SUCCESS in 9m 55s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/9bb64a5dd5da4577b5ab54bcb5ce5e9c : SUCCESS in 7m 00s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/fe210963c6224080a3aa874c42c80778 : SUCCESS in 8m 04s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/624f6fa3031147669d837ca3a72f9876 : SUCCESS in 10m 23s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/eaca3401f4ac450f87f9f10995ebaff2 : SUCCESS in 12m 58s\n- openstack-tox-pip-check-reqs https://zuul.opendev.org/t/openstack/build/2e584d7e9a914138964c29b4e38be3fe : SUCCESS in 7m 54s\n- openstack-tox-functional-py36 https://zuul.opendev.org/t/openstack/build/1da598b6e7384c4da089d1a6e2e1ab4e : SUCCESS in 9m 46s\n- octavia-v2-dsvm-noop-api https://zuul.opendev.org/t/openstack/build/442e1ae00aaa411cb88aa1adf6726329 : SUCCESS in 1h 37m 33s\n- octavia-v2-dsvm-scenario https://zuul.opendev.org/t/openstack/build/6b99cf7f44ed46068d1e99772843127c : SUCCESS in 1h 52m 41s\n- octavia-v2-dsvm-tls-barbican https://zuul.opendev.org/t/openstack/build/a71b88233eb94f9aac716b5cd75ed0fa : SUCCESS in 43m 39s\n- octavia-v2-dsvm-spare-pool https://zuul.opendev.org/t/openstack/build/b67492dd06e04faabcec710972620dae : SUCCESS in 39m 52s\n- octavia-v2-act-stdby-dsvm-scenario https://zuul.opendev.org/t/openstack/build/294acce2427d4158a4917192f574d068 : SUCCESS in 42m 44s\n- octavia-grenade https://zuul.opendev.org/t/openstack/build/bc577f3201284a2c8ac4c6fb57cded44 : SUCCESS in 1h 03m 57s","accounts_in_message":[],"_revision_number":2},{"id":"d6fc98f93b5dafcc638a661bc69cd71c8e6e3f08","tag":"autogenerated:gerrit:merged","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-06-30 16:29:43.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":2},{"id":"cd727d93cbde4dc0ef05e939818c9bc29f914cf3","tag":"autogenerated:zuul:promote","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-06-30 16:32:10.000000000","message":"Patch Set 2:\n\nBuild succeeded (promote pipeline).\n\n- promote-openstack-tox-docs https://zuul.opendev.org/t/openstack/build/c0a94f3350334395a9131f6de15c881f : SUCCESS in 1m 57s","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"bd01b2b0b86444bc1fa4501d90c7285ec3368c21","revisions":{"8db3a766429ba5985373eb8ea0801895d14b7306":{"kind":"REWORK","_number":1,"created":"2021-06-28 07:13:51.000000000","uploader":{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},"ref":"refs/changes/55/798255/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/octavia","ref":"refs/changes/55/798255/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/octavia refs/changes/55/798255/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/octavia refs/changes/55/798255/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/octavia refs/changes/55/798255/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/octavia refs/changes/55/798255/1"}}},"commit":{"parents":[{"commit":"c657b82f406cb8f0ec8bc688b5c793e8acbc6979","subject":"Merge \"Fix jobboard_enabled setting in devstack\" into stable/wallaby","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/octavia/commit/c657b82f406cb8f0ec8bc688b5c793e8acbc6979"}]}],"author":{"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","date":"2021-05-03 14:09:25.000000000","tz":120},"committer":{"name":"Carlos Goncalves","email":"cgoncalves@redhat.com","date":"2021-06-28 07:13:05.000000000","tz":120},"subject":"Fix race conditions between API and worker DB calls","message":"Fix race conditions between API and worker DB calls\n\nThe Octavia API creates new resource in the DB and passes the resource\nand its ID to the worker. The worker then accesses to the resource using\nthe database, but it might try to fetch the element before it\u0027s\ncommited by the API.\nThis commit ensures that the resource has been commited to the DB before\nprocessing the request.\n\nStory 2008878\nTask 42423\n\nChange-Id: I55136f570db33cfd22e919019c18e9028a6a9bf7\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/octavia/commit/8db3a766429ba5985373eb8ea0801895d14b7306"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/octavia/commit/8db3a766429ba5985373eb8ea0801895d14b7306"}]},"branch":"refs/heads/stable/wallaby"},"bd01b2b0b86444bc1fa4501d90c7285ec3368c21":{"kind":"NO_CODE_CHANGE","_number":2,"created":"2021-06-28 07:14:57.000000000","uploader":{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},"ref":"refs/changes/55/798255/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/octavia","ref":"refs/changes/55/798255/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/octavia refs/changes/55/798255/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/octavia refs/changes/55/798255/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/octavia refs/changes/55/798255/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/octavia refs/changes/55/798255/2"}}},"commit":{"parents":[{"commit":"c657b82f406cb8f0ec8bc688b5c793e8acbc6979","subject":"Merge \"Fix jobboard_enabled setting in devstack\" into stable/wallaby","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/octavia/commit/c657b82f406cb8f0ec8bc688b5c793e8acbc6979"}]}],"author":{"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","date":"2021-05-03 14:09:25.000000000","tz":120},"committer":{"name":"Carlos Goncalves","email":"cgoncalves@redhat.com","date":"2021-06-28 07:14:47.000000000","tz":120},"subject":"Fix race conditions between API and worker DB calls","message":"Fix race conditions between API and worker DB calls\n\nThe Octavia API creates new resource in the DB and passes the resource\nand its ID to the worker. The worker then accesses to the resource using\nthe database, but it might try to fetch the element before it\u0027s\ncommited by the API.\nThis commit ensures that the resource has been commited to the DB before\nprocessing the request.\n\nStory 2008878\nTask 42423\n\nChange-Id: I55136f570db33cfd22e919019c18e9028a6a9bf7\n(cherry picked from commit cd272ae5a948c22e26b0ea678ea452f559e14d62)\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/octavia/commit/bd01b2b0b86444bc1fa4501d90c7285ec3368c21"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/octavia/commit/bd01b2b0b86444bc1fa4501d90c7285ec3368c21"}]},"branch":"refs/heads/stable/wallaby"}},"requirements":[],"submit_records":[{"status":"CLOSED","labels":[{"label":"Verified","status":"OK","applied_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}},{"label":"Code-Review","status":"OK","applied_by":{"_account_id":7249,"name":"Ann Taraday","email":"akamyshnikova@mirantis.com","username":"AKamyshnikova"}},{"label":"Workflow","status":"OK","applied_by":{"_account_id":7249,"name":"Ann Taraday","email":"akamyshnikova@mirantis.com","username":"AKamyshnikova"}},{"label":"Backport-Candidate","status":"MAY"}]}],"submit_requirements":[{"name":"Verified","status":"SATISFIED","is_legacy":true,"submittability_expression_result":{"expression":"label:Verified\u003dCustom-Rule","fulfilled":true,"status":"PASS","passing_atoms":["label:Verified\u003dCustom-Rule"],"failing_atoms":[],"atom_explanations":{}}},{"name":"Workflow","status":"SATISFIED","is_legacy":true,"submittability_expression_result":{"expression":"label:Workflow\u003dCustom-Rule","fulfilled":true,"status":"PASS","passing_atoms":["label:Workflow\u003dCustom-Rule"],"failing_atoms":[],"atom_explanations":{}}},{"name":"Code-Review","status":"SATISFIED","is_legacy":true,"submittability_expression_result":{"expression":"label:Code-Review\u003dCustom-Rule","fulfilled":true,"status":"PASS","passing_atoms":["label:Code-Review\u003dCustom-Rule"],"failing_atoms":[],"atom_explanations":{}}}]}