Skip to content

Include read_global_vars.yml in pre-run: zuul.d/pods.yaml#3277

Merged
amartyasinha merged 1 commit intoopenstack-k8s-operators:mainfrom
amartyasinha:pods-zuul-read-group-vars
Sep 11, 2025
Merged

Include read_global_vars.yml in pre-run: zuul.d/pods.yaml#3277
amartyasinha merged 1 commit intoopenstack-k8s-operators:mainfrom
amartyasinha:pods-zuul-read-group-vars

Conversation

@amartyasinha
Copy link
Copy Markdown
Contributor

@amartyasinha amartyasinha commented Sep 10, 2025

The reason to make the change one file per commit is our flakey jobs. It is pain to get all jobs pass at once.

Depends-On: #3280

The reason to make the change one file per commit is our flakey jobs. It is pain to get all jobs pass at once
@amartyasinha amartyasinha requested a review from a team as a code owner September 10, 2025 07:08
@openshift-ci
Copy link
Copy Markdown
Contributor

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

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/f641504687eb4a219e237c449c745a44

✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 30m 34s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 17m 11s
cifmw-crc-podified-edpm-baremetal RETRY_LIMIT in 19m 46s
✔️ cifmw-pod-zuul-files SUCCESS in 4m 53s
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-ansible-test SUCCESS in 9m 03s
✔️ cifmw-pod-k8s-snippets-source SUCCESS in 4m 24s
✔️ cifmw-pod-pre-commit SUCCESS in 8m 33s
✔️ build-push-container-cifmw-client SUCCESS in 18m 57s

@amartyasinha
Copy link
Copy Markdown
Contributor Author

recheck

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

⚠️ openstack-k8s-operators-content-provider CANCELED
⚠️ podified-multinode-edpm-deployment-crc CANCELED
⚠️ cifmw-crc-podified-edpm-baremetal CANCELED
cifmw-pod-zuul-files FAILURE in 4m 17s
✔️ noop SUCCESS in 0s
⚠️ cifmw-pod-ansible-test CANCELED
⚠️ cifmw-pod-k8s-snippets-source CANCELED
⚠️ cifmw-pod-pre-commit CANCELED
⚠️ build-push-container-cifmw-client CANCELED

@amartyasinha
Copy link
Copy Markdown
Contributor Author

recheck

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

openstack-k8s-operators-content-provider TIMED_OUT in 31m 04s
⚠️ podified-multinode-edpm-deployment-crc SKIPPED Skipped due to failed job openstack-k8s-operators-content-provider
⚠️ cifmw-crc-podified-edpm-baremetal SKIPPED Skipped due to failed job openstack-k8s-operators-content-provider
✔️ cifmw-pod-zuul-files SUCCESS in 5m 06s
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-ansible-test SUCCESS in 8m 47s
✔️ cifmw-pod-k8s-snippets-source SUCCESS in 4m 56s
✔️ cifmw-pod-pre-commit SUCCESS in 8m 18s
✔️ build-push-container-cifmw-client SUCCESS in 17m 03s

@amartyasinha
Copy link
Copy Markdown
Contributor Author

recheck

1 similar comment
@amartyasinha
Copy link
Copy Markdown
Contributor Author

recheck

@bshewale
Copy link
Copy Markdown
Contributor

@amartyasinha So far we don't have any var in the https://github.com/openstack-k8s-operators/ci-framework/blob/main/group_vars/all.yml so how we are testing this change. I mean if we add any of the commonly used var in group_vars/all.yaml and remove that var from all other places then that change will give us more idea about this change wdyt?

@amartyasinha
Copy link
Copy Markdown
Contributor Author

@amartyasinha So far we don't have any var in the https://github.com/openstack-k8s-operators/ci-framework/blob/main/group_vars/all.yml so how we are testing this change. I mean if we add any of the commonly used var in group_vars/all.yaml and remove that var from all other places then that change will give us more idea about this change wdyt?

The reason I'm doing small PRs is flaky jobs. I've broken down the original PR #3237 into multiple small PRs to ensure gate jobs pass. Once all jobs are reading the group_vars/all.yaml file through the read_global_vars.yaml playbooks, I'll create a PR to add vars in group_vars/all.yaml and replace it everywhere in the framework. The test related to that will be tested there.

I also wanted to make changes in the single PR, but our flaky jobs are not allowing us.

@amartyasinha amartyasinha merged commit eddd0d5 into openstack-k8s-operators:main Sep 11, 2025
7 checks passed
@amartyasinha amartyasinha deleted the pods-zuul-read-group-vars 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