You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Per the discussion in PR 2561 it'd be good to run the acceptance tests against multiple versions of Kubernetes to ensure that changes work across the different versions.
A couple questions I have with regards to this:
What range of versions do we want to test against? My suggestion would be to a newer version of kind and use the latest minor versions supported by that version. As an example we could use kind v0.23.0 and Kubernetes versions 1.25-1.30.
Do we want to make this change to the acceptance tests run against cloud providers? Given those have a cost associated and the kind tests should be covering roughly the same thing I would expect not. If we do, what version range? If we don't, should we increase the version used in those?
What range of versions do we want to test against? My suggestion would be to a newer version of kind and use the latest minor versions supported by that version. As an example we could use kind v0.23.0 and Kubernetes versions 1.25-1.30.
we actually have a range already when testing the manifest resource so we can have it match what's already covered there:
Do we want to make this change to the acceptance tests run against cloud providers? Given those have a cost associated and the kind tests should be covering roughly the same thing I would expect not. If we do, what version range? If we don't, should we increase the version used in those?
Agreed, we wouldn't want to do the multiple k8s test on the cloud provider tests, kind tests are sufficient.
Description
Per the discussion in PR 2561 it'd be good to run the acceptance tests against multiple versions of Kubernetes to ensure that changes work across the different versions.
A couple questions I have with regards to this:
pinging @BBBmau from the PR discussion.
References
PR with initial discussion: PR 2561
Community Note
The text was updated successfully, but these errors were encountered: