)]}'
{"id":"openstack%2Foctavia~854228","triplet_id":"openstack%2Foctavia~stable%2Ftrain~I494ea357afc072360765d7a30a9488e08bfda52c","project":"openstack/octavia","branch":"stable/train","attention_set":{},"removed_from_attention_set":{"29244":{"account":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"last_update":"2022-12-07 13:58:39.000000000","reason":"Change was submitted"}},"hashtags":[],"change_id":"I494ea357afc072360765d7a30a9488e08bfda52c","subject":"Fix bug when rolling back prov and op status for some API calls","status":"MERGED","created":"2022-08-23 15:14:14.000000000","updated":"2022-12-07 14:00:53.000000000","submitted":"2022-12-07 13:58:39.000000000","submitter":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"total_comment_count":1,"unresolved_comment_count":0,"has_review_started":true,"submission_id":"854228","meta_rev_id":"059c6c9cef83c90a5dbfd26175ef024051426637","_number":854228,"virtual_id_number":854228,"owner":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"value":0,"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},{"tag":"autogenerated:zuul:gate","value":2,"date":"2022-12-07 13:58:39.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":34429,"name":"Tom Weininger","email":"dienste@weinimo.de","username":"tweining"}],"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":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"all":[{"value":2,"date":"2022-08-23 21:10:31.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":2,"date":"2022-12-07 11:22:31.000000000","_account_id":34429,"name":"Tom Weininger","email":"dienste@weinimo.de","username":"tweining"}],"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":34429,"name":"Tom Weininger","email":"dienste@weinimo.de","username":"tweining"},"all":[{"value":0,"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":1,"date":"2022-12-07 11:22:31.000000000","_account_id":34429,"name":"Tom Weininger","email":"dienste@weinimo.de","username":"tweining"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Backport-Candidate":{"all":[{"value":0,"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":34429,"name":"Tom Weininger","email":"dienste@weinimo.de","username":"tweining"}],"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":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":34429,"name":"Tom Weininger","email":"dienste@weinimo.de","username":"tweining"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2022-08-23 17:36: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":"2022-08-23 21:10:31.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":"2022-12-07 11:22:31.000000000","updated_by":{"_account_id":34429,"name":"Tom Weininger","email":"dienste@weinimo.de","username":"tweining"},"reviewer":{"_account_id":34429,"name":"Tom Weininger","email":"dienste@weinimo.de","username":"tweining"},"state":"REVIEWER"}],"messages":[{"id":"5d846223c5ac19c6bac393601fee8d2eea96579c","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"date":"2022-08-23 15:14:14.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"f42ddd374f925ef16d793f63b15341e78d7eb01d","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2022-08-23 17:36:37.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/218ffbe7750649598bb8e00d44978d25\n\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/854f4a0badf14ea0955997c310c80692 : SUCCESS in 10m 09s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/2f5a7b8960f84fd083bae815db3fb042 : SUCCESS in 7m 00s\n- openstack-tox-py27 https://zuul.opendev.org/t/openstack/build/979aad632f9448d2861f1440f76d4642 : SUCCESS in 6m 05s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/831123609c4f4e24bc745efa4d277407 : SUCCESS in 5m 56s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/50421b9a7907484ea059a137eb47fd29 : SUCCESS in 6m 11s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/ef9eb974efef4f50bf4673abf11a4d62 : SUCCESS in 11m 54s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/4a326b90f5614692bb7564524d2f9c17 : SUCCESS in 4m 27s\n- octavia-tox-py37-tips https://zuul.opendev.org/t/openstack/build/1a51f909e19c467ebd4d68c6900b0973 : SUCCESS in 6m 28s\n- octavia-tox-functional-py37-tips https://zuul.opendev.org/t/openstack/build/b45c6f49baad493a931eaaa50cd0a02b : SUCCESS in 7m 43s\n- openstack-tox-functional https://zuul.opendev.org/t/openstack/build/bfd109682a374611b2d2356f36943072 : SUCCESS in 6m 28s\n- openstack-tox-functional-py36 https://zuul.opendev.org/t/openstack/build/e7f2645704dd418c8e6123deea6482ed : SUCCESS in 7m 35s\n- octavia-v2-dsvm-noop-api https://zuul.opendev.org/t/openstack/build/d22282f3c376473bb7218bfd08e74675 : SUCCESS in 1h 42m 40s\n- octavia-v2-dsvm-noop-py2-api https://zuul.opendev.org/t/openstack/build/8cf5b09298da44a7bbf2c4a2244bbb30 : SUCCESS in 1h 23m 47s\n- octavia-v2-dsvm-scenario https://zuul.opendev.org/t/openstack/build/779f605131b54df1be2b100f9ecc8f4a : SUCCESS in 1h 55m 35s\n- octavia-v2-dsvm-py2-scenario https://zuul.opendev.org/t/openstack/build/4873860c74f34eeab8fc15967522339e : SUCCESS in 2h 10m 31s (non-voting)\n- octavia-v2-dsvm-py2-scenario-centos-7 https://zuul.opendev.org/t/openstack/build/d911778bcaf34e75be4c5e57f7b87240 : FAILURE in 19m 38s (non-voting)\n- octavia-v2-dsvm-scenario-two-node https://zuul.opendev.org/t/openstack/build/99ae7dbcc615458d990a046c948eb8ca : RETRY_LIMIT in 10m 07s (non-voting)\n- octavia-v2-dsvm-py2-scenario-two-node https://zuul.opendev.org/t/openstack/build/5e9ea1cdba1f402fbc7e682231b51fce : RETRY_LIMIT in 18m 27s (non-voting)\n- octavia-v2-dsvm-tls-barbican https://zuul.opendev.org/t/openstack/build/e272da380ab14717ba8571342d86cb44 : SUCCESS in 1h 03m 14s (non-voting)\n- octavia-v2-dsvm-scenario-amphora-v2 https://zuul.opendev.org/t/openstack/build/c6da02c480ea4d87b85c2970ce962471 : SUCCESS in 1h 57m 12s (non-voting)\n- octavia-v2-act-stdby-iptables-dsvm-scenario https://zuul.opendev.org/t/openstack/build/f0ca7ea1b3f2481cbd38649ad070b940 : FAILURE in 40m 34s (non-voting)\n- octavia-v2-act-stdby-iptables-dsvm-py2-scenario https://zuul.opendev.org/t/openstack/build/0ba4c42f5d2c4aaaa306ddf11f883574 : FAILURE in 40m 48s (non-voting)\n- octavia-v2-act-stdby-iptables-dsvm-py2-scenario-centos-7 https://zuul.opendev.org/t/openstack/build/1fd07211fb2c4fc690fe273197278ec2 : FAILURE in 21m 45s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"9ec761631cb0c17d8a6e25d051eefeceb478c418","author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"date":"2022-08-23 21:10:31.000000000","message":"Patch Set 1: Code-Review+2\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"91178fc138bf57dd575dcbb7d09a95e260143863","author":{"_account_id":34429,"name":"Tom Weininger","email":"dienste@weinimo.de","username":"tweining"},"date":"2022-12-07 11:22:31.000000000","message":"Patch Set 1: Code-Review+2 Workflow+1","accounts_in_message":[],"_revision_number":1},{"id":"7e05a7473b940f5f1ffa9165084df3424207b044","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2022-12-07 11:23:20.000000000","message":"Patch Set 1: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":1},{"id":"703c59e33ffe384ebd186a75e67d74649e9db21f","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2022-12-07 13:58:39.000000000","message":"Patch Set 1: Verified+2\n\nBuild succeeded (gate pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/df8d9ec524a34007b51e885c8a2afbf2\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/e5215272374249edbc3801f602ab5125 : SUCCESS in 7m 23s\n- openstack-tox-py27 https://zuul.opendev.org/t/openstack/build/6dc440ee53a1442fa68addf317244df0 : SUCCESS in 4m 57s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/d64ac2bd2e844084931bd737dd58bbbe : SUCCESS in 5m 08s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/1f88a1d3d50c40859714bebfa0236750 : SUCCESS in 6m 14s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/8b3df4cdb6224cdfb078f36a552f2569 : SUCCESS in 13m 44s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/81f5ab18f3924ac7bdc556e34d6699aa : SUCCESS in 6m 46s\n- openstack-tox-functional https://zuul.opendev.org/t/openstack/build/d2dab3ffd7f448f0b4aea8e43bcb60ed : SUCCESS in 6m 07s\n- openstack-tox-functional-py36 https://zuul.opendev.org/t/openstack/build/76815abe97e642c08a1dbb9dd714e3f1 : SUCCESS in 7m 32s\n- octavia-v2-dsvm-noop-api https://zuul.opendev.org/t/openstack/build/aa780423e2d74c47a3d6101c0d1ad8b4 : SUCCESS in 41m 19s\n- octavia-v2-dsvm-noop-py2-api https://zuul.opendev.org/t/openstack/build/2f81f6436360419fb5d79987e8b1e16b : SUCCESS in 49m 46s\n- octavia-v2-dsvm-scenario https://zuul.opendev.org/t/openstack/build/33a86e35fc704fa4ab8296ebe3e9ae9b : SUCCESS in 1h 43m 17s","accounts_in_message":[],"_revision_number":1},{"id":"168040f58261065f6d9cf415d5f231c1bbba00ca","tag":"autogenerated:gerrit:merged","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2022-12-07 13:58:39.000000000","message":"Change has been successfully merged","accounts_in_message":[],"_revision_number":1},{"id":"059c6c9cef83c90a5dbfd26175ef024051426637","tag":"autogenerated:zuul:promote","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2022-12-07 14:00:53.000000000","message":"Patch Set 1:\n\nBuild failed (promote pipeline).  For information on how to proceed, see\nhttps://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing\n\nhttps://zuul.opendev.org/t/openstack/buildset/7abe5dcccd3f4766aed5c5f606e93254\n\n- promote-openstack-tox-docs https://zuul.opendev.org/t/openstack/build/af87b9b966264290aac35648067e62f0 : SUCCESS in 54s\n- promote-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/fa087a6bed9b400199c3ece2050560d5 : FAILURE in 36s","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"0a48b7bfb639279c2cd5e37e590ec9773735ef3b","revisions":{"0a48b7bfb639279c2cd5e37e590ec9773735ef3b":{"kind":"REWORK","_number":1,"created":"2022-08-23 15:14:14.000000000","uploader":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"ref":"refs/changes/28/854228/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/octavia","ref":"refs/changes/28/854228/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/octavia refs/changes/28/854228/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/octavia refs/changes/28/854228/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/octavia refs/changes/28/854228/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/octavia refs/changes/28/854228/1"}}},"commit":{"parents":[{"commit":"58e2e13b2b107b92f5a9a20ffc83689731f16a38","subject":"Merge \"Fix HealthMonitorToErrorOnRevertTask revert method\" into stable/train","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/octavia/commit/58e2e13b2b107b92f5a9a20ffc83689731f16a38"}]}],"author":{"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","date":"2022-08-08 16:14:45.000000000","tz":120},"committer":{"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","date":"2022-08-23 15:14:03.000000000","tz":120},"subject":"Fix bug when rolling back prov and op status for some API calls","message":"Fix bug when rolling back prov and op status for some API calls\n\nPOST l7rule API call and PUT pool API call had a bug that might have\nkept a load balancer in provisioning_status when the call was rejected\nby the provider.\n\nThe code didn\u0027t use the right locked DB session to set the statuses and\nthen to rollback them when an exception was caught.\n\nStory 2010210\nTask 45947\n\nConflicts:\n\toctavia/tests/functional/api/v2/test_pool.py\n\nChange-Id: I494ea357afc072360765d7a30a9488e08bfda52c\n(cherry picked from commit fdcea400c6f591485fcb116cb47dc07c5229898d)\n(cherry picked from commit 3e6f180b77f1c58265ae9351151e32bb1d215fe7)\n(cherry picked from commit 81cb9d5efccdeabe67b7a79d8de43fdbb66fdd4f)\n(cherry picked from commit 173f024b59a0d25cb66e25bd4f9ca3e62d8ad9b0)\n(cherry picked from commit ab39cc3881a9a05147e620df61d3865b43418ce5)\n(cherry picked from commit 64abc34c7325c695e84d87bab3d187e83e2cbd5d)\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/octavia/commit/0a48b7bfb639279c2cd5e37e590ec9773735ef3b"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/octavia/commit/0a48b7bfb639279c2cd5e37e590ec9773735ef3b"}]},"branch":"refs/heads/stable/train"}},"requirements":[],"submit_records":[{"rule_name":"gerrit~DefaultSubmitRule","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":34429,"name":"Tom Weininger","email":"dienste@weinimo.de","username":"tweining"}},{"label":"Workflow","status":"OK","applied_by":{"_account_id":34429,"name":"Tom Weininger","email":"dienste@weinimo.de","username":"tweining"}},{"label":"Backport-Candidate","status":"MAY"}]}],"submit_requirements":[]}