)]}'
{"id":"openstack%2Foctavia~985909","triplet_id":"openstack%2Foctavia~master~Icff02f7a621cc13a8a0383e1b322f96027c421b7","project":"openstack/octavia","branch":"master","topic":"rlin-retry-on-error-no-db","attention_set":{"12404":{"account":{"_account_id":12404,"name":"Rico Lin","email":"ricolin@ricolky.com","username":"rico.lin"},"last_update":"2026-05-27 02:43:24.000000000","reason":"Someone else replied on the change"}},"removed_from_attention_set":{"38562":{"account":{"_account_id":38562,"name":"Richard Cruise","email":"rcruise@redhat.com","username":"rcruise"},"last_update":"2026-05-27 02:43:24.000000000","reason":"removed on reply"},"38360":{"account":{"_account_id":38360,"name":"Zachary Mark Raines","display_name":"Zachary Raines","email":"zachary.raines@canonical.com","username":"raineszm","status":"Sustaining Engineer @ Canonical"},"last_update":"2026-06-01 20:20:11.000000000","reason":"\u003cGERRIT_ACCOUNT_38360\u003e replied on the change","reason_account":{"_account_id":38360,"name":"Zachary Mark Raines","display_name":"Zachary Raines","email":"zachary.raines@canonical.com","username":"raineszm","status":"Sustaining Engineer @ Canonical"}}},"hashtags":[],"change_id":"Icff02f7a621cc13a8a0383e1b322f96027c421b7","subject":"Allow failover on error","status":"NEW","created":"2026-04-23 04:02:45.000000000","updated":"2026-08-24 19:19:23.000000000","submit_type":"MERGE_IF_NECESSARY","mergeable":true,"submittable":false,"total_comment_count":53,"unresolved_comment_count":6,"has_review_started":true,"meta_rev_id":"87fd80ab429b4e56030f399e70211f010d0750bc","_number":985909,"virtual_id_number":985909,"owner":{"_account_id":12404,"name":"Rico Lin","email":"ricolin@ricolky.com","username":"rico.lin"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"_account_id":38562,"name":"Richard Cruise","email":"rcruise@redhat.com","username":"rcruise"},{"_account_id":38360,"name":"Zachary Mark Raines","display_name":"Zachary Raines","email":"zachary.raines@canonical.com","username":"raineszm","status":"Sustaining Engineer @ Canonical"},{"tag":"autogenerated:zuul:check","value":1,"date":"2026-08-14 10:37:00.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":38360,"name":"Zachary Mark Raines","display_name":"Zachary Raines","email":"zachary.raines@canonical.com","username":"raineszm","status":"Sustaining Engineer @ Canonical"},"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":38562,"name":"Richard Cruise","email":"rcruise@redhat.com","username":"rcruise"},{"value":-1,"date":"2026-08-24 19:19:23.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":38360,"name":"Zachary Mark Raines","display_name":"Zachary Raines","email":"zachary.raines@canonical.com","username":"raineszm","status":"Sustaining Engineer @ Canonical"},{"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":38562,"name":"Richard Cruise","email":"rcruise@redhat.com","username":"rcruise"},{"_account_id":38360,"name":"Zachary Mark Raines","display_name":"Zachary Raines","email":"zachary.raines@canonical.com","username":"raineszm","status":"Sustaining Engineer @ Canonical"},{"_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,"permitted_voting_range":{"min":-1,"max":1},"_account_id":38562,"name":"Richard Cruise","email":"rcruise@redhat.com","username":"rcruise"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":38360,"name":"Zachary Mark Raines","display_name":"Zachary Raines","email":"zachary.raines@canonical.com","username":"raineszm","status":"Sustaining Engineer @ Canonical"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_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":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":38360,"name":"Zachary Mark Raines","display_name":"Zachary Raines","email":"zachary.raines@canonical.com","username":"raineszm","status":"Sustaining Engineer @ Canonical"},{"_account_id":38562,"name":"Richard Cruise","email":"rcruise@redhat.com","username":"rcruise"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2026-04-23 07:15:12.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":"2026-04-23 09:50:47.000000000","updated_by":{"_account_id":38562,"name":"Richard Cruise","email":"rcruise@redhat.com","username":"rcruise"},"reviewer":{"_account_id":38562,"name":"Richard Cruise","email":"rcruise@redhat.com","username":"rcruise"},"state":"REVIEWER"},{"updated":"2026-04-23 20:50:18.000000000","updated_by":{"_account_id":38360,"name":"Zachary Mark Raines","display_name":"Zachary Raines","email":"zachary.raines@canonical.com","username":"raineszm","status":"Sustaining Engineer @ Canonical"},"reviewer":{"_account_id":38360,"name":"Zachary Mark Raines","display_name":"Zachary Raines","email":"zachary.raines@canonical.com","username":"raineszm","status":"Sustaining Engineer @ Canonical"},"state":"REVIEWER"}],"messages":[{"id":"19acf688adc493cbba8ec3399393d733748eee85","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":12404,"name":"Rico Lin","email":"ricolin@ricolky.com","username":"rico.lin"},"date":"2026-04-23 04:02:45.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"9dc9b93d21261f35feac8e06401a084fee326fee","author":{"_account_id":12404,"name":"Rico Lin","email":"ricolin@ricolky.com","username":"rico.lin"},"date":"2026-04-23 04:04:21.000000000","message":"Patch Set 1:\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"93792019062fa94302a1d745d322bd679083d9b7","author":{"_account_id":12404,"name":"Rico Lin","email":"ricolin@ricolky.com","username":"rico.lin"},"date":"2026-04-23 07:01:25.000000000","message":"Patch Set 1:\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"da03dace749d2fc134b355a53aaec8e404ca6498","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2026-04-23 07:15:12.000000000","message":"Patch Set 1: Verified-1\n\n(4 comments)\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttps://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing\nand https://docs.openstack.org/project-team-guide/testing.html#how-to-handle-test-failures\n\nhttps://zuul.opendev.org/t/openstack/buildset/4d511932d3bc4eb08804ac9b5e7147d4\n\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/8f1edd0efdcc4bf6ba5e9cfda75be7cf : SUCCESS in 8m 32s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/b8bd462512644484b0599c0e49b746a5 : FAILURE in 3m 02s\n- openstack-tox-py311 https://zuul.opendev.org/t/openstack/build/8ccc34b8f5a249e4a57aa499e7a8221e : SUCCESS in 4m 42s\n- openstack-tox-py313 https://zuul.opendev.org/t/openstack/build/2aae860c8e104eed966bc6e7e0a2522b : SUCCESS in 4m 59s\n- openstack-tox-py314 https://zuul.opendev.org/t/openstack/build/fa790b34e49e4dbe968055be868982dc : SUCCESS in 6m 43s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/858a817771504a069cc4ee49c733051d : SUCCESS in 11m 27s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/e4c3bbcc4a744b07980d1a237a307a51 : SUCCESS in 3m 11s\n- octavia-tox-py313-tips https://zuul.opendev.org/t/openstack/build/c365fb176017435cb0a80b695114fb50 : SUCCESS in 2m 51s\n- octavia-tox-functional-py313-tips https://zuul.opendev.org/t/openstack/build/ce890e7755c8479189a0ee039093bea0 : SUCCESS in 7m 23s\n- openstack-tox-pip-check-reqs https://zuul.opendev.org/t/openstack/build/3cd954d556404e5fa71ec34c8be78ed6 : SUCCESS in 3m 57s\n- openstack-tox-functional-py310 https://zuul.opendev.org/t/openstack/build/8d6ee67bfcd24e2f9ac505e95fad5a19 : SUCCESS in 4m 03s\n- openstack-tox-functional-py313 https://zuul.opendev.org/t/openstack/build/f6a4922c30f244158c13f633b5433252 : SUCCESS in 7m 07s\n- octavia-v2-dsvm-noop-api https://zuul.opendev.org/t/openstack/build/6170a1319e194c5eadb919453248c8eb : SUCCESS in 42m 40s\n- octavia-v2-dsvm-scenario-traffic-ops https://zuul.opendev.org/t/openstack/build/68967b2e02c04adca746ac9ccd7c4cae : SUCCESS in 1h 51m 01s\n- octavia-v2-dsvm-scenario-non-traffic-ops https://zuul.opendev.org/t/openstack/build/f6168fd7963f4f66b27de6194a00f21a : SUCCESS in 1h 15m 01s\n- octavia-v2-dsvm-scenario-traffic-ops-jobboard https://zuul.opendev.org/t/openstack/build/1c3cf942cb754bf6a16dfbfc78c6bb42 : SUCCESS in 1h 51m 43s\n- octavia-v2-dsvm-scenario-non-traffic-ops-jobboard https://zuul.opendev.org/t/openstack/build/56e62b6cf0a745d6ab551fd3f8157e81 : SUCCESS in 1h 10m 53s\n- octavia-v2-dsvm-tls-barbican https://zuul.opendev.org/t/openstack/build/3b6a9380c68a43618fe5059351f4fc7f : SUCCESS in 47m 53s\n- octavia-grenade https://zuul.opendev.org/t/openstack/build/3740c35f4cb64b3397c4a0ef9117801c : SUCCESS in 49m 47s\n- octavia-grenade-skip-level https://zuul.opendev.org/t/openstack/build/96e1c6f891b34856983a5466d305b711 : SUCCESS in 56m 06s\n- octavia-v2-act-stdby-dsvm-scenario https://zuul.opendev.org/t/openstack/build/5afabf5bed6c49ae87b52770285d0e94 : SUCCESS in 47m 17s (non-voting)\n- octavia-v2-dsvm-cinder-amphora https://zuul.opendev.org/t/openstack/build/b8666bd486d44d348df06f4ed41cc090 : SUCCESS in 49m 56s (non-voting)\n- octavia-v2-dsvm-scenario-two-node https://zuul.opendev.org/t/openstack/build/2aa5f704566f4db8a527ce3dbc1dae10 : TIMED_OUT in 3h 04m 26s (non-voting)\n- octavia-v2-dsvm-scenario-ipv6-only https://zuul.opendev.org/t/openstack/build/c8e35f27fe204bd4b780d9953af0ee7b : SUCCESS in 1h 44m 42s (non-voting)\n- octavia-v2-dsvm-scenario-centos-10-stream-traffic-ops https://zuul.opendev.org/t/openstack/build/b5e1cbe55ed64cb3948fab5a22938fa8 : SUCCESS in 1h 43m 08s (non-voting)\n- octavia-v2-dsvm-scenario-centos-10-stream-non-traffic-ops https://zuul.opendev.org/t/openstack/build/9162f7d032f74742a1b956b14050448c : RETRY_LIMIT in 9m 35s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"e0441b3d0263f685701f618b6497e41a6fe41ada","author":{"_account_id":38562,"name":"Richard Cruise","email":"rcruise@redhat.com","username":"rcruise"},"date":"2026-04-23 09:50:47.000000000","message":"Patch Set 1: Code-Review-1\n\n(3 comments)","accounts_in_message":[],"_revision_number":1},{"id":"2cc07754a654e5591cd2ab70a61870f8e89bb636","author":{"_account_id":38360,"name":"Zachary Mark Raines","display_name":"Zachary Raines","email":"zachary.raines@canonical.com","username":"raineszm","status":"Sustaining Engineer @ Canonical"},"date":"2026-04-23 20:50:18.000000000","message":"Patch Set 1: Code-Review-1\n\n(17 comments)","accounts_in_message":[],"_revision_number":1},{"id":"0e898935fe2c972778d79fe2d7ad5e2a4f657b52","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":12404,"name":"Rico Lin","email":"ricolin@ricolky.com","username":"rico.lin"},"date":"2026-05-20 10:37:54.000000000","message":"Uploaded patch set 2.\n\nOutdated Votes:\n* Code-Review-1 (copy condition: \"changekind:TRIVIAL_REBASE OR is:MIN\")\n* Verified-1 (copy condition: \"NEVER\")\n","accounts_in_message":[],"_revision_number":2},{"id":"4007f419b3178872af28b288ed13226b82d13662","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2026-05-20 13:13:05.000000000","message":"Patch Set 2: 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\nand https://docs.openstack.org/project-team-guide/testing.html#how-to-handle-test-failures\n\nhttps://zuul.opendev.org/t/openstack/buildset/f3ca196c6d73476db3c44410b0b65556\n\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/1eb89427a0204458b8da371bf2cf87a0 : SUCCESS in 9m 04s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/770bf5cdd6e34927b074d37283d1976a : SUCCESS in 2m 45s\n- openstack-tox-py311 https://zuul.opendev.org/t/openstack/build/b0e3dd742e714f0e8260767cae3d8266 : SUCCESS in 3m 27s\n- openstack-tox-py313 https://zuul.opendev.org/t/openstack/build/d15a7fe7051d4386a687153fb43db8aa : SUCCESS in 3m 08s\n- openstack-tox-py314 https://zuul.opendev.org/t/openstack/build/8c3a561c765046e08e32dfb1b3e9bc56 : SUCCESS in 7m 04s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/2ba2fff6dd40482dad58fbcf9d206baa : SUCCESS in 9m 03s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/a12e7d9e9f83432c8dd8f9259fd73444 : SUCCESS in 8m 17s\n- octavia-tox-py313-tips https://zuul.opendev.org/t/openstack/build/3b726adf0c6d494194c641d62fcb780a : SUCCESS in 4m 43s\n- octavia-tox-functional-py313-tips https://zuul.opendev.org/t/openstack/build/643ce23d22884065a5da5637cb7fa017 : SUCCESS in 6m 04s\n- openstack-tox-pip-check-reqs https://zuul.opendev.org/t/openstack/build/af6c924ba02343d798700bfa5fd51116 : SUCCESS in 2m 34s\n- openstack-tox-functional-py310 https://zuul.opendev.org/t/openstack/build/5ed1ebb400fd4d298f4387a4c383d7f9 : FAILURE in 2m 39s\n- openstack-tox-functional-py313 https://zuul.opendev.org/t/openstack/build/6df738d9a17b443fac08e6e86ef49774 : SUCCESS in 3m 39s\n- octavia-v2-dsvm-noop-api https://zuul.opendev.org/t/openstack/build/f0ac74022cd14d6c92dc5dcdbb50d198 : SUCCESS in 27m 39s\n- octavia-v2-dsvm-scenario-traffic-ops https://zuul.opendev.org/t/openstack/build/e4581df176c44d6fb0245dbaf7ce8d6d : SUCCESS in 1h 28m 33s\n- octavia-v2-dsvm-scenario-non-traffic-ops https://zuul.opendev.org/t/openstack/build/a3ceabb1e6a449b28aadb9b356beb106 : SUCCESS in 1h 04m 06s\n- octavia-v2-dsvm-scenario-traffic-ops-jobboard https://zuul.opendev.org/t/openstack/build/7933c9c339a54afc9f2d9f13455e092e : SUCCESS in 1h 41m 59s\n- octavia-v2-dsvm-scenario-non-traffic-ops-jobboard https://zuul.opendev.org/t/openstack/build/8d72077142694d73b41c2e7f3f12ba18 : SUCCESS in 1h 10m 03s\n- octavia-v2-dsvm-tls-barbican https://zuul.opendev.org/t/openstack/build/394fbdc356b1444b943b426c8d937ab1 : SUCCESS in 36m 58s\n- octavia-grenade https://zuul.opendev.org/t/openstack/build/29a6b99391914923a99f1e3dcd880f6b : SUCCESS in 48m 18s\n- octavia-grenade-skip-level https://zuul.opendev.org/t/openstack/build/dc68e07be9d341d39b8ee215eee5dd4d : SUCCESS in 41m 55s\n- octavia-v2-act-stdby-dsvm-scenario https://zuul.opendev.org/t/openstack/build/3cc1432b31b24ac58247ae9dd430db5a : SUCCESS in 35m 27s (non-voting)\n- octavia-v2-dsvm-cinder-amphora https://zuul.opendev.org/t/openstack/build/e37df77439c74ddcb0eb775099dc472f : SUCCESS in 34m 28s (non-voting)\n- octavia-v2-dsvm-scenario-two-node https://zuul.opendev.org/t/openstack/build/47b0d6d71d974f5296a090cb807f956a : SUCCESS in 2h 33m 40s (non-voting)\n- octavia-v2-dsvm-scenario-ipv6-only https://zuul.opendev.org/t/openstack/build/de67374b3f1f4b73bd97670a8526d4aa : SUCCESS in 1h 27m 46s (non-voting)\n- octavia-v2-dsvm-scenario-centos-10-stream-traffic-ops https://zuul.opendev.org/t/openstack/build/c55db998de694ead9e3501b40f3a281e : SUCCESS in 1h 24m 54s (non-voting)\n- octavia-v2-dsvm-scenario-centos-10-stream-non-traffic-ops https://zuul.opendev.org/t/openstack/build/ab0b9b9440ff4972851c189eb1e57090 : SUCCESS in 59m 56s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"52f6ba56d0f341094239e2da636bcd68c872c5fe","author":{"_account_id":12404,"name":"Rico Lin","email":"ricolin@ricolky.com","username":"rico.lin"},"date":"2026-05-25 05:33:26.000000000","message":"Patch Set 2:\n\n(8 comments)","accounts_in_message":[],"_revision_number":2},{"id":"0e3aa02bc637a65e7d66497e217ac4a88f22a421","author":{"_account_id":12404,"name":"Rico Lin","email":"ricolin@ricolky.com","username":"rico.lin"},"date":"2026-05-25 05:42:24.000000000","message":"Patch Set 2:\n\n(8 comments)","accounts_in_message":[],"_revision_number":2},{"id":"563023cf418922bc053e854d98330c19cdfa5447","author":{"_account_id":12404,"name":"Rico Lin","email":"ricolin@ricolky.com","username":"rico.lin"},"date":"2026-05-25 05:42:58.000000000","message":"Patch Set 2:\n\n(8 comments)\n\nResponding to the remaining review threads from patch set 1. These were addressed in patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"e17be14238693a08106332880532c5304f278186","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2026-05-25 08:50:12.000000000","message":"Patch Set 2:\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttps://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing\nand https://docs.openstack.org/project-team-guide/testing.html#how-to-handle-test-failures\n\nhttps://zuul.opendev.org/t/openstack/buildset/8aa99e3da2c74403a4193e8dc6670b36\n\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/222813f1867d4897895eb5e06bc392da : SUCCESS in 9m 08s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/1040f28f592a41c9ab9c37df53aa55b9 : SUCCESS in 4m 38s\n- openstack-tox-py311 https://zuul.opendev.org/t/openstack/build/619c64e7ad9b4ed5b91904ad9cd5a6ad : SUCCESS in 4m 23s\n- openstack-tox-py313 https://zuul.opendev.org/t/openstack/build/6a099d7a9f824ccfbcc9a9d2b3597cca : SUCCESS in 3m 59s\n- openstack-tox-py314 https://zuul.opendev.org/t/openstack/build/0f40f763aaee4b59aa211774136819f9 : SUCCESS in 7m 15s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/faccd189d1bd454190d6c2bf60bf8c35 : SUCCESS in 6m 07s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/6a7cb4639c59452f85e675c739fbe32e : SUCCESS in 8m 30s\n- octavia-tox-py313-tips https://zuul.opendev.org/t/openstack/build/0c10c27905d141c5aa1a4189f19a704f : SUCCESS in 3m 26s\n- octavia-tox-functional-py313-tips https://zuul.opendev.org/t/openstack/build/290db32cb49845a8a73dd61bb78b18fb : SUCCESS in 5m 43s\n- openstack-tox-pip-check-reqs https://zuul.opendev.org/t/openstack/build/28db1284017148c9a85cd0e6f57342f6 : SUCCESS in 2m 09s\n- openstack-tox-functional-py313 https://zuul.opendev.org/t/openstack/build/e0033077ca97459fb1e459f3d88ac26f : SUCCESS in 5m 56s\n- octavia-v2-dsvm-noop-api https://zuul.opendev.org/t/openstack/build/581928bd40434d4aab98620c1318fcfa : SUCCESS in 1h 14m 03s\n- octavia-v2-dsvm-scenario-traffic-ops https://zuul.opendev.org/t/openstack/build/abc961c9f84346199895c2033bee7743 : SUCCESS in 1h 37m 01s\n- octavia-v2-dsvm-scenario-non-traffic-ops https://zuul.opendev.org/t/openstack/build/3ad2d8c6abb84cc79a0a267b2ab5a4fc : TIMED_OUT in 2h 32m 44s\n- octavia-v2-dsvm-scenario-traffic-ops-jobboard https://zuul.opendev.org/t/openstack/build/9779157f640749d3a7c6f0c2a8a1ad2a : SUCCESS in 2h 12m 27s\n- octavia-v2-dsvm-scenario-non-traffic-ops-jobboard https://zuul.opendev.org/t/openstack/build/1e8c64543129431288705d88bdb162e2 : SUCCESS in 1h 29m 11s\n- octavia-v2-dsvm-tls-barbican https://zuul.opendev.org/t/openstack/build/eec3e249b36c4893bcd7f7cbf0a662ac : SUCCESS in 55m 09s\n- octavia-grenade https://zuul.opendev.org/t/openstack/build/cd96d1fdff27496baafee8dc35d01377 : SUCCESS in 1h 12m 21s\n- octavia-grenade-skip-level https://zuul.opendev.org/t/openstack/build/fe95e6f45f04462094bf7cad903fe392 : SUCCESS in 37m 36s\n- octavia-v2-act-stdby-dsvm-scenario https://zuul.opendev.org/t/openstack/build/dbc0240267184ec6bd47b49a76a9e324 : SUCCESS in 52m 40s (non-voting)\n- octavia-v2-dsvm-cinder-amphora https://zuul.opendev.org/t/openstack/build/37665beb9d894b02a812238e192ee2f2 : SUCCESS in 53m 23s (non-voting)\n- octavia-v2-dsvm-scenario-two-node https://zuul.opendev.org/t/openstack/build/4ba70e3dbae24d959e04e74323cc258f : TIMED_OUT in 3h 05m 02s (non-voting)\n- octavia-v2-dsvm-scenario-ipv6-only https://zuul.opendev.org/t/openstack/build/2457d67eea774b859e7f9ae016c46f7b : TIMED_OUT in 2h 32m 56s (non-voting)\n- octavia-v2-dsvm-scenario-centos-10-stream-traffic-ops https://zuul.opendev.org/t/openstack/build/060c3928c1a04fa69c8f5e5a2e2b0baf : SUCCESS in 2h 05m 23s (non-voting)\n- octavia-v2-dsvm-scenario-centos-10-stream-non-traffic-ops https://zuul.opendev.org/t/openstack/build/72760889f91246af9263526223488c57 : SUCCESS in 1h 06m 47s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"b2af98977aa28312aaf78e965255da1997ec8d2d","tag":"autogenerated:claude-review","author":{"_account_id":38562,"name":"Richard Cruise","email":"rcruise@redhat.com","username":"rcruise"},"date":"2026-05-27 02:43:24.000000000","message":"Patch Set 2:\n\n*DevStack integration tests by claude-sonnet-4-6*\n\n**Tested:** change #985909 PS1\n\n## Tests Run\n\n| Test | Result |\n|------|--------|\n| Service Startup and Configuration Validation | ✅ PASS |\n| API Functionality Verification | ✅ PASS |\n| Load Balancer Creation | ✅ PASS |\n| Amphora Status Verification | ✅ PASS |\n| Code Implementation Review | ✅ PASS |\n| Health Manager Log Verification | ✅ PASS |\n\n## Test Results Summary\n\n**Overall Status:** ✅ PASS\n\n**Tests Passed:** 6/6\n**Tests Failed:** 0\n\n**Key Findings:**\n- All Octavia services start successfully with the new code\n- New configuration options are properly loaded and logged\n- Load balancer creation works correctly (backward compatibility maintained)\n- Amphorae are created and managed properly\n- API functionality is fully operational\n- Implementation matches the design specification\n- Code quality is high with good separation of concerns\n\n**Issues Found:** None\n\n**Recommendations:**\n\n1. **For Change Author:**\n   - ✅ Code is ready for review\n   - ✅ Implementation is clean and well-documented\n   - Consider adding a note in documentation about which ERROR conditions can be resolved by failover\n   - Consider adding functional tests for the retry limit and backoff logic\n\n2. **For Operators:**\n   - This feature should be enabled with caution in production\n   - Start with conservative settings (e.g., `max_retries \u003d 1` or `2`)\n   - Monitor resource consumption and amphora creation rate\n   - Ensure adequate compute capacity for failover operations\n   - Review ERROR amphora logs to understand root causes before enabling auto-failover\n\n3. **Additional Testing Suggestions:**\n   - Functional tests to simulate amphora ERROR states and verify retry behavior\n   - Tests to verify retry count calculation from amphora history\n   - Tests to verify exponential backoff timing\n   - Tests to verify max_retries limit enforcement\n   - Integration tests with real amphora failures\n\n---\n\n---\n*This report was generated by an AI and may contain errors.*","accounts_in_message":[],"_revision_number":2},{"id":"ebeaac667e352e215c30e2590d4375ab86dc9f4c","tag":"autogenerated:claude-review","author":{"_account_id":38562,"name":"Richard Cruise","email":"rcruise@redhat.com","username":"rcruise"},"date":"2026-05-27 03:50:36.000000000","message":"Patch Set 2:\n\n*DevStack integration tests by claude-sonnet-4-6*\n\n**Tested:** change #985909 PS1\n\n## Tests Run\n\n| Test | Result |\n|------|--------|\n| New Config Options Registered Correctly | ✅ PASS |\n| New Methods Present in Repository and Controller Worker | ✅ PASS |\n| Health Manager Passes count_error_retries\u003dTrue | ✅ PASS |\n| get_stale_amphora Conditionally Includes ERROR Status | ✅ PASS |\n| Load Balancer Creation and Amphora in ERROR State | ✅ PASS |\n| get_error_retry_count with Live ERROR Amphora | ✅ PASS |\n| _error_retry_gate_passed Logic with Live Amphora | ✅ PASS |\n| Manual Amphora Failover (count_error_retries Code Path) | ⚠️ PARTIAL PASS |\n| Unit Tests — Health Manager | ✅ PASS |\n| Unit Tests — Controller Worker | ✅ PASS |\n| Unit Tests — Common Config | ✅ PASS |\n| Retry Gate Boundary Conditions | ✅ PASS |\n\n## Test Results Summary\n\n**Overall Status:** ✅ PASS (with noted caveats)\n\n**Tests Passed:** 11/12\n**Tests Partial:** 1/12 (Test 8 — failover blocked by Nova quota, not by patch code)\n**Tests Failed:** 0\n\n**Key Findings:**\n\n1. **All four new config options** are correctly defined with appropriate defaults; `failover_on_error\u003dFalse` preserves backward compatibility.\n2. **`get_error_retry_count()`** correctly derives the retry count from amphora history rows without any new database schema changes.\n3. **`_error_retry_gate_passed()`** correctly enforces both max-retries and `wait_random_exponential` backoff.\n4. **`get_stale_amphora()`** correctly extends the `allow_status` list with `ERROR` only when `failover_on_error\u003dTrue`.\n5. **Health manager** correctly passes `count_error_retries\u003dTrue` to `failover_amphora()` during automated health checks.\n6. **All existing unit tests pass** (4 health manager tests, 61 controller worker tests, 5 config tests) — no regressions.\n7. **The feature is opt-in** (disabled by default), reducing risk for operators who don\u0027t need it.\n8. **A live ERROR amphora** was observed during testing (LB BACKUP in ERROR), demonstrating the real-world relevance of this feature.\n\n**Issues Found:**\n\n1. **⚠️ Missing unit tests for new methods:** The patch does not include unit tests for `get_error_retry_count()` or `_error_retry_gate_passed()`. These are new, non-trivial methods and should have test coverage. In particular:\n   - `get_error_retry_count()` — test with no history rows, single ERROR row, multiple consecutive ERROR rows, rows outside the window, and amphorae without a `load_balancer_id`\n   - `_error_retry_gate_passed()` — test retry-limit exceeded, limit\u003d-1, backoff gate (elapsed \u003c required_wait), and happy path\n\n---\n*This report was generated by an AI and may contain errors.*","accounts_in_message":[],"_revision_number":2},{"id":"a63b17e07f437ad16761b833aef90e115dd33d40","tag":"autogenerated:claude-review","author":{"_account_id":38562,"name":"Richard Cruise","email":"rcruise@redhat.com","username":"rcruise"},"date":"2026-05-28 06:46:31.000000000","message":"Patch Set 2:\n\n*DevStack integration tests by claude-sonnet-4-6*\n\n**Tested:** change #985909 PS1\n\n## Tests Run\n\n| Test | Result |\n|------|--------|\n| Service Startup and Configuration Validation | ✅ PASS |\n| API Functionality Verification | ✅ PASS |\n| Load Balancer Creation | ✅ PASS |\n| Amphora Status Verification | ✅ PASS |\n| Code Implementation Review | ✅ PASS |\n| Health Manager Log Verification | ✅ PASS |\n| Service Startup | ✅ PASS |\n| Load Balancer Create | ✅ PASS |\n| Listener Create | ✅ PASS |\n| Pool Create | ✅ PASS |\n| Amphora Status and ERROR Detection | ✅ PASS |\n| Log Inspection | ✅ PASS |\n| Cleanup Verification | ✅ PASS |\n\n## Test Results Summary\n\n**Overall Status:** ✅ PASS\n\n**Tests Passed:** 6/6\n**Tests Failed:** 0\n\n**Key Findings:**\n- All Octavia services start successfully with the new code\n- New configuration options are properly loaded and logged\n- Load balancer creation works correctly (backward compatibility maintained)\n- Amphorae are created and managed properly\n- API functionality is fully operational\n- Implementation matches the design specification\n- Code quality is high with good separation of concerns\n\n**Issues Found:** None\n\n**Recommendations:**\n\n1. **For Change Author:**\n   - ✅ Code is ready for review\n   - ✅ Implementation is clean and well-documented\n   - Consider adding a note in documentation about which ERROR conditions can be resolved by failover\n   - Consider adding functional tests for the retry limit and backoff logic\n\n2. **For Operators:**\n   - This feature should be enabled with caution in production\n   - Start with conservative settings (e.g., `max_retries \u003d 1` or `2`)\n   - Monitor resource consumption and amphora creation rate\n   - Ensure adequate compute capacity for failover operations\n   - Review ERROR amphora logs to understand root causes before enabling auto-failover\n\n3. **Additional Testing Suggestions:**\n   - Functional tests to simulate amphora ERROR states and verify retry behavior\n   - Tests to verify retry count calculation from amphora history\n   - Tests to verify exponential backoff timing\n   - Tests to verify max_retries limit enforcement\n   - Integration tests with real amphora failures\n\n---\n\n---\n*This report was generated by an AI and may contain errors.*","accounts_in_message":[],"_revision_number":2},{"id":"b68769d68eaa2fe53c8a2f2760721fdbfc2ed587","author":{"_account_id":38360,"name":"Zachary Mark Raines","display_name":"Zachary Raines","email":"zachary.raines@canonical.com","username":"raineszm","status":"Sustaining Engineer @ Canonical"},"date":"2026-06-01 20:20:11.000000000","message":"Patch Set 2:\n\n(6 comments)","accounts_in_message":[],"_revision_number":2},{"id":"8d071f076ada64c256ab943041a8b08eae806c1a","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":12404,"name":"Rico Lin","email":"ricolin@ricolky.com","username":"rico.lin"},"date":"2026-08-14 08:23:13.000000000","message":"Uploaded patch set 3.\n\nOutdated Votes:\n* Verified-1 (copy condition: \"NEVER\")\n","accounts_in_message":[],"_revision_number":3},{"id":"a701c4e760ad78050e9891a5aafa6ada6f0c07bf","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2026-08-14 10:37:00.000000000","message":"Patch Set 3: Verified+1\n\nBuild succeeded (check pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/cd97639c46074605827166cb487289d8\n\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/9839dead868f4fe997e85ae123e98af8 : SUCCESS in 9m 24s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/e84c0ecee1d8496ea2867536f3774e5c : SUCCESS in 3m 05s\n- openstack-tox-py311 https://zuul.opendev.org/t/openstack/build/f8c238ec55164271815cecb08c33b4c6 : SUCCESS in 3m 14s\n- openstack-tox-py313 https://zuul.opendev.org/t/openstack/build/dee36f362b524744ab2b2da9d4f5eb20 : SUCCESS in 4m 54s\n- openstack-tox-py314 https://zuul.opendev.org/t/openstack/build/cda96ff4be3c4cf1ba39592d35b7b5e6 : SUCCESS in 9m 54s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/8ec8eb171bb54edaba72fcde651efb3b : SUCCESS in 11m 51s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/dd5d8c5698164092acbc17c29013fc5b : SUCCESS in 8m 39s\n- octavia-tox-py313-tips https://zuul.opendev.org/t/openstack/build/0570b37233fb4776b543f6506f7bc8ff : SUCCESS in 3m 30s\n- octavia-tox-functional-py313-tips https://zuul.opendev.org/t/openstack/build/05767df72e594fed851d087b0fd42946 : SUCCESS in 5m 04s\n- openstack-tox-pip-check-reqs https://zuul.opendev.org/t/openstack/build/ccc65cae44774853a4a92e04285eea16 : SUCCESS in 3m 18s\n- openstack-tox-functional-py313 https://zuul.opendev.org/t/openstack/build/f4dc8ad138974e038b5f4fc212b477ae : SUCCESS in 5m 31s\n- octavia-v2-dsvm-noop-api https://zuul.opendev.org/t/openstack/build/eb903da030a349259f387914f41541ea : SUCCESS in 36m 42s\n- octavia-v2-dsvm-scenario-traffic-ops https://zuul.opendev.org/t/openstack/build/7c11854a19eb4d38bf57626e86738bc9 : SUCCESS in 1h 21m 44s\n- octavia-v2-dsvm-scenario-non-traffic-ops https://zuul.opendev.org/t/openstack/build/be2c56a7e7854513903e0d324e246504 : SUCCESS in 1h 13m 21s\n- octavia-v2-dsvm-scenario-traffic-ops-jobboard https://zuul.opendev.org/t/openstack/build/87c8a836ebf740a0ae60db9f1787bee4 : SUCCESS in 1h 34m 15s\n- octavia-v2-dsvm-scenario-non-traffic-ops-jobboard https://zuul.opendev.org/t/openstack/build/e4769b19258447bf9e36953a89d75c93 : SUCCESS in 1h 11m 29s\n- octavia-v2-dsvm-tls-barbican https://zuul.opendev.org/t/openstack/build/22af0db0f8b741449aba2d0503b5a97b : SUCCESS in 38m 45s\n- octavia-grenade https://zuul.opendev.org/t/openstack/build/f1e2366f1bb44484aa0e4d64a884bdd6 : SUCCESS in 49m 36s\n- octavia-grenade-skip-level https://zuul.opendev.org/t/openstack/build/4fcae7d71aaf413abff76bf148e6943f : SUCCESS in 41m 27s\n- octavia-v2-act-stdby-dsvm-scenario https://zuul.opendev.org/t/openstack/build/73d0f2d6f6964fea803c22f5fe0e40dd : SUCCESS in 30m 49s (non-voting)\n- octavia-v2-dsvm-cinder-amphora https://zuul.opendev.org/t/openstack/build/a675c7053c544ff2bc00e2dc2f9fd716 : FAILURE in 8m 35s (non-voting)\n- octavia-v2-dsvm-scenario-two-node https://zuul.opendev.org/t/openstack/build/60077519ef7f426296447e56ac41b9ca : FAILURE in 2h 12m 11s (non-voting)\n- octavia-v2-dsvm-scenario-ipv6-only https://zuul.opendev.org/t/openstack/build/f4d9c1d29de04be9878aa48c0857d3a0 : SUCCESS in 1h 18m 18s (non-voting)\n- octavia-v2-dsvm-scenario-centos-10-stream-traffic-ops https://zuul.opendev.org/t/openstack/build/25062cdda2d04cdea2828557d36acd76 : SUCCESS in 1h 34m 09s (non-voting)\n- octavia-v2-dsvm-scenario-centos-10-stream-non-traffic-ops https://zuul.opendev.org/t/openstack/build/c46a8340a56e4420af28321b2f4adefc : SUCCESS in 1h 14m 08s (non-voting)","accounts_in_message":[],"_revision_number":3},{"id":"87fd80ab429b4e56030f399e70211f010d0750bc","author":{"_account_id":38360,"name":"Zachary Mark Raines","display_name":"Zachary Raines","email":"zachary.raines@canonical.com","username":"raineszm","status":"Sustaining Engineer @ Canonical"},"date":"2026-08-24 19:19:23.000000000","message":"Patch Set 3: Code-Review-1\n\n(1 comment)","accounts_in_message":[],"_revision_number":3}],"current_revision_number":3,"current_revision":"5547dd135586d0f6978ef74f3ae7d02bcdf2b9aa","revisions":{"82ee78d863ee214e0c1dafb886247a320540eaa0":{"kind":"REWORK","_number":1,"created":"2026-04-23 04:02:45.000000000","uploader":{"_account_id":12404,"name":"Rico Lin","email":"ricolin@ricolky.com","username":"rico.lin"},"ref":"refs/changes/09/985909/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/octavia","ref":"refs/changes/09/985909/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/octavia refs/changes/09/985909/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/octavia refs/changes/09/985909/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/octavia refs/changes/09/985909/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/octavia refs/changes/09/985909/1"}}},"commit":{"parents":[{"commit":"9f2335b85a92d0eb76e301299d3daa0b82b846f0","subject":"Merge \"Remove amphora_health record on revert CreateAmphoraInDB\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/octavia/commit/9f2335b85a92d0eb76e301299d3daa0b82b846f0"}]}],"author":{"name":"ricolin","email":"rlin@vexxhost.com","date":"2026-04-23 03:21:09.000000000","tz":480},"committer":{"name":"ricolin","email":"rlin@vexxhost.com","date":"2026-04-23 04:00:08.000000000","tz":480},"subject":"Allow failover on error","message":"Allow failover on error\n\nAdd config `[health_manager]/failover_on_error` to let the health\nmanager pick up amphorae that are in ERROR status in addition to the\nexisting ALLOCATED / FAILOVER_STOPPED statuses.\n\nAlso add:\n\n* `failover_on_error_max_retries` (default 3, -1 disables the limit)\n  caps the number of consecutive automatic failovers for an amphora in\n  ERROR.\n* `failover_on_error_retry_max_interval` (default 60s) and\n  `failover_on_error_retry_multiplier` (default 1.0) implement a\n  randomized exponential backoff between retries, following tenacity\u0027s\n  `wait_random_exponential` strategy: the delay before retry N\n  (1-indexed) is a uniform random value in\n  [0, min(max_interval, multiplier * 2 ** (N - 1))].\n\nUnlike the previous revision of this change, no schema change is\nrequired. The retry count for a given ERROR amphora is derived from\nexisting amphora history rows (same load_balancer_id and role, recent\nupdated_at) by `AmphoraRepository.get_error_retry_count`. The backoff\ngate uses the amphora\u0027s existing `updated_at` column as the\n\u0027last attempt time\u0027.\n\nThe retry-count and backoff checks live in\n`ControllerWorker.failover_amphora` so the single-SELECT scheduling in\n`AmphoraHealthRepository.get_stale_amphora` stays simple. When a retry\nis deferred, the `amphora_health.busy` lock is released so the next\nscheduling tick can pick the amphora up again once the backoff elapses.\n\nMany amphora ERROR states can be resolved by running a failover on the\nload balancer. Without changing existing behavior, these new configs\nallow environments to automatically failover amphorae in ERROR status.\n\nAssisted-by: Claude \u003cnoreply@anthropic.com\u003e\nChange-Id: Icff02f7a621cc13a8a0383e1b322f96027c421b7\nSigned-off-by: ricolin \u003crlin@vexxhost.com\u003e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/octavia/commit/82ee78d863ee214e0c1dafb886247a320540eaa0"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/octavia/commit/82ee78d863ee214e0c1dafb886247a320540eaa0"}]},"branch":"refs/heads/master"},"a556e7a269570c3f07a4bbca15d729693537a48b":{"kind":"REWORK","_number":2,"created":"2026-05-20 10:37:54.000000000","uploader":{"_account_id":12404,"name":"Rico Lin","email":"ricolin@ricolky.com","username":"rico.lin"},"ref":"refs/changes/09/985909/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/octavia","ref":"refs/changes/09/985909/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/octavia refs/changes/09/985909/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/octavia refs/changes/09/985909/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/octavia refs/changes/09/985909/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/octavia refs/changes/09/985909/2"}}},"commit":{"parents":[{"commit":"9f2335b85a92d0eb76e301299d3daa0b82b846f0","subject":"Merge \"Remove amphora_health record on revert CreateAmphoraInDB\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/octavia/commit/9f2335b85a92d0eb76e301299d3daa0b82b846f0"}]}],"author":{"name":"ricolin","email":"rlin@vexxhost.com","date":"2026-04-23 03:21:09.000000000","tz":480},"committer":{"name":"ricolin","email":"rlin@vexxhost.com","date":"2026-05-20 09:49:07.000000000","tz":480},"subject":"Allow failover on error","message":"Allow failover on error\n\nAdd config `[health_manager]/failover_on_error` to let the health\nmanager pick up amphorae that are in ERROR status in addition to the\nexisting ALLOCATED / FAILOVER_STOPPED statuses.\n\nAlso add:\n\n* `failover_on_error_max_retries` (default 3, -1 disables the limit)\n  caps the number of automatic failover attempts for an amphora in\n  ERROR.\n* `failover_on_error_retry_max_interval` (default 60s),\n  `failover_on_error_retry_min_interval` (default 10s), and\n  `failover_on_error_retry_multiplier` (default 1.0) implement a\n  bounded randomized exponential backoff between retries: the delay\n  before retry N is a uniform random value in\n  [min_interval, min(max_interval, multiplier * 2 ** (N - 1))]. The\n  chosen delay is deterministic for a given amphora attempt so health\n  manager ticks do not redraw the wait distribution.\n\nUnlike the previous revision of this change, no schema change is\nrequired. The retry count for a given ERROR amphora is derived from\nexisting amphora history rows (same load_balancer_id and role, recent\nupdated_at) by `AmphoraRepository.get_error_retry_count` using a filtered\nCOUNT query. The backoff gate uses the amphora\u0027s existing `updated_at`\ncolumn as the \u0027last attempt time\u0027.\n\nThe retry-count and backoff checks live in\n`ControllerWorker.failover_amphora` so the single-SELECT scheduling in\n`AmphoraHealthRepository.get_stale_amphora` stays simple. When a retry is\ndeferred or refused, the health-manager lock and load balancer\nprovisioning status are released outside the retry predicate.\n\nMany amphora ERROR states can be resolved by running a failover on the\nload balancer. Without changing existing behavior, these new configs\nallow environments to automatically failover amphorae in ERROR status.\n\nAssisted-by: Claude \u003cnoreply@anthropic.com\u003e\nAssisted-By: OpenAI Codex \u003cnoreply@openai.com\u003e\nChange-Id: Icff02f7a621cc13a8a0383e1b322f96027c421b7\nSigned-off-by: ricolin \u003crlin@vexxhost.com\u003e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/octavia/commit/a556e7a269570c3f07a4bbca15d729693537a48b"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/octavia/commit/a556e7a269570c3f07a4bbca15d729693537a48b"}]},"branch":"refs/heads/master"},"5547dd135586d0f6978ef74f3ae7d02bcdf2b9aa":{"kind":"REWORK","_number":3,"created":"2026-08-14 08:23:13.000000000","uploader":{"_account_id":12404,"name":"Rico Lin","email":"ricolin@ricolky.com","username":"rico.lin"},"ref":"refs/changes/09/985909/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/octavia","ref":"refs/changes/09/985909/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/octavia refs/changes/09/985909/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/octavia refs/changes/09/985909/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/octavia refs/changes/09/985909/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/octavia refs/changes/09/985909/3"}}},"commit":{"parents":[{"commit":"9f2335b85a92d0eb76e301299d3daa0b82b846f0","subject":"Merge \"Remove amphora_health record on revert CreateAmphoraInDB\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/octavia/commit/9f2335b85a92d0eb76e301299d3daa0b82b846f0"}]}],"author":{"name":"Rico Lin","email":"rlin@vexxhost.com","date":"2026-04-23 03:21:09.000000000","tz":480},"committer":{"name":"Rico Lin","email":"rlin@vexxhost.com","date":"2026-08-14 08:23:06.000000000","tz":480},"subject":"Allow failover on error","message":"Allow failover on error\n\nAdd config `[health_manager]/failover_on_error` to let the health\nmanager pick up amphorae that are in ERROR status in addition to the\nexisting ALLOCATED / FAILOVER_STOPPED statuses.\n\nAlso add:\n\n* `failover_on_error_max_retries` (default 3, -1 disables the limit)\n  caps the number of automatic failover attempts for an amphora in\n  ERROR.\n* `failover_on_error_retry_max_interval` (default 60s),\n  `failover_on_error_retry_min_interval` (default 10s), and\n  `failover_on_error_retry_multiplier` (default 1.0) implement a\n  bounded randomized exponential backoff between retries: the delay\n  before retry N is a uniform random value in\n  [min_interval, min(max_interval, multiplier * 2 ** (N - 1))]. The\n  chosen delay is deterministic for a given amphora attempt so health\n  manager ticks do not redraw the wait distribution.\n\nUnlike the previous revision of this change, no schema change is\nrequired. The retry count for a given ERROR amphora is derived from\nexisting amphora history rows (same load_balancer_id and role, recent\nupdated_at) by `AmphoraRepository.get_error_retry_count` using a filtered\nCOUNT query. The backoff gate uses the amphora\u0027s existing `updated_at`\ncolumn as the \u0027last attempt time\u0027.\n\nThe retry-count and backoff checks live in\n`ControllerWorker.failover_amphora` so the single-SELECT scheduling in\n`AmphoraHealthRepository.get_stale_amphora` stays simple. When a retry is\ndeferred or refused, the health-manager lock and load balancer\nprovisioning status are released outside the retry predicate.\n\nMany amphora ERROR states can be resolved by running a failover on the\nload balancer. Without changing existing behavior, these new configs\nallow environments to automatically failover amphorae in ERROR status.\n\nChange-Id: Icff02f7a621cc13a8a0383e1b322f96027c421b7\nSigned-off-by: Rico Lin \u003crlin@vexxhost.com\u003e\nAssisted-By: Claude \u003cnoreply@anthropic.com\u003e\nAssisted-By: OpenAI Codex \u003cnoreply@openai.com\u003e\nAssisted-By: Gemini \u003cnoreply@google.com\u003e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/octavia/commit/5547dd135586d0f6978ef74f3ae7d02bcdf2b9aa"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/octavia/commit/5547dd135586d0f6978ef74f3ae7d02bcdf2b9aa"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[{"rule_name":"gerrit~DefaultSubmitRule","status":"OK","labels":[{"label":"Verified","status":"MAY","applied_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}},{"label":"Code-Review","status":"MAY","applied_by":{"_account_id":38360,"name":"Zachary Mark Raines","display_name":"Zachary Raines","email":"zachary.raines@canonical.com","username":"raineszm","status":"Sustaining Engineer @ Canonical"}},{"label":"Workflow","status":"MAY"},{"label":"Backport-Candidate","status":"MAY"}]}],"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":{"label:Verified\u003dMAX":"","label:Verified\u003dMIN":""}}},{"name":"Backport-Candidate","description":"Backport candidate status","status":"NOT_APPLICABLE","is_legacy":false,"applicability_expression_result":{"fulfilled":false,"status":"FAIL"},"submittability_expression_result":{"expression":"is:true","fulfilled":true,"status":"NOT_EVALUATED","passing_atoms":[],"failing_atoms":[],"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":{"label:Code-Review\u003dMAX":"","label:Code-Review\u003dMIN":""}}},{"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":{"label:Workflow\u003dMAX":"","label:Workflow\u003dMIN":""}}}]}