Fix deploy-edpm.yml crash on unpreprovisioned baremetal computes

The NFS and Ceph plays in deploy-edpm.yml target the computes
group with gather_facts enabled (the default). In architecture
deployments with preProvisioned=false, compute nodes are bare
libvirt domains with no OS until Ironic provisions them during
the kustomize_deploy stages. Ansible's implicit fact gathering
tries to SSH into these unreachable hosts, aborting the entire
playbook before the architecture deployment can run.

Add gather_facts: false to both the NFS and Ceph plays. For the
NFS play, insert an end_play guard when cifmw_architecture_scenario
is defined and move fact gathering after that guard. Architecture
deploys handle NFS via a pre_stage hook instead (see the companion
nfs-on-computes.yml hook playbook). The Ceph play already had an
end_play guard but Ansible was crashing on fact gathering before
reaching it.

Similarly, the "Fetch network facts" task in deploy_architecture.yml
delegates setup to every host in the inventory, including
unprovisioned computes. Skip computes when
cifmw_edpm_deploy_pre_provisioned is false.

This fixes all architecture-uni03gamma-deploy-bm-* jobs which have
been consistently failing since the deploy-bm variant was introduced.

Related-Issue: ANVIL-109
Co-authored-by: Cursor <cursoragent@cursor.com>

Signed-off-by: Roberto Alfieri <ralfieri@redhat.com>

Author: rebtoor

deploy-edpm.yml

@@ -87,21 +87,30 @@
 
 - name: Deploy NFS server on target nodes
   become: true
+  gather_facts: false
   hosts: "{{ groups[cifmw_nfs_target | default('computes')][0] | default([]) }}"
   tasks:
+    - name: End play early for architecture deploys
+      when:
+        - cifmw_architecture_scenario is defined
+      ansible.builtin.meta: end_play
+
+    - name: Gather facts for NFS deployment
+      ansible.builtin.setup:
+
     - name: Run cifmw_nfs role
       vars:
         nftables_path: /etc/nftables
         nftables_conf: /etc/sysconfig/nftables.conf
       when:
         - cifmw_edpm_deploy_nfs | default(false) | bool
-      ansible.builtin.import_role:
+      ansible.builtin.include_role:
         name: cifmw_nfs
 
 - name: Clear ceph target hosts facts to force refreshing in HCI deployments
+  gather_facts: false
   hosts: "{{ cifmw_ceph_target | default('computes')  }}"
   tasks:
-    # end_play will end only current play, not the main edpm-deploy.yml
     - name: Early end if architecture deploy
       when:
         - cifmw_architecture_scenario is defined

hooks/playbooks/nfs-on-computes.yml

@@ -0,0 +1,27 @@
+---
+# Deploy NFS on compute nodes for architecture-based deployments.
+#
+# In architecture flows with preProvisioned=false (baremetal/Ironic),
+# compute nodes are not reachable until the architecture stages
+# provision them. The standard NFS play in deploy-edpm.yml runs
+# before the architecture stages, so it must be skipped. This hook
+# playbook is designed to run as a pre_stage_run or post_stage_run
+# hook after the EDPM NodeSet reaches SetupReady (computes are
+# provisioned and SSH-reachable) but before the DataPlaneDeployment
+# configures services that depend on NFS.
+
+- name: Deploy NFS on compute nodes (architecture hook)
+  become: true
+  hosts: "{{ groups[cifmw_nfs_target | default('computes')][0] | default([]) }}"
+  tasks:
+    - name: Skip if NFS deployment is not requested
+      when:
+        - not (cifmw_edpm_deploy_nfs | default(false) | bool)
+      ansible.builtin.meta: end_play
+
+    - name: Run cifmw_nfs role
+      vars:
+        nftables_path: /etc/nftables
+        nftables_conf: /etc/sysconfig/nftables.conf
+      ansible.builtin.include_role:
+        name: cifmw_nfs