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

Enhance Kuttl tests coverage #279

Conversation

omersch381
Copy link
Contributor

This patch enhances Kuttl coverage by adding the following tests:
* Scale up all Designate services
* Scale down all Designate services
* Scale up again all Designate services
* Scale down to 0 all Designate services

The tests will assert the Bind9 & Mdns predictable IPs and generated pools.yaml file were changed accordingly (except for the scale down to 0)

@omersch381 omersch381 requested a review from beagles January 31, 2025 12:27
Copy link
Contributor

openshift-ci bot commented Jan 31, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: omersch381

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

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

@omersch381 omersch381 force-pushed the enhance_kuttl_tests branch 12 times, most recently from 03f978a to ba94ad4 Compare February 4, 2025 19:30
This patch enhances Kuttl coverage by adding the following tests:
	* Scale up all Designate services
	* Scale down all Designate services
	* Scale up again all Designate services
	* Scale down to 0 all Designate services

The tests will assert the Bind9 & Mdns predictable IPs and generated
pools.yaml file were changed accordingly (except for the scale down to
0)
@beagles
Copy link
Collaborator

beagles commented Feb 5, 2025

/lgtm

@openshift-ci openshift-ci bot added the lgtm label Feb 5, 2025
@openshift-merge-bot openshift-merge-bot bot merged commit ae40ae1 into openstack-k8s-operators:main Feb 5, 2025
5 checks passed
@omersch381 omersch381 deleted the enhance_kuttl_tests branch February 6, 2025 12:47
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