Use include_role in enter/exit maintenance

import_role can result in confusing variable shadowing issues.

Author: markgoddard

roles/enter_maintenance/tasks/enter.yml

@@ -1,6 +1,6 @@
 ---
 - name: Check if host can enter maintenance mode
-  ansible.builtin.import_role:
+  ansible.builtin.include_role:
     name: stackhpc.cephadm.commands
   vars:
     cephadm_commands:
@@ -18,15 +18,15 @@
     fail_msg: "{{ cephadm_commands_result.results[0].stderr }}"
 
 - name: Fail over Ceph manager
-  ansible.builtin.import_role:
+  ansible.builtin.include_role:
     name: stackhpc.cephadm.commands
   vars:
     cephadm_commands:
       - "mgr fail"
   when: '"Cannot stop active Mgr daemon" in cephadm_commands_result.results[0].stderr'
 
 - name: Ensure host is in maintenance mode
-  ansible.builtin.import_role:
+  ansible.builtin.include_role:
     name: stackhpc.cephadm.commands
   vars:
     cephadm_commands:

roles/enter_maintenance/tasks/main.yml

@@ -8,7 +8,7 @@
       whether it is safe to stop them.
 
 - name: List hosts in maintenance
-  ansible.builtin.import_role:
+  ansible.builtin.include_role:
     name: stackhpc.cephadm.commands
   vars:
     cephadm_commands:

roles/exit_maintenance/tasks/exit.yml

@@ -1,6 +1,6 @@
 ---
 - name: Ensure host has exited maintenance mode
-  ansible.builtin.import_role:
+  ansible.builtin.include_role:
     name: stackhpc.cephadm.commands
   vars:
     cephadm_commands:

roles/exit_maintenance/tasks/main.yml

@@ -7,7 +7,7 @@
       Hosts must be removed from maintenance one at a time.
 
 - name: List hosts
-  ansible.builtin.import_role:
+  ansible.builtin.include_role:
     name: stackhpc.cephadm.commands
   vars:
     cephadm_commands: