diff --git a/ansible/roles/haproxy/tasks/deploy.yml b/ansible/roles/haproxy/tasks/deploy.yml index 4768324898..e2768de7e2 100644 --- a/ansible/roles/haproxy/tasks/deploy.yml +++ b/ansible/roles/haproxy/tasks/deploy.yml @@ -3,5 +3,7 @@ - include_tasks: config.yml -- name: Flush handlers - meta: flush_handlers +# NOTE(yoctozepto): haproxy role handlers should not be flushed early. +# site.yml handles all haproxy things in a dedicated play. +# This is to avoid extra haproxy service restart. +# See: https://bugs.launchpad.net/kolla-ansible/+bug/1875228 diff --git a/ansible/roles/haproxy/tasks/upgrade.yml b/ansible/roles/haproxy/tasks/upgrade.yml index ad22684c4b..32a5d4d716 100644 --- a/ansible/roles/haproxy/tasks/upgrade.yml +++ b/ansible/roles/haproxy/tasks/upgrade.yml @@ -16,5 +16,7 @@ notify: - Restart keepalived container -- name: Flush handlers - meta: flush_handlers +# NOTE(yoctozepto): haproxy role handlers should not be flushed early. +# site.yml handles all haproxy things in a dedicated play. +# This is to avoid extra haproxy service restart. +# See: https://bugs.launchpad.net/kolla-ansible/+bug/1875228 diff --git a/releasenotes/notes/bug-1864810-5a5d0f91c0171b19.yaml b/releasenotes/notes/bug-1864810-5a5d0f91c0171b19.yaml new file mode 100644 index 0000000000..a21654fb13 --- /dev/null +++ b/releasenotes/notes/bug-1864810-5a5d0f91c0171b19.yaml @@ -0,0 +1,7 @@ +--- +fixes: + - | + Fixes haproxy role to avoid restarting haproxy service multiple times + in a single Ansible run. + `LP#1864810 `__ + `LP#1875228 `__