Skip to content

Add all molecule files to the scope of check_zuul_jobs#3348

Merged
amartyasinha merged 1 commit intoopenstack-k8s-operators:mainfrom
amartyasinha:trigger-check_zuul_files
Sep 29, 2025
Merged

Add all molecule files to the scope of check_zuul_jobs#3348
amartyasinha merged 1 commit intoopenstack-k8s-operators:mainfrom
amartyasinha:trigger-check_zuul_files

Conversation

@amartyasinha
Copy link
Copy Markdown
Contributor

@amartyasinha amartyasinha commented Sep 29, 2025

check_zuul_jobs take care of multiple zuul related files. It also includes molecule job file. If any change is made to molecule within role definition, this job is not triggered, and it causes manual changes issue for molecule.yml zuul file. This commit ensure this job is run every time when any change is made to molecule file of any role. It will ensure jobs are not broken due to such changes.

Jira

Depends-On: #3346

@amartyasinha amartyasinha requested a review from a team as a code owner September 29, 2025 10:51
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Sep 29, 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

@softwarefactory-project-zuul
Copy link
Copy Markdown

Zuul encountered a syntax error while parsing its
configuration in the repo openstack-k8s-operators/ci-framework on branch main. The
problem was:

multiple repeat at position 8

The problem appears in the the "cifmw-pod-zuul-files" job stanza:

job:
name: cifmw-pod-zuul-files
parent: cifmw-pod-base
vars:
run_test: "check_zuul_files"
files:
- ^zuul.d/.*
- ^ci/templates/.*
- ^ci/config/.*
...

in "openstack-k8s-operators/ci-framework/zuul.d/pods.yaml@main", line 28

@amartyasinha amartyasinha force-pushed the trigger-check_zuul_files branch from 9dc5206 to d49290d Compare September 29, 2025 11:04
@softwarefactory-project-zuul
Copy link
Copy Markdown

Zuul encountered a syntax error while parsing its
configuration in the repo openstack-k8s-operators/ci-framework on branch main. The
problem was:

multiple repeat at position 8

The problem appears in the the "cifmw-pod-zuul-files" job stanza:

job:
name: cifmw-pod-zuul-files
parent: cifmw-pod-base
vars:
run_test: "check_zuul_files"
files:
- ^zuul.d/.*
- ^ci/templates/.*
- ^ci/config/.*
...

in "openstack-k8s-operators/ci-framework/zuul.d/pods.yaml@main", line 28

@amartyasinha amartyasinha force-pushed the trigger-check_zuul_files branch from d49290d to 128e472 Compare September 29, 2025 11:04
check_zuul_jobs take care of multiple zuul related files. It also includes molecule job file. If any change is made to molecule within role definition, this job is not triggered, and it causes manual changes issue for molecule.yml zuul file. This commit ensure this job is run every time when any change is made to molecule file of any role. It will ensure jobs are not broken due to such changes.
@amartyasinha amartyasinha force-pushed the trigger-check_zuul_files branch from 128e472 to 14dc59b Compare September 29, 2025 11:09
@softwarefactory-project-zuul
Copy link
Copy Markdown

This change depends on a change that failed to merge.

Change #3346 is needed.

@amartyasinha
Copy link
Copy Markdown
Contributor Author

recheck

@amartyasinha amartyasinha enabled auto-merge (rebase) September 29, 2025 14:02
@amartyasinha amartyasinha merged commit 8bb0a78 into openstack-k8s-operators:main Sep 29, 2025
4 checks passed
@amartyasinha amartyasinha deleted the trigger-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