-
Notifications
You must be signed in to change notification settings - Fork 116
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
Bleeding Edge Kubernetes CI #185
Comments
Mentioned during community meeting (#174) |
Hi @otaviof , I think we also need to think about the solution when using Travis: 1, How to test different latest and stable 2, Is there a better way to auto-detect the latest and stable |
👋
I think we can add more jobs to Travis-CI, and make sure KinD uses the latest available Kubernetes cluster for testing, and possibly in parallel, testing against supported versions of Kubernetes. So just, two different test jobs in Travis-CI would do, in my opinion.
We install KinD via However, for Tekton I think we should follow the regular |
👍 , it is cool if Travis can run two different jobs in parallel! :) |
Ok, I like this to be only a matter of the Kind node version. This is very simple, in the travis.yml you should use something like:
then consume KIND_VERSION in https://github.com/redhat-developer/build/blob/master/hack/install-kind.sh#L13 and that will trigger 2 runs of the tests in different Kinds. Playing the role of the bad guy here: I get the point on knowing in advance if a very new k8s version will break us,but I would like to know the step after that, do we want to fix this immediately? while if not, this will imply all PRs to be red for a while. Also, do we need to know this if neither IKS/openshift is using this versions(time consuming concern) |
We should prioritize fixing that I would say - unless we have a good reason not to :) |
Lets make sure the Travis-CI tests are duplicated to test against latest Kubernetes versions, and therefore in advance, we will be notified about compatibility problems that either the operator, or surrounding tooling (like
operator-sdk
) may have.The text was updated successfully, but these errors were encountered: