Skip to content
This repository has been archived by the owner on Feb 29, 2024. It is now read-only.

Commit

Permalink
Revert rolling certificate updates for HA services
Browse files Browse the repository at this point in the history
Currently galera and ovn require a coordinated restart across
the controller node when certmonger determines the certificate
for a node has expired and it needs to regenerate it.

But right now, when the tripleo certmonger puppet module is
called to assert to state of the certificates, it ends up
regenerating new certificate unconditionally. So the galera and
ovn get restarted on stack update, even when there is no need to.

To mitigate these unecessary restarts, disable the post-action
for now until we fix the behaviour of tripleo's certmonger puppet
module. This has the side effect that services won't get restarted
automatically if no stack update takes place until the certificate
expiration date is reached.

Related-Bug: #1906505

Change-Id: I17f1364932e43b8487515084e41b525e186888db
  • Loading branch information
dciabrin committed Dec 2, 2020
1 parent d04421d commit 8b16911
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 4 deletions.
2 changes: 0 additions & 2 deletions deployment/database/mysql-pacemaker-puppet.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -173,8 +173,6 @@ outputs:
if:
- internal_tls_enabled
-
tripleo::certmonger::mysql::postsave_cmd:
/usr/bin/certmonger-ha-resource-refresh.sh mysql galera galera-bundle Master
tripleo::profile::pacemaker::database::mysql_bundle::ca_file:
get_param: InternalTLSCAFile
- {}
Expand Down
2 changes: 0 additions & 2 deletions deployment/ovn/ovn-dbs-pacemaker-puppet.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -170,8 +170,6 @@ outputs:
- if:
- internal_tls_enabled
- generate_service_certificates: true
tripleo::certmonger::ovn_dbs::postsave_cmd:
/usr/bin/certmonger-ha-resource-refresh.sh ovn_dbs ovndb_servers ovn-dbs-bundle Slave Master
tripleo::profile::pacemaker::ovn_dbs_bundle::ca_file:
get_param: InternalTLSCAFile
tripleo::profile::base::neutron::agents::ovn::protocol: 'ssl'
Expand Down

0 comments on commit 8b16911

Please sign in to comment.