Skip to content

[reproducer] Replace all reproducer vars with zuul vars#3794

Draft
danpawlik wants to merge 1 commit intoopenstack-k8s-operators:mainfrom
danpawlik:add-missing-vars
Draft

[reproducer] Replace all reproducer vars with zuul vars#3794
danpawlik wants to merge 1 commit intoopenstack-k8s-operators:mainfrom
danpawlik:add-missing-vars

Conversation

@danpawlik
Copy link
Contributor

Some variables are missing, especially related to the registry credentials. That complicate the situation, where some keys might not exists in reproducer file.

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Mar 24, 2026

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

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Mar 24, 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 dasm 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

@danpawlik danpawlik force-pushed the add-missing-vars branch 2 times, most recently from cb132dc to 6a1b430 Compare March 24, 2026 17:40
Some variables are missing, especially related to the registry
credentials. That complicate the situation, where some keys
might not exists in reproducer file. Some of the variables
are jinja2 template, which are "translated" by Ansible.
Alternative way was to use yq command like:

    yq eval-all -P 'select(fileIndex == 0) * select(fileIndex == 1)' \
        /tmp/reproducer-variables.yml \
        /home/zuul/configs/zuul_vars.yaml > merged.yaml

The yq is available in latest version available for current python3
version installed on hypervisor. In that case, we can not use
the yq tool, but use a python script.

Co-authored-by: Claude (AI Assistant) claude@anthropic.com

Signed-off-by: Daniel Pawlik <dpawlik@redhat.com>
@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/8560d523cce242b28ec457c07f18e35a

✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 50m 01s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 16m 59s
cifmw-crc-podified-edpm-baremetal RETRY_LIMIT in 35m 02s
✔️ cifmw-pod-zuul-files SUCCESS in 4m 37s
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-ansible-test SUCCESS in 8m 53s
✔️ cifmw-pod-pre-commit SUCCESS in 8m 02s
✔️ cifmw-molecule-reproducer SUCCESS in 15m 24s

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.

1 participant