Skip to content

Add more envTests to increase coverage#412

Merged
openshift-merge-bot[bot] merged 3 commits into
openstack-k8s-operators:mainfrom
kstrenkova:add-more-env-tests
Feb 18, 2026
Merged

Add more envTests to increase coverage#412
openshift-merge-bot[bot] merged 3 commits into
openstack-k8s-operators:mainfrom
kstrenkova:add-more-env-tests

Conversation

@kstrenkova
Copy link
Copy Markdown
Contributor

@kstrenkova kstrenkova commented Feb 5, 2026

The last PR regarding adding envTests added simple envTests for all controllers. This PR adds more specific tests and aims to increase test coverage and inlude more scenarios.

Assisted-By: Claude Sonet

@openshift-ci
Copy link
Copy Markdown

openshift-ci Bot commented Feb 5, 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

@softwarefactory-project-zuul
Copy link
Copy Markdown

Unable to freeze job graph: Job podified-multinode-edpm-deployment-crc-test-operator depends on openstack-k8s-operators-content-provider which was not run.

The order of InputReadyCondition checking was incorrect for Tobiko
and HorizonTest, as they also validate test-operator-clouds-config
config map. Now the order is correct, while I also moved the input
validation before PVC creation to fail faster in case of an issue
with missing inputs.
@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/d0f60288f03a45a0ac9e71cb884833e4

openstack-k8s-operators-content-provider NODE_FAILURE Node request 100-0008160444 failed in 0s
⚠️ podified-multinode-edpm-deployment-crc-test-operator SKIPPED Skipped due to failed job openstack-k8s-operators-content-provider

@kstrenkova kstrenkova force-pushed the add-more-env-tests branch 4 times, most recently from 1942adc to 7a9e97d Compare February 12, 2026 10:16
@kstrenkova kstrenkova marked this pull request as ready for review February 12, 2026 10:16
The last PR regarding adding envTests added simple envTests for
all controllers. This PR adds more specific tests and aims to
increase test coverage and inlude more scenarios.
This change is an addition to the main change that adds more
envTests to test-operator. It is specifically adding the testing
needed for Tempest and Tobiko when network attachments are
specified.
@openshift-ci
Copy link
Copy Markdown

openshift-ci Bot commented Feb 18, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: adrianfusco, kstrenkova

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

The pull request process is described 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

@openshift-ci openshift-ci Bot added the lgtm label Feb 18, 2026
@openshift-merge-bot openshift-merge-bot Bot merged commit 7ec84c0 into openstack-k8s-operators:main Feb 18, 2026
7 checks passed
@kstrenkova kstrenkova deleted the add-more-env-tests branch February 24, 2026 10:11
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