Skip to content

Conversation

@amartyasinha
Copy link
Contributor

The way ceph playbook from ci-framework is used in adoption jobs needs custom modules to be called without FQCN (i.e. through relative path).

Making custom module with FQCN work there would require to make symlink of the entire cifmw collection there, which does not make sense. Better to have an exception for such usage.

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jan 7, 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 fultonj 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

@amartyasinha amartyasinha requested a review from a team January 7, 2026 11:11
The way ceph playbook from ci-framework is used in adoption jobs needs custom modules to be called without FQCN (i.e. through relative path).

Making custom module with FQCN work there would require to make symlink of the entire cifmw collection there, which does not make sense. Better to have an exception for such usage.

Signed-off-by: Amartya Sinha <[email protected]>
@amartyasinha amartyasinha force-pushed the exception_for_cephx_key_fqcn branch from 2c9691a to 50086fe Compare January 7, 2026 11:13
@amartyasinha amartyasinha changed the title Do not use FQCN for cephx_module module (exception) Do not use FQCN for cephx_key module (exception) Jan 7, 2026
@amartyasinha amartyasinha merged commit 3af40cb into openstack-k8s-operators:main Jan 7, 2026
7 of 9 checks passed
@amartyasinha amartyasinha deleted the exception_for_cephx_key_fqcn branch January 9, 2026 06:59
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.

3 participants