Skip to content

Use explicit Kubernetes versions for integration tests - #60

Closed
mweibel wants to merge 1 commit into
mainfrom
feat/explicit-k8s-versions
Closed

Use explicit Kubernetes versions for integration tests#60
mweibel wants to merge 1 commit into
mainfrom
feat/explicit-k8s-versions

Conversation

@mweibel

@mweibel mweibel commented Aug 28, 2026

Copy link
Copy Markdown
Collaborator
  • Add .github/kubernetes-versions.json config file
  • Update helpers/test-matrix to read from config instead of using dynamic release-set detection from k8test
  • Document version update process in README.md

This prevents CI breakage when new Kubernetes releases outpace cri-tools support. Versions are now updated manually per the 3-latest-minor-releases policy.

- Add .github/kubernetes-versions.json config file
- Update helpers/test-matrix to read from config instead of using
  dynamic release-set detection from k8test
- Document version update process in README.md

This prevents CI breakage when new Kubernetes releases outpace
cri-tools support. Versions are now updated manually per the
3-latest-minor-releases policy.
@mweibel
mweibel force-pushed the feat/explicit-k8s-versions branch from 5f32ead to 7be51a0 Compare August 31, 2026 06:20
@alakae

alakae commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

IMHO this is not the right fix, as far as I can tell from https://github.com/kubernetes-sigs/cri-tools#compatibility-matrix-cri-tools--kubernetes running K8S 1.37 with 1.36 crictl is supported. So I'd suggest to use the a more enhanced algorithm to detect the cri-tools version: pick the highest cri-tools tag whose major.minor is <= the target Kubernetes minor

@mweibel

mweibel commented Sep 1, 2026

Copy link
Copy Markdown
Collaborator Author

Fixed by cloudscale-ch/k8test#7 (and cri-tools released meanwhile). helpers/run-in-test-cluster does automatically pull k8test latest main, so no change necessary.

@mweibel mweibel closed this Sep 1, 2026
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.

2 participants