Skip to content

Commit

Permalink
Correct mariadb restart handlers
Browse files Browse the repository at this point in the history
This behaviour was changed 65325d9 but
does not work for mariadb and mariadb-clustercheck. A fix was included
in master as part of 23413d4, let's do
the same in our fork.

Change-Id: Idfb5b814e16056d439cf1e1318f86bbcea82d981
  • Loading branch information
priteau authored and MoteHue committed Jan 31, 2025
1 parent b4b9138 commit 30fa3fa
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
4 changes: 2 additions & 2 deletions ansible/roles/mariadb/handlers/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -60,7 +60,7 @@
- groups[mariadb_shard_group + '_port_alive_True'] is defined
- inventory_hostname in groups[mariadb_shard_group + '_port_alive_True']
- kolla_action != "config"
listen: restart mariadb
listen: Restart mariadb container

- name: Start MariaDB on new nodes
group_by:
Expand All @@ -70,7 +70,7 @@
- groups[mariadb_shard_group + '_port_alive_False'] is defined
- inventory_hostname in groups[mariadb_shard_group + '_port_alive_False']
- kolla_action != "config"
listen: restart mariadb
listen: Restart mariadb container

- name: Restart mariadb-clustercheck container
vars:
Expand Down
4 changes: 2 additions & 2 deletions ansible/roles/mariadb/tasks/config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@
become: true
with_dict: "{{ mariadb_services | select_services_enabled_and_mapped_to_host }}"
notify:
- "restart {{ item.key }}"
- "Restart {{ item.key }} container"

- name: Copying over config.json files for mariabackup
vars:
Expand Down Expand Up @@ -72,4 +72,4 @@
become: true
when: service | service_enabled_and_mapped_to_host
notify:
- restart mariadb
- Restart mariadb container

0 comments on commit 30fa3fa

Please sign in to comment.