Skip to content

Conversation

@belolipa
Copy link
Contributor

@belolipa belolipa commented Jan 23, 2026

[cifmw_cephadm] fix: drop FQCN for ci_script

The Zuul executor runs a restricted Ansible environment where the 'cifmw' collection is not formally installed. It relies on finding plugins via relative paths rather than collection namespaces. Consequently, the FQCN 'cifmw.general.ci_script' cannot be resolved.

This patch reverts the module call to the short name 'ci_script' to ensure the plugin is correctly located.

JIRA: OSPCIX-1169
Link: https://issues.redhat.com/browse/OSPCIX-1169

Signed-off-by: Oleg Belolipetskii [email protected]

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jan 23, 2026

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign gouthampacha for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found 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

Copy link
Contributor

@michburk michburk left a comment

Choose a reason for hiding this comment

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

Add [cifmw_cephadm] to the beginning of your commit title and add description and a signed-off-by line to the commit body to pass the two failing checks

@belolipa belolipa changed the title fix: drop FQCN for ci_script [cifmw_cephadm] fix: drop FQCN for ci_script Jan 23, 2026
@michburk
Copy link
Contributor

michburk commented Jan 23, 2026

The commit message body check is still failing, as the body requires a minimum amount of characters (not including the signed-off-by line). I think the description of this pr does a good job describing what this patch is doing, so feel free to include the lines about 'The zuul executor...' and 'This patch reverts...' in the commit body.
Sorry for being so picky about little hygiene things!

The Zuul executor runs a restricted Ansible environment where the 'cifmw' collection is not formally installed. It relies on finding plugins via relative paths rather than collection namespaces. Consequently, the FQCN 'cifmw.general.ci_script' cannot be resolved.

This patch reverts the module call to the short name 'ci_script' to ensure the plugin is correctly located.

JIRA: OSPCIX-1169
Link: https://issues.redhat.com/browse/OSPCIX-1169

Signed-off-by: Oleg Belolipetskii <[email protected]>
@softwarefactory-project-zuul
Copy link

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://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/1dec2f811c5949f4a44bffdbef0b8236

✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 16m 56s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 23m 54s
✔️ cifmw-crc-podified-edpm-baremetal SUCCESS in 1h 34m 10s
✔️ cifmw-crc-podified-edpm-baremetal-minor-update SUCCESS in 2h 02m 27s
✔️ podified-multinode-hci-deployment-crc SUCCESS in 1h 44m 20s
✔️ cifmw-pod-zuul-files SUCCESS in 4m 26s
✔️ noop SUCCESS in 0s
cifmw-pod-ansible-test TIMED_OUT in 30m 57s
✔️ cifmw-pod-pre-commit SUCCESS in 7m 57s
✔️ cifmw-molecule-cifmw_cephadm SUCCESS in 4m 19s

@michburk
Copy link
Contributor

recheck

@softwarefactory-project-zuul
Copy link

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://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/fb765232fafa4c9e917cc4efe3a717be

✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 03m 55s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 27m 23s
✔️ cifmw-crc-podified-edpm-baremetal SUCCESS in 1h 37m 38s
cifmw-crc-podified-edpm-baremetal-minor-update FAILURE in 24m 14s
✔️ podified-multinode-hci-deployment-crc SUCCESS in 1h 47m 01s
✔️ cifmw-pod-zuul-files SUCCESS in 4m 51s
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-ansible-test SUCCESS in 8m 14s
✔️ cifmw-pod-pre-commit SUCCESS in 8m 11s
✔️ cifmw-molecule-cifmw_cephadm SUCCESS in 4m 23s

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants