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

k8s test matrix via kuttl + upgrade cass-operator to 1.8.0 in tests [K8SSAND-1008] [K8SSAND-1013] #80

Merged
merged 4 commits into from
Nov 11, 2021

Conversation

Miles-Garnsey
Copy link
Member

@Miles-Garnsey Miles-Garnsey commented Oct 29, 2021

What does this fix?

This PR fixes #79, which moves the cass-operator versions used in testing to 1.8.0.

#79 is required to address #76 because k8s 1.22 removes support for CRD v1beta1 APIs in favour of v1 only. Cass-operator 1.8.0 upgrades the CRD versions to v1.

This PR also addresses #82, using kuttl to implement a matrix of k8s versions so that we can test versions 1.20-1.22 as a preliminary step.

@Miles-Garnsey Miles-Garnsey marked this pull request as draft October 29, 2021 07:07
@Miles-Garnsey Miles-Garnsey marked this pull request as ready for review November 1, 2021 02:49
@Miles-Garnsey Miles-Garnsey force-pushed the miles/kuttl-test-2 branch 27 times, most recently from f792624 to 0910be6 Compare November 1, 2021 09:09
@Miles-Garnsey Miles-Garnsey force-pushed the miles/kuttl-test-2 branch 4 times, most recently from 3b328a2 to 535a7e5 Compare November 1, 2021 10:14
@Miles-Garnsey Miles-Garnsey changed the title Miles/kuttl test k8s test matrix via kuttl + upgrade cass-operator to 1.8.0 in tests Nov 1, 2021
@Miles-Garnsey Miles-Garnsey self-assigned this Nov 1, 2021
@Miles-Garnsey
Copy link
Member Author

Miles-Garnsey commented Nov 1, 2021

This PR is now ready for review. Note that attempting to run make e2e-test on k8s 1.22 will still fail at this stage. I will discuss with the team whether we want to update or remove those tests since they duplicate what we are now doing with kuttl.

I also rebased onto current master which makes this ready for merge whenever approved.

* Use kustomize for all kuttl tests.
* Migrate cass-operator in kuttl tests to v 1.8.0.
* Makefile steps for kuttl tests.
* Github actions workflows for kuttl tests.
* Reduce kind worker count down to 1 for all tests.
* Add additional kind configs to allow testing on more k8s versions.
* No multiple nodes per worker. Test only requires one node.
* Kind logs in .gitignore.
@jdonenine jdonenine changed the title k8s test matrix via kuttl + upgrade cass-operator to 1.8.0 in tests k8s test matrix via kuttl + upgrade cass-operator to 1.8.0 in tests [K8SSAND-1008] [K8SSAND-1013] Nov 4, 2021
Copy link
Contributor

@jsanda jsanda left a comment

Choose a reason for hiding this comment

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

@Miles-Garnsey can you please rebase?

@Miles-Garnsey
Copy link
Member Author

Conflict resolved. We can remove the additional merge commit when we squash (we need to do that anyway).

@sonarcloud
Copy link

sonarcloud bot commented Nov 11, 2021

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@Miles-Garnsey Miles-Garnsey merged commit d01d2b7 into k8ssandra:master Nov 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Rework test kustomizations so that they point at cass-operator 1.8.0 resources.
2 participants