Skip to content

ci: test across several k8s versions #394

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

Merged
merged 1 commit into from
Apr 15, 2025
Merged

ci: test across several k8s versions #394

merged 1 commit into from
Apr 15, 2025

Conversation

pmalek
Copy link
Member

@pmalek pmalek commented Apr 15, 2025

What this PR does / why we need it:

This PR:

Which issue this PR fixes

Fixes #393

Special notes for your reviewer:

A follow up PR will release 1.3.2

PR Readiness Checklist:

Complete these before marking the PR as ready to review:

  • the CHANGELOG.md release notes have been updated to reflect significant changes

@pmalek pmalek self-assigned this Apr 15, 2025
@pmalek pmalek force-pushed the ci-test-multiple-k8s branch 2 times, most recently from f15fae3 to ae11d64 Compare April 15, 2025 10:57
@pmalek
Copy link
Member Author

pmalek commented Apr 15, 2025

For reference: change in 1.32 made CEL reserved keywords (like namespace) to be used kubernetes/kubernetes#126977.

This is the reason installing CRDs in Kubernetes versions older than 1.32 currently fails.

@pmalek pmalek force-pushed the ci-test-multiple-k8s branch 11 times, most recently from b24a85a to 6a064f0 Compare April 15, 2025 12:02
@pmalek pmalek force-pushed the ci-test-multiple-k8s branch from 6a064f0 to 03b6308 Compare April 15, 2025 13:19
@pmalek pmalek added this to the KGO v1.6.x milestone Apr 15, 2025
@pmalek pmalek linked an issue Apr 15, 2025 that may be closed by this pull request
1 task
@pmalek pmalek marked this pull request as ready for review April 15, 2025 13:23
@pmalek pmalek requested a review from a team as a code owner April 15, 2025 13:23
Copy link
Member

@mlavacca mlavacca left a comment

Choose a reason for hiding this comment

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

Cool, 🚢

@pmalek pmalek merged commit ce54839 into main Apr 15, 2025
17 checks passed
@pmalek pmalek deleted the ci-test-multiple-k8s branch April 15, 2025 14:27
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.

Run CRD validation tests against a set of supported k8s versions
2 participants