Skip to content

Commit

Permalink
Move and add retries to ci_local_storage
Browse files Browse the repository at this point in the history
SSH connection when fetching hostnames seems to be lost
randomly. Add SSH retries and moving the role before
OSP operators deployment may help the connection role
to succeed.
  • Loading branch information
pablintino authored and openshift-merge-bot[bot] committed Jun 6, 2024
1 parent 5dc1865 commit 327f2db
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 9 deletions.
16 changes: 8 additions & 8 deletions playbooks/06-deploy-architecture.yml
Original file line number Diff line number Diff line change
Expand Up @@ -135,6 +135,14 @@
tags:
- edpm_bootstrap

- name: Configure Storage Class
ansible.builtin.import_role:
name: ci_local_storage
when: not cifmw_use_lvms | default(false)
tags:
- storage
- edpm_bootstrap

- name: Deploy OSP operators
ansible.builtin.import_role:
name: kustomize_deploy
Expand All @@ -152,14 +160,6 @@
- update_containers
- edpm_bootstrap

- name: Configure Storage Class
ansible.builtin.import_role:
name: ci_local_storage
when: not cifmw_use_lvms | default(false)
tags:
- storage
- edpm_bootstrap

- name: Configure LVMS Storage Class
ansible.builtin.include_role:
name: ci_lvms_storage
Expand Down
4 changes: 3 additions & 1 deletion roles/ci_local_storage/tasks/fetch_names.yml
Original file line number Diff line number Diff line change
Expand Up @@ -10,9 +10,11 @@
register: cifmw_ci_local_storage_k8s_nodes_out

- name: Fetch hostnames for all hosts
register: _hostnames
vars:
ansible_ssh_retries: 5
ansible.builtin.command:
cmd: hostname
register: _hostnames
delegate_to: "{{ item }}"
loop: "{{ hostvars.keys() | list }}"

Expand Down

0 comments on commit 327f2db

Please sign in to comment.