Skip to content

Commit

Permalink
Split edpm_baremetal into 2 tasks
Browse files Browse the repository at this point in the history
The documented process is to call devsetup/Makefile
edpm_baremetal_compute and then Makefile edpm_deploy_baremetal. This
change does that, and also aligns the task names with the compute
tasks in 06-deploy-edpm.yml.

Once this has landed, target edpm_baremetal can be deleted so there is
no coupling between the 2 Makefiles.
  • Loading branch information
steveb committed Oct 10, 2023
1 parent 909cfb2 commit 5f50788
Showing 1 changed file with 13 additions and 4 deletions.
17 changes: 13 additions & 4 deletions ci_framework/roles/edpm_deploy_baremetal/tasks/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -44,14 +44,23 @@
ansible.builtin.include_role:
name: set_openstack_containers

- name: Install Dataplane with baremetal
- name: Create virtual baremetal
vars:
make_edpm_baremetal_env: "{{ cifmw_edpm_deploy_baremetal_common_env |
make_edpm_baremetal_compute_env: "{{ cifmw_edpm_deploy_baremetal_common_env |
combine(cifmw_edpm_deploy_baremetal_make_openstack_env | from_yaml)}}"
make_edpm_baremetal_dryrun: "{{ cifmw_edpm_deploy_baremetal_dry_run }}"
make_edpm_baremetal_compute_dryrun: "{{ cifmw_edpm_deploy_baremetal_dry_run }}"
ansible.builtin.include_role:
name: 'install_yamls_makes'
tasks_from: 'make_edpm_baremetal'
tasks_from: 'make_edpm_baremetal_compute'

- name: Deploy EDPM with baremetal
vars:
make_edpm_deploy_baremetal_env: "{{ cifmw_edpm_deploy_baremetal_common_env |
combine(cifmw_edpm_deploy_baremetal_make_openstack_env | from_yaml)}}"
make_edpm_deploy_baremetal_dryrun: "{{ cifmw_edpm_deploy_baremetal_dry_run }}"
ansible.builtin.include_role:
name: 'install_yamls_makes'
tasks_from: 'make_edpm_deploy_baremetal'

- name: Wait for Ironic to be ready
when: not cifmw_edpm_deploy_baremetal_dry_run
Expand Down

0 comments on commit 5f50788

Please sign in to comment.