Skip to content

[reproducer] Skip compute configuration when preProvisioned is false#3974

Merged
openshift-merge-bot[bot] merged 1 commit into
openstack-k8s-operators:mainfrom
rebtoor:fix/reproducer-skip-bm-compute-config
Jul 2, 2026
Merged

[reproducer] Skip compute configuration when preProvisioned is false#3974
openshift-merge-bot[bot] merged 1 commit into
openstack-k8s-operators:mainfrom
rebtoor:fix/reproducer-skip-bm-compute-config

Conversation

@rebtoor

@rebtoor rebtoor commented Jun 3, 2026

Copy link
Copy Markdown
Contributor

The "Configure EDPM based nodes ctlplane network" task in libvirt_layout.yml delegates SSH to all compute hosts. In baremetal deployments with preProvisioned=false, Ironic-provisioned computes do not accept SSH from the hypervisor, causing the reproducer to fail with Permission denied.

The task's own comment states it configures pre-provisioned nodes. Add the matching condition to enforce that.

Related-Issue: ANVIL-109

@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/21a436d5fab0434aaa962fb69850a467

✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 46m 22s
podified-multinode-edpm-deployment-crc RETRY_LIMIT in 10m 00s
✔️ cifmw-crc-podified-edpm-baremetal SUCCESS in 1h 32m 38s
✔️ cifmw-pod-zuul-files SUCCESS in 4m 48s
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-ansible-test SUCCESS in 9m 12s
✔️ cifmw-pod-pre-commit SUCCESS in 8m 43s
✔️ cifmw-molecule-reproducer SUCCESS in 15m 43s

@github-actions

Copy link
Copy Markdown

This PR is stale because it has been for over 15 days with no activity.
Remove stale label or comment or this will be closed in 7 days.

@github-actions github-actions Bot added the Stale label Jun 19, 2026
@github-actions github-actions Bot closed this Jun 27, 2026
@rebtoor rebtoor reopened this Jul 1, 2026
The "Configure EDPM based nodes ctlplane network" task in
libvirt_layout.yml delegates SSH to all compute hosts. In baremetal
deployments with preProvisioned=false, Ironic-provisioned computes
do not accept SSH from the hypervisor, causing the reproducer to
fail with Permission denied.

The task's own comment states it configures pre-provisioned nodes.
Add the matching condition to enforce that.

Related-Issue: ANVIL-109
Co-authored-by: Cursor <cursoragent@cursor.com>
Signed-off-by: Roberto Alfieri <ralfieri@redhat.com>
@rebtoor rebtoor force-pushed the fix/reproducer-skip-bm-compute-config branch from 785451d to 871e2f4 Compare July 1, 2026 13:48
@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/0b7a1db518d945e7913d62efc0824022

✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 58m 52s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 41m 31s
cifmw-crc-podified-edpm-baremetal NODE_FAILURE Node(set) request 099-0000132644 failed in 0s
✔️ cifmw-pod-zuul-files SUCCESS in 6m 02s
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-ansible-test SUCCESS in 10m 38s
✔️ cifmw-pod-pre-commit SUCCESS in 9m 57s
✔️ cifmw-molecule-reproducer SUCCESS in 13m 58s

@rebtoor

rebtoor commented Jul 1, 2026

Copy link
Copy Markdown
Contributor Author

recheck

@evallesp evallesp left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

/lgtm

@michburk michburk left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

/approve

@openshift-ci

openshift-ci Bot commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: michburk

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 Jul 2, 2026
@openshift-merge-bot openshift-merge-bot Bot merged commit 81950e0 into openstack-k8s-operators:main Jul 2, 2026
9 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