Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Change how tempest is configured in test_operator role #2338

Closed
wants to merge 1 commit into from

Conversation

cjeanner
Copy link
Collaborator

Until now, whenever someone wanted to add a new option to their tempest
configuration, they had to add it directly in the defaults.yml.

This change proposes another approach, like we have for tobiko
configuration: a default set of configuration entries, and a parameter
allowing to override it.

In order to avoid breaking existing configurations, the default value is
still consuming the various parameters introduced over time - meaning
most of the existing jobs shouldn't be affected.

Copy link
Contributor

openshift-ci bot commented Sep 11, 2024

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

Copy link
Contributor

openshift-ci bot commented Sep 11, 2024

[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 ask for approval from cjeanner. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found here.

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

Until now, whenever someone wanted to add a new option to their tempest
configuration, they had to add it directly in the `defaults.yml`.

This change proposes another approach, like we have for tobiko
configuration: a default set of configuration entries, and a parameter
allowing to override it.

In order to avoid breaking existing configurations, the default value is
still consuming the various parameters introduced over time - meaning
most of the existing jobs shouldn't be affected.
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/680a59fdab5242d596ac35c5a6df3f06

✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 39m 46s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 16m 48s
✔️ cifmw-crc-podified-edpm-baremetal SUCCESS in 1h 28m 35s
cifmw-multinode-tempest FAILURE in 1h 22m 22s
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-ansible-test SUCCESS in 7m 02s
✔️ cifmw-pod-pre-commit SUCCESS in 7m 08s
✔️ build-push-container-cifmw-client SUCCESS in 21m 05s
✔️ cifmw-molecule-test_operator SUCCESS in 4m 09s

@openshift-merge-robot
Copy link
Contributor

PR needs rebase.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

Copy link

This PR is stale because it has been for over 15 days with no activity.
Remove stale label or comment or this will be closed in 7 days.

@github-actions github-actions bot added the Stale label Oct 24, 2024
@github-actions github-actions bot closed this Nov 1, 2024
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.

2 participants