Skip to content

[reproducer] Remove add_host that breaks reuse_ocp SSH connectivity#3969

Merged
openshift-merge-bot[bot] merged 1 commit into
openstack-k8s-operators:mainfrom
mnietoji:fix/reuse-ocp-bootstrap-include-role
Jun 2, 2026
Merged

[reproducer] Remove add_host that breaks reuse_ocp SSH connectivity#3969
openshift-merge-bot[bot] merged 1 commit into
openstack-k8s-operators:mainfrom
mnietoji:fix/reuse-ocp-bootstrap-include-role

Conversation

@mnietoji
Copy link
Copy Markdown
Contributor

@mnietoji mnietoji commented May 29, 2026

[reproducer] Remove add_host that breaks reuse_ocp SSH connectivity
The add_host task in reuse_main.yaml reads the inventory from
controller-0 and re-registers hosts with ansible_host set to
controller-0.utility. This hostname only resolves from the
hypervisor, not from the Zuul executor. The add_host also
overwrites the existing SSH ProxyJump configuration, causing
all subsequent delegated tasks to fail with UNREACHABLE because
the executor tries to connect directly to controller-0.utility.

This task was introduced in 10121fb as part of the refactor
from command-based to role-based bootstrap execution.

Co-Authored-By: Claude Opus 4.6 (1M context) noreply@anthropic.com
Signed-off-by: Miguel Angel Nieto Jimenez mnietoji@redhat.com

@mnietoji mnietoji force-pushed the fix/reuse-ocp-bootstrap-include-role branch 3 times, most recently from 80b7f22 to 823bdec Compare May 29, 2026 12:55
@mnietoji
Copy link
Copy Markdown
Contributor Author

recheck

@mnietoji mnietoji changed the title [reproducer] Fix reuse_ocp bootstrap by using include_role [reproducer] Remove add_host that breaks reuse_ocp SSH connectivity May 29, 2026
@centosinfra-prod-github-app
Copy link
Copy Markdown

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/buildset/d75a7b000abf49a9a2b93b101f73a25c

openstack-k8s-operators-content-provider FAILURE in 7m 58s
⚠️ podified-multinode-edpm-deployment-crc SKIPPED Skipped due to failed job openstack-k8s-operators-content-provider
⚠️ cifmw-crc-podified-edpm-baremetal SKIPPED Skipped due to failed job openstack-k8s-operators-content-provider
✔️ cifmw-pod-zuul-files SUCCESS in 5m 00s
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-ansible-test SUCCESS in 8m 18s
✔️ cifmw-pod-pre-commit SUCCESS in 10m 05s
cifmw-molecule-reproducer FAILURE in 10m 59s

The add_host task in reuse_main.yaml reads the inventory from
controller-0 and re-registers hosts with ansible_host set to
controller-0.utility. This hostname only resolves from the
hypervisor, not from the Zuul executor. The add_host also
overwrites the existing SSH ProxyJump configuration, causing
all subsequent delegated tasks to fail with UNREACHABLE because
the executor tries to connect directly to controller-0.utility.

This task was introduced in 10121fb as part of the refactor
from command-based to role-based bootstrap execution.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Signed-off-by: Miguel Angel Nieto Jimenez <mnietoji@redhat.com>
@mnietoji mnietoji force-pushed the fix/reuse-ocp-bootstrap-include-role branch from 823bdec to 2ebd3e7 Compare June 1, 2026 09:18
@centosinfra-prod-github-app
Copy link
Copy Markdown

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/buildset/90aac122b1c34887a24f34121fe4f43f

✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 46m 43s
podified-multinode-edpm-deployment-crc FAILURE in 1h 15m 11s
✔️ cifmw-crc-podified-edpm-baremetal SUCCESS in 1h 35m 09s
✔️ cifmw-pod-zuul-files SUCCESS in 4m 51s
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-ansible-test SUCCESS in 8m 32s
✔️ cifmw-pod-pre-commit SUCCESS in 8m 18s
✔️ cifmw-molecule-reproducer SUCCESS in 13m 59s

@mnietoji mnietoji requested a review from danpawlik June 1, 2026 14:07
@mnietoji mnietoji enabled auto-merge (rebase) June 2, 2026 07:28
@mnietoji
Copy link
Copy Markdown
Contributor Author

mnietoji commented Jun 2, 2026

recheck

Copy link
Copy Markdown
Contributor

@evallesp evallesp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@nemarjan
Copy link
Copy Markdown
Contributor

nemarjan commented Jun 2, 2026

/lgtm
/approve

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Jun 2, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: nemarjan

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci openshift-ci Bot added the approved label Jun 2, 2026
@openshift-merge-bot openshift-merge-bot Bot merged commit d9aafe5 into openstack-k8s-operators:main Jun 2, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants