Skip to content

Increase the scope of cifmw-pods-zuul-files job#3355

Merged
amartyasinha merged 1 commit intoopenstack-k8s-operators:mainfrom
amartyasinha:increase_scope_of_check_zuul_files
Sep 30, 2025
Merged

Increase the scope of cifmw-pods-zuul-files job#3355
amartyasinha merged 1 commit intoopenstack-k8s-operators:mainfrom
amartyasinha:increase_scope_of_check_zuul_files

Conversation

@amartyasinha
Copy link
Copy Markdown
Contributor

@amartyasinha amartyasinha commented Sep 30, 2025

Just checking the changes to molecule files solve only half of our problem. If a user add a new role, even then molecule job definition for that role is not added, and this job starts to fail in future. Though running this job on any change made in roles dir is quite wider scope. But seems like it is not possible to just run the job when new role is added.

Related to: #3348

Just checking the changes to molecule files solve only half of our problem. If a user add a new role, even then molecule job definition for that role is not added, and this job starts to fail in future. Though running this job on any change made in roles dir is quite wider scope. But seems like it is not possible to just run the job when new role is added.
@amartyasinha amartyasinha requested a review from a team as a code owner September 30, 2025 11:33
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Sep 30, 2025

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:

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

@danpawlik
Copy link
Copy Markdown
Contributor

why not. Better to be sure.

Copy link
Copy Markdown
Contributor

@Valkyrie00 Valkyrie00 left a comment

Choose a reason for hiding this comment

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

Ok LGTM

@amartyasinha
Copy link
Copy Markdown
Contributor Author

recheck

@amartyasinha amartyasinha merged commit bcdd70e into openstack-k8s-operators:main Sep 30, 2025
5 checks passed
@amartyasinha amartyasinha deleted the increase_scope_of_check_zuul_files branch January 9, 2026 06:58
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