Replace run_once: _oraswgi_meta_configure_cluster with when condition

The execution of some roles on multiple hosts was not possivle, because some tasks had run_once to limit the execution on 1 node in RAC environments only. This does NOT enable the installation of multiple RAC cluster in 1 playbook run.
oravirt · Mar 16, 2024 · 2eb2f28 · 2eb2f28
1 parent f341e40
commit 2eb2f28
Show file tree

Hide file tree

Showing 32 changed files with 296 additions and 191 deletions.
diff --git a/changelogs/fragments/run_once.yml b/changelogs/fragments/run_once.yml
@@ -0,0 +1,3 @@
+---
+minor_changes:
+  - "Replace run_once: _oraswgi_meta_configure_cluster with when condition ()"
diff --git a/roles/oradb_manage_db/tasks/manage-db.yml b/roles/oradb_manage_db/tasks/manage-db.yml
@@ -90,8 +90,9 @@
     # loop_var dbh was replaced by odb in outer loop
     # => no need to change all dbca-templates...
     dbh: "{{ odb }}"
-  run_once: "{{ _oraswgi_meta_configure_cluster }}"
-  when: odb.state == 'present'
+  when:
+    - odb.state == 'present'
+    - _orasw_meta_primary_node | bool
   tags:
     - create_db
     - dbcatemplate
@@ -125,10 +126,11 @@
         initparams: "{{ _oradb_manage_db_init_params_list | default(omit) }}"
         output: verbose
         state: "{{ odb.state }}"
-      run_once: "{{ _oraswgi_meta_configure_cluster }}"
       become: true
       become_user: "{{ oracle_user }}"
-      when: odb.state == 'present'
+      when:
+        - odb.state == 'present'
+        - _orasw_meta_primary_node | bool
       register: dbca_create
 
   rescue:
@@ -159,8 +161,9 @@
 
 - ansible.builtin.debug:
     msg: "Removing DB - {{ odb.oracle_db_name }}, ORACLE_HOME - {{ _oracle_home_db }}"  # noqa name[missing]
-  run_once: "{{ _oraswgi_meta_configure_cluster }}"
-  when: odb.state == 'absent'
+  when:
+    - odb.state == 'absent'
+    - _orasw_meta_primary_node | bool
 
 - name: manage_db | remove database
   opitzconsulting.ansible_oracle.oracle_db:
@@ -171,10 +174,11 @@
     db_name: "{{ odb.oracle_db_name }}"
     output: verbose
     state: "{{ odb.state }}"
-  run_once: "{{ _oraswgi_meta_configure_cluster }}"
   become: true
   become_user: "{{ oracle_user }}"
-  when: odb.state == 'absent'
+  when:
+    - odb.state == 'absent'
+    - _orasw_meta_primary_node | bool
   register: dbca_remove
   tags: remove_db,dbca
 

diff --git a/roles/oradb_manage_grants/tasks/main.yml b/roles/oradb_manage_grants/tasks/main.yml
@@ -21,11 +21,11 @@
       skip_missing: true
   environment: "{{ _oracle_env_cdb }}"
   when:
+    - _orasw_meta_primary_node | bool
     - oracle_databases is defined
     - odb.0.state == 'present'
     - oracledb_facts[_oracle_db_unique_name]['state'] | default('') == 'present'
     - (odb.1.grants is defined or odb.1.object_privs is defined)
-  run_once: "{{ _oraswgi_meta_configure_cluster }}"
   become: true
   become_user: "{{ oracle_user }}"
   loop_control:
@@ -62,6 +62,7 @@
       skip_missing: true
   environment: "{{ _oracle_env_pdb }}"
   when:
+    - _orasw_meta_primary_node | bool
     - oracle_pdbs is defined
     - opdb.0 is defined
     - opdb.0.state == 'present'
@@ -70,7 +71,6 @@
     - >-
       oracle_databases | selectattr('oracle_db_name', 'equalto', opdb.0.cdb)
       | map(attribute='state') | list | first == 'present'
-  run_once: "{{ _oraswgi_meta_configure_cluster }}"
   become: true
   become_user: "{{ oracle_user }}"
   loop_control:
@@ -110,11 +110,11 @@
       skip_missing: true
   environment: "{{ _oracle_env }}"
   when:
+    - _orasw_meta_primary_node | bool
     - oracle_databases is defined
     - odb.0.state == 'present'
     - oracledb_facts[_oracle_db_unique_name]['state'] | default('') == 'present'
     - (odb.1.grants is defined or odb.1.object_privs is defined)
-  run_once: "{{ _oraswgi_meta_configure_cluster }}"
   become: true
   become_user: "{{ oracle_user }}"
   loop_control:
@@ -151,6 +151,7 @@
       skip_missing: true
   environment: "{{ _oracle_env_pdb }}"
   when:
+    - _orasw_meta_primary_node | bool
     - oracle_pdbs is defined
     - opdb.0 is defined
     - opdb.0.state == 'present'
@@ -159,7 +160,6 @@
     - >-
       oracle_databases | selectattr('oracle_db_name', 'equalto', opdb.0.cdb)
       | map(attribute='state') | list | first == 'present'
-  run_once: "{{ _oraswgi_meta_configure_cluster }}"
   become: true
   become_user: "{{ oracle_user }}"
   loop_control:

diff --git a/roles/oradb_manage_parameters/tasks/main.yml b/roles/oradb_manage_parameters/tasks/main.yml
@@ -14,13 +14,13 @@
     state: "{{ odb.1.state }}"
     scope: "{{ odb.1.scope | default ('both') }}"
   environment: "{{ _oracle_env }}"
-  run_once: "{{ _oraswgi_meta_configure_cluster }}"
   with_subelements:
     - "{{ oracle_databases }}"
     - init_parameters
     - flags:
       skip_missing: true
   when:
+    - _orasw_meta_primary_node | bool
     - oracle_databases is defined
     - odb.0.state == 'present'
     - oracledb_facts[_oracle_db_unique_name]['state'] | default('') == 'present'
@@ -57,13 +57,13 @@
     state: "{{ opdb.1.state }}"
     scope: "{{ opdb.1.scope | default ('both') }}"
   environment: "{{ _oracle_env_pdb }}"
-  run_once: "{{ _oraswgi_meta_configure_cluster }}"
   with_subelements:
     - "{{ oracle_pdbs }}"
     - init_parameters
     - flags:
       skip_missing: true
   when:
+    - _orasw_meta_primary_node | bool
     - oracle_pdbs is defined
     - opdb.0.state == 'present'
     - oracledb_facts[_oracle_db_unique_name]['state'] | default('') == 'present'
@@ -99,11 +99,11 @@
     sid: "{{ _oracle_db_instance_name | default(omit) }}"
     state: "restarted"
     force_restart: false
-  run_once: "{{ _oraswgi_meta_configure_cluster }}"
   with_nested:
     - "{{ oracle_databases }}"
     - ""  # dummy to force odb.0 instead of item.
   when:
+    - _orasw_meta_primary_node | bool
     - odb.0.restart_spparameter_changed | default(true)
     - oracledb_facts[_oracle_db_unique_name]['state'] | default('') == 'present'
     - restart_spparameter_changed

diff --git a/roles/oradb_manage_pdb/tasks/main.yml b/roles/oradb_manage_pdb/tasks/main.yml
@@ -25,7 +25,6 @@
         port: "{{ listener_port_template }}"
         state: "{{ odb[1].state | default(omit) }}"
       environment: "{{ _oracle_env }}"
-      run_once: "{{ _oraswgi_meta_configure_cluster }}"
       with_nested:
         - "{{ oracle_databases }}"
         - "{{ oracle_pdbs }}"
@@ -39,6 +38,7 @@
           db_state: {{ oracledb_facts[_oracle_db_unique_name]['state'] | default('') }}
         loop_var: odb
       when:
+        - _orasw_meta_primary_node | bool
         - oracle_databases is defined
         - odb[0].oracle_db_name
         - oracle_pdbs is defined

diff --git a/roles/oradb_manage_profiles/tasks/main.yml b/roles/oradb_manage_profiles/tasks/main.yml
@@ -37,11 +37,11 @@
           attributes: {{ odb.1.attributes }}
           db_state: {{ oracledb_facts[_oracle_db_unique_name]['state'] | default('') }}
       when:
+        - _orasw_meta_primary_node | bool
         - oracle_databases is defined
         - odb.0.state == 'present'
         - oracledb_facts[_oracle_db_unique_name]['state'] | default('') == 'present'
         - odb.1 is defined
-      run_once: "{{ _oraswgi_meta_configure_cluster }}"
       become: true
       become_user: "{{ oracle_user }}"
       vars:
@@ -88,6 +88,7 @@
           attributes: {{ opdb.1.attributes }}
           cdb_state: {{ oracledb_facts[_oracle_db_unique_name]['state'] | default('') }}
       when:
+        - _orasw_meta_primary_node | bool
         - oracle_pdbs is defined
         - opdb.0 is defined
         - opdb.0.state == 'present'
@@ -96,7 +97,6 @@
         - >-
           oracle_databases | selectattr('oracle_db_name', 'equalto', opdb.0.cdb)
           | map(attribute='state') | list | first == 'present'
-      run_once: "{{ _oraswgi_meta_configure_cluster }}"
       become: true
       become_user: "{{ oracle_user }}"
       vars:

diff --git a/roles/oradb_manage_redo/tasks/main.yml b/roles/oradb_manage_redo/tasks/main.yml
@@ -23,9 +23,9 @@
         size: "{{ odb.redolog_size }}"
         groups: "{{ odb.redolog_groups }}"
       environment: "{{ _oracle_env }}"
-      run_once: "{{ _oraswgi_meta_configure_cluster }}"
       loop: "{{ oracle_databases }}"
       when:
+        - _orasw_meta_primary_node | bool
         - oracle_databases is defined
         - odb.redolog_size is defined
         - odb.redolog_groups is defined

diff --git a/roles/oradb_manage_roles/README.md b/roles/oradb_manage_roles/README.md
@@ -19,7 +19,9 @@ Manage roles in Oracle
 
 ## Discovered Tags
 
-**_users,grants_**
+**_grants_**
+
+**_users_**
 
 
 ## Dependencies

diff --git a/roles/oradb_manage_roles/tasks/main.yml b/roles/oradb_manage_roles/tasks/main.yml
@@ -17,6 +17,7 @@
       skip_missing: true
   environment: "{{ _oracle_env }}"
   when:
+    - _orasw_meta_primary_node | bool
     - oracle_databases is defined
     - odb.0.state == 'present'
     - oracledb_facts[_oracle_db_unique_name]['state'] | default('') == 'present'
@@ -30,10 +31,11 @@
       role {{ odb.1.name }}
       state {{ odb.1.state }}
       db_state: {{ oracledb_facts[_oracle_db_unique_name]['state'] | default('') }}
-  run_once: "{{ _oraswgi_meta_configure_cluster }}"
   become: true
   become_user: "{{ oracle_user }}"
-  tags: users,grants
+  tags:
+    - users
+    - grants
 
 - name: Manage roles (pdb)
   opitzconsulting.ansible_oracle.oracle_role:
@@ -52,6 +54,7 @@
       skip_missing: true
   environment: "{{ _oracle_env_pdb }}"
   when:
+    - _orasw_meta_primary_node | bool
     - oracle_pdbs is defined
     - opdb.0 is defined
     - opdb.0.state == 'present'
@@ -69,7 +72,8 @@
       role: {{ opdb.1.name }},
       state: {{ opdb.1.state }}
       cdb_state: {{ oracledb_facts[_oracle_db_unique_name]['state'] | default('') }}
-  run_once: "{{ _oraswgi_meta_configure_cluster }}"
   become: true
   become_user: "{{ oracle_user }}"
-  tags: users,grants
+  tags:
+    - users
+    - grants
diff --git a/roles/oradb_manage_services/README.md b/roles/oradb_manage_services/README.md
@@ -20,9 +20,11 @@ Manage services in Oracle
 
 ## Discovered Tags
 
-**_create_service,services_**
+**_create_service_**
 
-**_start_service,services_**
+**_services_**
+
+**_start_service_**
 
 ## Open Tasks