Move all pcs commands to the host
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
tripleo |
Expired
|
High
|
Unassigned |
Bug Description
Currently we create pcmk resources via especially crafted containers by running pcs commands inside. This has always been a bit constraining because pacemaker was just never built with the possibility to run these commands inside a container in mind. Every small discrepancy between host and containers keeps hurtings us (libqb CVEs, minor pacemaker changes, etc.).
We should strive to execute these commands from the host and not have to deal with these issues any longer.
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to tripleo-heat-templates (master) | #1 |
OpenStack Infra (hudson-openstack) wrote : | #2 |
Related fix proposed to branch: master
Review: https:/
OpenStack Infra (hudson-openstack) wrote : | #3 |
Related fix proposed to branch: master
Review: https:/
OpenStack Infra (hudson-openstack) wrote : | #4 |
Related fix proposed to branch: master
Review: https:/
OpenStack Infra (hudson-openstack) wrote : | #5 |
Related fix proposed to branch: master
Review: https:/
tags: | added: tech-debt train-backport-potential |
tags: | added: queens-backport-potential |
tags: | added: containers |
Changed in tripleo: | |
milestone: | none → ussuri-3 |
OpenStack Infra (hudson-openstack) wrote : | #6 |
Related fix proposed to branch: master
Review: https:/
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to puppet-tripleo (master) | #7 |
Related fix proposed to branch: master
Review: https:/
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to tripleo-heat-templates (master) | #8 |
Related fix proposed to branch: master
Review: https:/
OpenStack Infra (hudson-openstack) wrote : | #9 |
Related fix proposed to branch: master
Review: https:/
Changed in tripleo: | |
milestone: | ussuri-3 → ussuri-rc3 |
OpenStack Infra (hudson-openstack) wrote : Related fix merged to tripleo-heat-templates (master) | #10 |
Reviewed: https:/
Committed: https:/
Submitter: Zuul
Branch: master
commit 4d8eb351147c359
Author: Michele Baldessari <email address hidden>
Date: Fri Feb 21 17:05:46 2020 +0100
Drop bootstrap_host_exec from pacemaker_
bootstrap_
existence. Since, barring argument checking, it is three lines of shell
script [1] we just use it directly. We also add an extra echo to make it
simpler to debug any bootstrap vs non-bootstrap issued.
[1] https:/
Change-Id: Ia850286682f09c
Related-Bug: #1863442
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to tripleo-ansible (master) | #11 |
Related fix proposed to branch: master
Review: https:/
OpenStack Infra (hudson-openstack) wrote : Change abandoned on tripleo-heat-templates (master) | #12 |
Change abandoned by Michele Baldessari (<email address hidden>) on branch: master
Review: https:/
OpenStack Infra (hudson-openstack) wrote : Related fix merged to puppet-tripleo (master) | #13 |
Reviewed: https:/
Committed: https:/
Submitter: Zuul
Branch: master
commit d185cbf032e02ee
Author: Michele Baldessari <email address hidden>
Date: Wed Feb 19 21:07:09 2020 +0100
Allow OCF resources to be created with --force
While moving to running pcs commands on the host and off short-lived
containers, we are confronted with the issue that pcs usually checks
for the resource agent's existence on the host before creating it.
Since we'd rather avoid installing the needed resource agents on the
host (as it is inside a container), we allow a new 'force_ocf' parameter
to be passed to those situations where we might need it.
Depends-On: I20eb78a061a334
Related-Bug: #1863442
Change-Id: If9048196b5c03e
Changed in tripleo: | |
milestone: | ussuri-rc3 → victoria-1 |
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to puppet-tripleo (stable/ussuri) | #14 |
Related fix proposed to branch: stable/ussuri
Review: https:/
OpenStack Infra (hudson-openstack) wrote : Related fix merged to puppet-tripleo (stable/ussuri) | #15 |
Reviewed: https:/
Committed: https:/
Submitter: Zuul
Branch: stable/ussuri
commit 844975ef4be5e52
Author: Michele Baldessari <email address hidden>
Date: Wed Feb 19 21:07:09 2020 +0100
Allow OCF resources to be created with --force
While moving to running pcs commands on the host and off short-lived
containers, we are confronted with the issue that pcs usually checks
for the resource agent's existence on the host before creating it.
Since we'd rather avoid installing the needed resource agents on the
host (as it is inside a container), we allow a new 'force_ocf' parameter
to be passed to those situations where we might need it.
Depends-On: I20eb78a061a334
Related-Bug: #1863442
Change-Id: If9048196b5c03e
(cherry picked from commit d185cbf032e02ee
tags: | added: in-stable-ussuri |
Changed in tripleo: | |
milestone: | victoria-1 → victoria-3 |
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to tripleo-heat-templates (stable/train) | #16 |
Related fix proposed to branch: stable/train
Review: https:/
OpenStack Infra (hudson-openstack) wrote : Related fix merged to tripleo-heat-templates (stable/train) | #17 |
Reviewed: https:/
Committed: https:/
Submitter: Zuul
Branch: stable/train
commit da6b7b424c2eb70
Author: Michele Baldessari <email address hidden>
Date: Fri Feb 21 17:05:46 2020 +0100
Drop bootstrap_host_exec from pacemaker_
bootstrap_
existence. Since, barring argument checking, it is three lines of shell
script [1] we just use it directly. We also add an extra echo to make it
simpler to debug any bootstrap vs non-bootstrap issued.
[1] https:/
Change-Id: Ia850286682f09c
Related-Bug: #1863442
(cherry picked from commit 4d8eb351147c359
tags: | added: in-stable-train |
OpenStack Infra (hudson-openstack) wrote : Related fix merged to tripleo-ansible (master) | #18 |
Reviewed: https:/
Committed: https:/
Submitter: Zuul
Branch: master
commit d5782863b8b1da7
Author: Michele Baldessari <email address hidden>
Date: Fri May 1 12:43:46 2020 +0200
Add tripleo_ha_wrapper role
This role is meant to help moving the HA pcs tasks to pure deployment
steps run on the host and not in containers any more. We tested this
ansible review with the THT patches at
https:/
Here is the testing protocol that was used in order to verify we did
not break anything:
1) Deployed overcloud + tempest
2) Ran a successful redeploy
2.1) Verified that a redeploy restarted containers globally when the
2.2) Verified that a redeploy did not restart containers which config
did not change
2.3) Verified that the ordering of restarting bundles allows for
brownfield deployments (i.e. first the bundle is restarted due
to new bind-mounts and then the bundle is restarted with new
2.4) Verified that converge step after minor update is successful
3) Ran an initial successful minor update
3.1) Verified that the new image is indeed being run by pacemaker
(i.e. :pcmklatest tag is applied to the new containers)
3.2) Verified that after converge no container is restarted
4) Re-run a subsequent successful noop minor update
4.1) Verify that containers are not restarted a second time after
the cluster is stopped and started
Depends-On: I367cf4b65300be
Change-Id: Iaa7e89f0d25221
Related-Bug: #1863442
OpenStack Infra (hudson-openstack) wrote : Related fix merged to tripleo-heat-templates (master) | #19 |
Reviewed: https:/
Committed: https:/
Submitter: Zuul
Branch: master
commit da3d5e80562f7aa
Author: Michele Baldessari <email address hidden>
Date: Fri May 1 13:12:46 2020 +0200
pcs commands on host: mysql
This implements the creation of the haproxy bundle on the host.
The testing protocol used is documented in the depends-on.
The reason for adding a post_update task is that during a minor update
the deployment tasks are not run during the node update procedure but
only during the final converge. So we ran the role again there to make
sure that any config change will trigger a restart during the minor
update, so the disruption is only local to the single node being
updated. If we did not do this a final converge could potentially
trigger a global restart of HA bundles which would be quite disruptive.
NB: in this case we keep the container init_bundle (renamed to
wait_bundle) around just use it to wait for galera to be up.
Depends-On: Iaa7e89f0d25221
Change-Id: Ie14819b66cecdb
Related-Bug: #1863442
OpenStack Infra (hudson-openstack) wrote : | #20 |
Reviewed: https:/
Committed: https:/
Submitter: Zuul
Branch: master
commit a11622359002597
Author: Michele Baldessari <email address hidden>
Date: Fri May 1 13:07:13 2020 +0200
pcs commands on host: haproxy bundle
This implements the creation of the haproxy bundle on the host.
The testing protocol used is documented in the depends-on.
The reason for adding a post_update task is that during a minor update
the deployment tasks are not run during the node update procedure but
only during the final converge. So we ran the role again there to make
sure that any config change will trigger a restart during the minor
update, so the disruption is only local to the single node being
updated. If we did not do this a final converge could potentially
trigger a global restart of HA bundles which would be quite disruptive.
NB: We had to remove a few tags that were present befaure because we want
to avoid that puppet checks for /etc/haproxy/
on the host.
Related-Bug: #1863442
Depends-On: Iaa7e89f0d25221
Change-Id: I20dc57a02a315c
OpenStack Infra (hudson-openstack) wrote : | #21 |
Reviewed: https:/
Committed: https:/
Submitter: Zuul
Branch: master
commit a1bee391361cde4
Author: Michele Baldessari <email address hidden>
Date: Fri May 1 13:28:36 2020 +0200
pcs commands on host: manila-share
This implements the creation of the manila-share bundle on the host.
The testing protocol used is documented in the depends-on.
The reason for adding a post_update task is that during a minor update
the deployment tasks are not run during the node update procedure but
only during the final converge. So we ran the role again there to make
sure that any config change will trigger a restart during the minor
update, so the disruption is only local to the single node being
updated. If we did not do this a final converge could potentially
trigger a global restart of HA bundles which would be quite disruptive.
Depends-On: Iaa7e89f0d25221
Change-Id: If5730dba4973a5
Related-Bug: #1863442
OpenStack Infra (hudson-openstack) wrote : | #22 |
Reviewed: https:/
Committed: https:/
Submitter: Zuul
Branch: master
commit f37f21f31df12e5
Author: Michele Baldessari <email address hidden>
Date: Fri May 1 13:19:58 2020 +0200
pcs commands on host: cinder backup/volume
This implements the creation of the haproxy bundle on the host.
The testing protocol used is documented in the depends-on.
The reason for adding a post_update task is that during a minor update
the deployment tasks are not run during the node update procedure but
only during the final converge. So we ran the role again there to make
sure that any config change will trigger a restart during the minor
update, so the disruption is only local to the single node being
updated. If we did not do this a final converge could potentially
trigger a global restart of HA bundles which would be quite disruptive.
Depends-On: Iaa7e89f0d25221
Change-Id: Ia4399b632257e6
Related-Bug: #1863442
OpenStack Infra (hudson-openstack) wrote : | #23 |
Reviewed: https:/
Committed: https:/
Submitter: Zuul
Branch: master
commit 83d1cb585268177
Author: Michele Baldessari <email address hidden>
Date: Tue Aug 11 15:04:36 2020 +0200
pcs commands on host: redis bundle
This implements the creation of the redis bundle on the host.
The testing protocol used is documented in the depends-on.
The full rationale is contained in the LP bug.
The reason for adding a post_update task is that during a minor update
the deployment tasks are not run during the node update procedure but
only during the final converge. So we ran the role again there to make
sure that any config change will trigger a restart during the minor
update, so the disruption is only local to the single node being
updated. If we did not do this a final converge could potentially
trigger a global restart of HA bundles which would be quite disruptive.
Related-Bug: #1863442
Depends-On: Iaa7e89f0d25221
Change-Id: I5ce8367363d535
OpenStack Infra (hudson-openstack) wrote : | #24 |
Reviewed: https:/
Committed: https:/
Submitter: Zuul
Branch: master
commit 4e76e1fb72ce1af
Author: Michele Baldessari <email address hidden>
Date: Fri May 1 13:23:46 2020 +0200
pcs commands on host: ovn dbs
We add the following depends on because at this stage ovn will be
created on the host and the resource-agent won't be present so
we have to be able to tell pcs to ignore the non existance of the
ovn-dbs ocfg resource on the host.
Depends-On: If9048196b5c03e
Depends-On: Iaa7e89f0d25221
Change-Id: I918b6c16db6ed7
Related-Bug: #1863442
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to tripleo-ansible (stable/ussuri) | #25 |
Related fix proposed to branch: stable/ussuri
Review: https:/
OpenStack Infra (hudson-openstack) wrote : Related fix merged to tripleo-ansible (stable/ussuri) | #26 |
Reviewed: https:/
Committed: https:/
Submitter: Zuul
Branch: stable/ussuri
commit 8daba50a383d303
Author: Michele Baldessari <email address hidden>
Date: Fri May 1 12:43:46 2020 +0200
Add tripleo_ha_wrapper role
This role is meant to help moving the HA pcs tasks to pure deployment
steps run on the host and not in containers any more. We tested this
ansible review with the THT patches at
https:/
Here is the testing protocol that was used in order to verify we did
not break anything:
1) Deployed overcloud + tempest
2) Ran a successful redeploy
2.1) Verified that a redeploy restarted containers globally when the
2.2) Verified that a redeploy did not restart containers which config
did not change
2.3) Verified that the ordering of restarting bundles allows for
brownfield deployments (i.e. first the bundle is restarted due
to new bind-mounts and then the bundle is restarted with new
2.4) Verified that converge step after minor update is successful
3) Ran an initial successful minor update
3.1) Verified that the new image is indeed being run by pacemaker
(i.e. :pcmklatest tag is applied to the new containers)
3.2) Verified that after converge no container is restarted
4) Re-run a subsequent successful noop minor update
4.1) Verify that containers are not restarted a second time after
the cluster is stopped and started
Depends-On: I367cf4b65300be
Change-Id: Iaa7e89f0d25221
Related-Bug: #1863442
(cherry picked from commit d5782863b8b1da7
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to tripleo-ansible (stable/train) | #27 |
Related fix proposed to branch: stable/train
Review: https:/
OpenStack Infra (hudson-openstack) wrote : Related fix merged to tripleo-ansible (stable/train) | #28 |
Reviewed: https:/
Committed: https:/
Submitter: Zuul
Branch: stable/train
commit 53ccc3fddf97d6e
Author: Michele Baldessari <email address hidden>
Date: Fri May 1 12:43:46 2020 +0200
Add tripleo_ha_wrapper role
This role is meant to help moving the HA pcs tasks to pure deployment
steps run on the host and not in containers any more. We tested this
ansible review with the THT patches at
https:/
Here is the testing protocol that was used in order to verify we did
not break anything:
1) Deployed overcloud + tempest
2) Ran a successful redeploy
2.1) Verified that a redeploy restarted containers globally when the
2.2) Verified that a redeploy did not restart containers which config
did not change
2.3) Verified that the ordering of restarting bundles allows for
brownfield deployments (i.e. first the bundle is restarted due
to new bind-mounts and then the bundle is restarted with new
2.4) Verified that converge step after minor update is successful
3) Ran an initial successful minor update
3.1) Verified that the new image is indeed being run by pacemaker
(i.e. :pcmklatest tag is applied to the new containers)
3.2) Verified that after converge no container is restarted
4) Re-run a subsequent successful noop minor update
4.1) Verify that containers are not restarted a second time after
the cluster is stopped and started
Depends-On: I367cf4b65300be
NB: Not 100% clean in zuul.d/
Small tweaks/renames to molecule folder for CI
Change-Id: Iaa7e89f0d25221
Related-Bug: #1863442
(cherry picked from commit d5782863b8b1da7
(cherry picked from commit 8daba50a383d303
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to tripleo-heat-templates (stable/ussuri) | #29 |
Related fix proposed to branch: stable/ussuri
Review: https:/
OpenStack Infra (hudson-openstack) wrote : | #30 |
Related fix proposed to branch: stable/ussuri
Review: https:/
OpenStack Infra (hudson-openstack) wrote : | #31 |
Related fix proposed to branch: stable/ussuri
Review: https:/
OpenStack Infra (hudson-openstack) wrote : | #32 |
Related fix proposed to branch: stable/ussuri
Review: https:/
OpenStack Infra (hudson-openstack) wrote : | #33 |
Related fix proposed to branch: stable/ussuri
Review: https:/
OpenStack Infra (hudson-openstack) wrote : | #34 |
Related fix proposed to branch: stable/ussuri
Review: https:/
OpenStack Infra (hudson-openstack) wrote : Change abandoned on tripleo-heat-templates (stable/ussuri) | #35 |
Change abandoned by Michele Baldessari (<email address hidden>) on branch: stable/ussuri
Review: https:/
Reason: don't want to merge this before we understand the upgrade failures on the other pcs host patches
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to tripleo-heat-templates (master) | #36 |
Related fix proposed to branch: master
Review: https:/
OpenStack Infra (hudson-openstack) wrote : | #37 |
Related fix proposed to branch: master
Review: https:/
OpenStack Infra (hudson-openstack) wrote : Related fix merged to tripleo-heat-templates (master) | #38 |
Reviewed: https:/
Committed: https:/
Submitter: Zuul
Branch: master
commit a584099b9d581f4
Author: Michele Baldessari <email address hidden>
Date: Wed Sep 9 19:48:14 2020 +0200
Make sure we noop the Rabbitmq_ providers
This is needed because the Rabbitmq_ providers might prefetch
the data from rabbitmq on the system and we do not want to run
any task on the host that is not strictly pcmk-related.
Co-Authored-By: Damien Ciabrini <email address hidden>
Change-Id: I0762b3ed7d881b
Related-Bug: #1863442
OpenStack Infra (hudson-openstack) wrote : | #39 |
Reviewed: https:/
Committed: https:/
Submitter: Zuul
Branch: master
commit 4b0cc36efe92659
Author: Michele Baldessari <email address hidden>
Date: Wed Sep 9 16:37:18 2020 +0200
Make sure we noop the Mysql_ providers
This is needed because the Mysql_ providers will prefetch
the the mysql users if facter finds the '/bin/mysql' executable
on the system and we do not want to run any mysql task on the host
directly.
Co-Authored-By: Damien Ciabrini <email address hidden>
Change-Id: Ic6c65e68493681
Related-Bug: #1863442
OpenStack Infra (hudson-openstack) wrote : Related fix merged to tripleo-heat-templates (stable/ussuri) | #40 |
Reviewed: https:/
Committed: https:/
Submitter: Zuul
Branch: stable/ussuri
commit 3f8c22b5614253d
Author: Michele Baldessari <email address hidden>
Date: Fri May 1 13:07:13 2020 +0200
pcs commands on host: haproxy bundle
This implements the creation of the haproxy bundle on the host.
The testing protocol used is documented in the depends-on.
The reason for adding a post_update task is that during a minor update
the deployment tasks are not run during the node update procedure but
only during the final converge. So we ran the role again there to make
sure that any config change will trigger a restart during the minor
update, so the disruption is only local to the single node being
updated. If we did not do this a final converge could potentially
trigger a global restart of HA bundles which would be quite disruptive.
NB: We had to remove a few tags that were present befaure because we want
to avoid that puppet checks for /etc/haproxy/
on the host.
Related-Bug: #1863442
Depends-On: Iaa7e89f0d25221
Change-Id: I20dc57a02a315c
(cherry picked from commit a11622359002597
OpenStack Infra (hudson-openstack) wrote : | #41 |
Reviewed: https:/
Committed: https:/
Submitter: Zuul
Branch: stable/ussuri
commit be1fee4af6a5ed7
Author: Michele Baldessari <email address hidden>
Date: Fri May 1 13:12:46 2020 +0200
pcs commands on host: mysql
This implements the creation of the haproxy bundle on the host.
The testing protocol used is documented in the depends-on.
The reason for adding a post_update task is that during a minor update
the deployment tasks are not run during the node update procedure but
only during the final converge. So we ran the role again there to make
sure that any config change will trigger a restart during the minor
update, so the disruption is only local to the single node being
updated. If we did not do this a final converge could potentially
trigger a global restart of HA bundles which would be quite disruptive.
NB: in this case we keep the container init_bundle (renamed to
wait_bundle) around just use it to wait for galera to be up.
Depends-On: Iaa7e89f0d25221
NB: We also merged Ic6c65e68493681
in this review so that the ussuri upgrade job can pass
(Make sure we noop the Mysql_ providers)
Change-Id: Ie14819b66cecdb
Related-Bug: #1863442
(cherry picked from commit da3d5e80562f7aa
OpenStack Infra (hudson-openstack) wrote : | #42 |
Reviewed: https:/
Committed: https:/
Submitter: Zuul
Branch: stable/ussuri
commit 06209fa28f35f96
Author: Michele Baldessari <email address hidden>
Date: Fri May 1 13:28:36 2020 +0200
pcs commands on host: manila-share
This implements the creation of the manila-share bundle on the host.
The testing protocol used is documented in the depends-on.
The reason for adding a post_update task is that during a minor update
the deployment tasks are not run during the node update procedure but
only during the final converge. So we ran the role again there to make
sure that any config change will trigger a restart during the minor
update, so the disruption is only local to the single node being
updated. If we did not do this a final converge could potentially
trigger a global restart of HA bundles which would be quite disruptive.
Depends-On: Iaa7e89f0d25221
Change-Id: If5730dba4973a5
Related-Bug: #1863442
(cherry picked from commit a1bee391361cde4
OpenStack Infra (hudson-openstack) wrote : | #43 |
Reviewed: https:/
Committed: https:/
Submitter: Zuul
Branch: stable/ussuri
commit 4f56371f912f468
Author: Michele Baldessari <email address hidden>
Date: Fri May 1 13:19:58 2020 +0200
pcs commands on host: cinder backup/volume
This implements the creation of the haproxy bundle on the host.
The testing protocol used is documented in the depends-on.
The reason for adding a post_update task is that during a minor update
the deployment tasks are not run during the node update procedure but
only during the final converge. So we ran the role again there to make
sure that any config change will trigger a restart during the minor
update, so the disruption is only local to the single node being
updated. If we did not do this a final converge could potentially
trigger a global restart of HA bundles which would be quite disruptive.
Depends-On: Iaa7e89f0d25221
Change-Id: Ia4399b632257e6
Related-Bug: #1863442
(cherry picked from commit f37f21f31df12e5
OpenStack Infra (hudson-openstack) wrote : | #44 |
Reviewed: https:/
Committed: https:/
Submitter: Zuul
Branch: stable/ussuri
commit ec70716ad874d36
Author: Michele Baldessari <email address hidden>
Date: Fri May 1 13:23:46 2020 +0200
pcs commands on host: ovn dbs
We add the following depends on because at this stage ovn will be
created on the host and the resource-agent won't be present so
we have to be able to tell pcs to ignore the non existance of the
ovn-dbs ocfg resource on the host.
Depends-On: If9048196b5c03e
Depends-On: Iaa7e89f0d25221
Change-Id: I918b6c16db6ed7
Related-Bug: #1863442
(cherry picked from commit 4e76e1fb72ce1af
OpenStack Infra (hudson-openstack) wrote : | #45 |
Reviewed: https:/
Committed: https:/
Submitter: Zuul
Branch: stable/ussuri
commit 0eb998930011ee3
Author: Michele Baldessari <email address hidden>
Date: Tue Aug 11 15:04:36 2020 +0200
pcs commands on host: redis bundle
This implements the creation of the redis bundle on the host.
The testing protocol used is documented in the depends-on.
The full rationale is contained in the LP bug.
The reason for adding a post_update task is that during a minor update
the deployment tasks are not run during the node update procedure but
only during the final converge. So we ran the role again there to make
sure that any config change will trigger a restart during the minor
update, so the disruption is only local to the single node being
updated. If we did not do this a final converge could potentially
trigger a global restart of HA bundles which would be quite disruptive.
Related-Bug: #1863442
Depends-On: Iaa7e89f0d25221
Change-Id: I5ce8367363d535
(cherry picked from commit 83d1cb585268177
OpenStack Infra (hudson-openstack) wrote : | #46 |
Reviewed: https:/
Committed: https:/
Submitter: Zuul
Branch: stable/ussuri
commit 0ad2358067c0aff
Author: Michele Baldessari <email address hidden>
Date: Fri May 1 13:42:26 2020 +0200
pcs commands on host: rabbitmq
This implements the creation of the rabbitmq bundle on the host.
The testing protocol used is documented in the depends-on.
The reason for adding a post_update task is that during a minor update
the deployment tasks are not run during the node update procedure but
only during the final converge. So we ran the role again there to make
sure that any config change will trigger a restart during the minor
update, so the disruption is only local to the single node being
updated. If we did not do this a final converge could potentially
trigger a global restart of HA bundles which would be quite disruptive.
NB: The init_bundle now has become the wait_bundle
as it just waits for rabbitmq to be up and functional
NB2: Slightly not 100% due to different context (missing
I1873baab43
NB3: We cherry-picked I0762b3ed7d881b
inside this review in order for the upgrade job to pass
(Make sure we noop the Rabbitmq_ providers)
Related-Bug: #1863442
Depends-On: Iaa7e89f0d25221
Change-Id: I853bcf354f64ef
(cherry picked from commit 29bc3de01dc39bb
Changed in tripleo: | |
milestone: | victoria-3 → wallaby-1 |
Changed in tripleo: | |
milestone: | wallaby-1 → wallaby-2 |
Changed in tripleo: | |
milestone: | wallaby-2 → wallaby-3 |
Marios Andreou (marios-b) wrote : | #47 |
This is an automated action. Bug status has been set to 'Incomplete' and target milestone has been removed due to inactivity. If you disagree please re-set these values and reach out to us on freenode #tripleo
Changed in tripleo: | |
milestone: | wallaby-3 → none |
status: | Triaged → Incomplete |
Launchpad Janitor (janitor) wrote : | #48 |
[Expired for tripleo because there has been no activity for 60 days.]
Changed in tripleo: | |
status: | Incomplete → Expired |
Related fix proposed to branch: master /review. opendev. org/707953
Review: https:/