Skip to content

Conversation

@amartyasinha
Copy link
Contributor

This commit is one of the steps of replacing common used vars with group_vars to improve overall maintenance of variables in ci-framework

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Oct 22, 2025

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@amartyasinha amartyasinha changed the title Move cifmw_installyamls_repo var to group_vars WIP: Move cifmw_installyamls_repo var to group_vars Oct 22, 2025
@amartyasinha amartyasinha force-pushed the install_yamls_group_var branch from 704d7e8 to 64648b0 Compare October 22, 2025 12:34
@amartyasinha amartyasinha changed the title WIP: Move cifmw_installyamls_repo var to group_vars Move cifmw_installyamls_repo var to group_vars Oct 22, 2025
@amartyasinha
Copy link
Contributor Author

recheck

@amartyasinha amartyasinha changed the title Move cifmw_installyamls_repo var to group_vars Move cifmw_installyamls_repos var to group_vars Oct 22, 2025
@amartyasinha amartyasinha force-pushed the install_yamls_group_var branch from 64648b0 to be3fd92 Compare October 22, 2025 13:02
@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/446a96d2eaf740fb97bf35fb07ebb40e

openstack-k8s-operators-content-provider FAILURE in 5m 17s
⚠️ 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-multinode-tempest SKIPPED Skipped due to failed job openstack-k8s-operators-content-provider
✔️ cifmw-pod-zuul-files SUCCESS in 4m 18s
⚠️ adoption-standalone-to-crc-ceph-provider SKIPPED Skipped due to failed job openstack-k8s-operators-content-provider
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-ansible-test SUCCESS in 8m 45s
✔️ cifmw-pod-pre-commit SUCCESS in 7m 19s
cifmw-tcib FAILURE in 4m 40s
✔️ build-push-container-cifmw-client SUCCESS in 17m 19s
✔️ cifmw-molecule-edpm_deploy SUCCESS in 4m 44s
✔️ cifmw-molecule-edpm_deploy_baremetal SUCCESS in 4m 42s
✔️ cifmw-molecule-edpm_prepare SUCCESS in 4m 36s
✔️ cifmw-molecule-install_yamls SUCCESS in 4m 44s
✔️ cifmw-molecule-reproducer SUCCESS in 16m 02s
✔️ cifmw-molecule-tempest SUCCESS in 5m 04s
✔️ cifmw-molecule-tofu SUCCESS in 10m 07s
✔️ cifmw-molecule-update SUCCESS in 4m 46s

@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/59f5ac5e698c402a8690016fa6a7d406

openstack-k8s-operators-content-provider FAILURE in 5m 15s
⚠️ 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-multinode-tempest SKIPPED Skipped due to failed job openstack-k8s-operators-content-provider
✔️ cifmw-pod-zuul-files SUCCESS in 4m 52s
⚠️ adoption-standalone-to-crc-ceph-provider SKIPPED Skipped due to failed job openstack-k8s-operators-content-provider
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-ansible-test SUCCESS in 7m 57s
✔️ cifmw-pod-pre-commit SUCCESS in 7m 24s
cifmw-tcib FAILURE in 5m 00s
✔️ build-push-container-cifmw-client SUCCESS in 19m 35s
✔️ cifmw-molecule-edpm_deploy SUCCESS in 4m 35s
✔️ cifmw-molecule-edpm_deploy_baremetal SUCCESS in 4m 43s
✔️ cifmw-molecule-edpm_prepare SUCCESS in 4m 31s
✔️ cifmw-molecule-install_yamls SUCCESS in 5m 00s
✔️ cifmw-molecule-reproducer SUCCESS in 15m 22s
✔️ cifmw-molecule-tempest SUCCESS in 5m 10s
✔️ cifmw-molecule-tofu SUCCESS in 8m 06s
✔️ cifmw-molecule-update SUCCESS in 5m 00s

This commit is one of the steps of replacing common used vars with group_vars to improve overall maintenance of variables in ci-framework

Signed-off-by: Amartya Sinha <amsinha@redhat.com>
@amartyasinha amartyasinha force-pushed the install_yamls_group_var branch from be3fd92 to b35c719 Compare October 23, 2025 08:37
@amartyasinha amartyasinha marked this pull request as ready for review October 24, 2025 05:24
@amartyasinha amartyasinha requested review from a team and eshulman2 as code owners October 24, 2025 05:24
Copy link
Contributor

@sdatko sdatko left a comment

Choose a reason for hiding this comment

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

/lgtm
/approve

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Oct 24, 2025

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: sdatko

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

@sdatko sdatko enabled auto-merge (rebase) October 24, 2025 07:20
@sdatko sdatko disabled auto-merge October 24, 2025 07:20
Copy link
Contributor

@holser holser left a comment

Choose a reason for hiding this comment

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

LGTM

@amartyasinha amartyasinha enabled auto-merge (rebase) October 24, 2025 09:48
@amartyasinha amartyasinha merged commit f66a47b into openstack-k8s-operators:main Oct 26, 2025
7 of 8 checks passed
amoralej added a commit to amoralej/watcher-operator that referenced this pull request Oct 27, 2025
After openstack-k8s-operators/ci-framework#3430
we need to explicitely set the variable. In the edpm jobs we are
inheriting it from base definitions, but for kuttl one we need to add
it.

Signed-off-by: Alfredo Moralejo <amoralej@redhat.com>
@abays
Copy link
Contributor

abays commented Oct 28, 2025

This seems to have broken some Zuul jobs in a few repos. So far Nova and Watcher have been affected, but there may be more.

stuggi added a commit to stuggi/nova-operator that referenced this pull request Oct 28, 2025
After openstack-k8s-operators/ci-framework#3430 we need to
explicitely set the variable.

Currently the job is failing with:
''cifmw_installyamls_repos'' is undefined'

Signed-off-by: Martin Schuppert <mschuppert@redhat.com>
stuggi added a commit to stuggi/nova-operator that referenced this pull request Oct 28, 2025
After openstack-k8s-operators/ci-framework#3430 we need to
explicitly set the variable.

Currently the job is failing with:
''cifmw_installyamls_repos'' is undefined'

Signed-off-by: Martin Schuppert <mschuppert@redhat.com>
openshift-merge-bot bot pushed a commit to openstack-k8s-operators/watcher-operator that referenced this pull request Oct 28, 2025
After openstack-k8s-operators/ci-framework#3430
we need to explicitely set the variable. In the edpm jobs we are
inheriting it from base definitions, but for kuttl one we need to add
it.

Signed-off-by: Alfredo Moralejo <amoralej@redhat.com>
openshift-cherrypick-robot pushed a commit to openshift-cherrypick-robot/watcher-operator that referenced this pull request Oct 28, 2025
After openstack-k8s-operators/ci-framework#3430
we need to explicitely set the variable. In the edpm jobs we are
inheriting it from base definitions, but for kuttl one we need to add
it.

Signed-off-by: Alfredo Moralejo <amoralej@redhat.com>
@danpawlik
Copy link
Contributor

Let's try to fix in #3438

openshift-merge-bot bot pushed a commit to openstack-k8s-operators/watcher-operator that referenced this pull request Oct 30, 2025
After openstack-k8s-operators/ci-framework#3430
we need to explicitely set the variable. In the edpm jobs we are
inheriting it from base definitions, but for kuttl one we need to add
it.

Signed-off-by: Alfredo Moralejo <amoralej@redhat.com>
@amartyasinha amartyasinha deleted the install_yamls_group_var branch January 9, 2026 06:59
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.

7 participants