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

Pull kubectl-skew tests out from other upgrade tests, since it's a different matrix #109

Merged
merged 1 commit into from
Jun 7, 2016

Conversation

ikehz
Copy link
Contributor

@ikehz ikehz commented Jun 3, 2016

This is a result of kubernetes/kubernetes#25087; the matrix that we care about is different for node-master and client-master skews.

@ikehz
Copy link
Contributor Author

ikehz commented Jun 3, 2016

This also removes the usage of JENKINS_USE_SKEW_KUBECTL, since kubernetes/kubernetes#26789 skews the client by default. This should not go in until kubernetes/kubernetes#26789 is in.

@ikehz
Copy link
Contributor Author

ikehz commented Jun 3, 2016

cc @maisem

@ikehz ikehz force-pushed the separate-client-matrix branch from 01551b1 to 05609aa Compare June 3, 2016 18:08
provider: 'gke'
provider-env: '{gke-provider-env}'
jobs:
- 'kubernetes-e2e-{provider}-{version-cluster}-{version-client}-kubctl-skew':
Copy link
Contributor

Choose a reason for hiding this comment

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

kubectl :)

@ikehz ikehz force-pushed the separate-client-matrix branch from 05609aa to b9d4902 Compare June 3, 2016 20:58
@ikehz
Copy link
Contributor Author

ikehz commented Jun 3, 2016

@spxtr PTAL.

@spxtr
Copy link
Contributor

spxtr commented Jun 3, 2016

Looks fine to me. cc @fejta who is looking into cleaning up this file. I think the plan is to move the upgrade jobs into their own file. We'll have to duplicate the job template stuff, but I think that's fine.

@fejta
Copy link
Contributor

fejta commented Jun 3, 2016

I'll happily make sure to merge your changes

@spxtr spxtr added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Jun 3, 2016
k8s-github-robot pushed a commit to kubernetes/kubernetes that referenced this pull request Jun 7, 2016
Automatic merge from submit-queue

Always skew the client when running version-skew tests

As mentioned in the comments:

- for upgrade jobs, we want kubectl to be at the same version as master.
- for client skew tests, we want to use the skewed kubectl (that's what we're testing).

So, remove gate on `JENKINS_USE_SKEW_KUBECTL`, and always use the skewed `kubectl`.

This should go in before kubernetes/test-infra#109.
@ikehz
Copy link
Contributor Author

ikehz commented Jun 7, 2016

kubernetes/kubernetes#26789 is in; merging this.

@ikehz ikehz merged commit fe6661d into kubernetes:master Jun 7, 2016
simonswine pushed a commit to simonswine/test-infra that referenced this pull request Jan 16, 2018
mborsz pushed a commit to mborsz/test-infra that referenced this pull request Dec 14, 2018
ostromart pushed a commit to ostromart/test-infra that referenced this pull request Jul 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm "Looks good to me", indicates that a PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants