From 44e526556217a61bf755f417a79514fa8fd2ef7e Mon Sep 17 00:00:00 2001
From: Yacine31 <yacine.oumghar@gmail.com>
Date: Tue, 2 Dec 2025 16:32:09 +0100
Subject: [PATCH] Final cleanup of role defaults

- Remove remaining duplicate variables from role defaults
- Ensure all common variables are in group_vars/all.yml
- Clean up service configurations
---
 roles/oracle-db-postinstall/defaults/main.yml  |  7 +------
 roles/oracle-db-postinstall/tasks/services.yml |  2 +-
 roles/oracle-db-preinstall/defaults/main.yml   | 11 +----------
 3 files changed, 3 insertions(+), 17 deletions(-)

diff --git a/roles/oracle-db-postinstall/defaults/main.yml b/roles/oracle-db-postinstall/defaults/main.yml
index 8b10cb8..3c0ab6a 100644
--- a/roles/oracle-db-postinstall/defaults/main.yml
+++ b/roles/oracle-db-postinstall/defaults/main.yml
@@ -1,8 +1,3 @@
 ---
 # Variables par défaut surchargeables pour oracle-db-postinstall
-
-scripts_dir: "/home/oracle/scripts"
-
-oracle_version: "19.0.0"
-oracle_base: "/u01/app/oracle"
-oracle_home: "{{ oracle_base }}/product/{{oracle_version}}/dbhome_1"
\ No newline at end of file
+# Les Variables sont lues depuis le fichier group_vars/all.yml
\ No newline at end of file
diff --git a/roles/oracle-db-postinstall/tasks/services.yml b/roles/oracle-db-postinstall/tasks/services.yml
index 0d56e34..d61ab7c 100644
--- a/roles/oracle-db-postinstall/tasks/services.yml
+++ b/roles/oracle-db-postinstall/tasks/services.yml
@@ -14,8 +14,8 @@
 - name: Activation du service de démarrage automatique
   ansible.builtin.systemd:
     name: dbora
-    daemon_reload: yes
     enabled: yes
+  notify: Recharger systemd
   tags: dbora_init
 
 - name: Copie du script de nettoyage ADR Oracle
diff --git a/roles/oracle-db-preinstall/defaults/main.yml b/roles/oracle-db-preinstall/defaults/main.yml
index b068c93..edbdc03 100644
--- a/roles/oracle-db-preinstall/defaults/main.yml
+++ b/roles/oracle-db-preinstall/defaults/main.yml
@@ -2,13 +2,4 @@
 # Variables par défaut surchargeables par l'utilisateur
 # Ces vars peuvent être redéfinies via --extra-vars ou inventory
 
-full_configuration: true
-secure_configuration: false
-
-oracle_version: "19.0.0"
-oracle_base: "/u01/app/oracle"
-oracle_home: "{{ oracle_base }}/product/{{oracle_version}}/dbhome_1"
-oracle_inventory: "/u01/app/oraInventory"
-oracle_sources: "/u01/sources"
-oracle_oradata: "/u02/oradata/"
-oracle_fra: "/u03/fast_recovery_area/"
\ No newline at end of file
+# Les variables sont lues depuis le fichiers group_vars/all.yml
\ No newline at end of file