Skip to content

Commit

Permalink
Ensure ocp is reachable from ocpbm before deploying
Browse files Browse the repository at this point in the history
Since we now run the shifstack deployment using the reproducer
by toggling `cifmw_deploy_architecture` we must ensure the
iptables change we do is done before reaching the deployment step.
  • Loading branch information
pablintino authored and openshift-merge-bot[bot] committed Aug 5, 2024
1 parent dee1f78 commit 949d903
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 9 deletions.
10 changes: 10 additions & 0 deletions reproducer.yml
Original file line number Diff line number Diff line change
Expand Up @@ -68,3 +68,13 @@
when: cifmw_allow_vms_to_reach_osp_api | default ('false') | bool
ansible.builtin.command: # noqa: command-instead-of-module
cmd: iptables -I LIBVIRT_FWI 1 -o ocpbm -j ACCEPT

- name: Run deployment if instructed to
when:
- cifmw_deploy_architecture | default(false) | bool
no_log: "{{ cifmw_nolog | default(true) | bool }}"
async: "{{ 7200 + cifmw_test_operator_timeout | default(3600) }}" # 2h should be enough to deploy EDPM and rest for tests.
poll: 20
delegate_to: controller-0
ansible.builtin.command:
cmd: "/home/zuul/deploy-architecture.sh"
9 changes: 0 additions & 9 deletions roles/reproducer/tasks/configure_architecture.yml
Original file line number Diff line number Diff line change
Expand Up @@ -25,12 +25,3 @@
ansible.builtin.include_tasks: rotate_log.yml
loop:
- ansible-deploy-architecture.log

- name: Run deployment if instructed to
when:
- cifmw_deploy_architecture | default(false) | bool
no_log: "{{ cifmw_nolog | default(true) | bool }}"
async: "{{ 7200 + cifmw_test_operator_timeout|default(3600) }}" # 2h should be enough to deploy EDPM and rest for tests.
poll: 20
ansible.builtin.command:
cmd: "/home/zuul/deploy-architecture.sh"

0 comments on commit 949d903

Please sign in to comment.