From d0b93a631dec21e77ae095eea46dbd5c5ffd789c Mon Sep 17 00:00:00 2001
From: Michal Arbet <michal.arbet@ultimum.io>
Date: Tue, 6 Feb 2024 10:30:38 +0100
Subject: [PATCH] Fix mariadb role when used with check mode

This patch adds check_mode: false to tasks
in restart_services.yml which just checking
some WSREP status and if port is UP.

Closes-Bug: #2052501
Change-Id: I92a591900d85138a87991a18dd4339efd053ef1b
---
 ansible/roles/mariadb/tasks/restart_services.yml     | 2 ++
 releasenotes/notes/bug-2052501-6dfd9e5443fdc6d1.yaml | 5 +++++
 2 files changed, 7 insertions(+)
 create mode 100644 releasenotes/notes/bug-2052501-6dfd9e5443fdc6d1.yaml

diff --git a/ansible/roles/mariadb/tasks/restart_services.yml b/ansible/roles/mariadb/tasks/restart_services.yml
index aef0395fad..ca48e359ad 100644
--- a/ansible/roles/mariadb/tasks/restart_services.yml
+++ b/ansible/roles/mariadb/tasks/restart_services.yml
@@ -23,6 +23,7 @@
     timeout: 60
     search_regex: "MariaDB"
   register: check_mariadb_port
+  check_mode: false
   until: check_mariadb_port is success
   retries: 10
   delay: 6
@@ -41,6 +42,7 @@
   retries: 10
   delay: 6
   no_log: true
+  check_mode: false
   when:
     # NOTE(yoctozepto): we don't want to wait for new nodes to fully sync
     # with an existing cluster as this could take time
diff --git a/releasenotes/notes/bug-2052501-6dfd9e5443fdc6d1.yaml b/releasenotes/notes/bug-2052501-6dfd9e5443fdc6d1.yaml
new file mode 100644
index 0000000000..7e6116cbe3
--- /dev/null
+++ b/releasenotes/notes/bug-2052501-6dfd9e5443fdc6d1.yaml
@@ -0,0 +1,5 @@
+---
+fixes:
+  - |
+    Fixes mariadb role deployment when using Ansible check mode.
+    `LP#2052501 <https://bugs.launchpad.net/kolla-ansible/+bug/2052501>`__
-- 
GitLab