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
Drop hyperkube use from local-up-cluster.sh #81268
Drop hyperkube use from local-up-cluster.sh #81268
Conversation
We should just use the regular binaries used everywhere else and drop hyperkube. Only the local-up-cluster related CI jobs use hyperkube at the moment, so this will help deprecate and eliminate hyperkube at least from the main kubernetes repository. Change-Id: Ic53a4b56420565f546ac75be6cc23d214766c057
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: dims The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/priority important-soon |
/test pull-kubernetes-kubemark-e2e-gce-big |
/retest |
/test pull-kubernetes-integration |
1 similar comment
/test pull-kubernetes-integration |
/test pull-kubernetes-local-e2e |
/lgtm btw, should we add a release note for this type of change? |
found this in the logs:
|
/test pull-kubernetes-local-e2e |
@neolit123 looks like we have to fix that in test-infra repo: |
@neolit123 we don't ship this and we don't tell users to use this (devs only). so we should be good here. |
yes, agreed. |
@dims do you intend to cherry pick this to the three patch branches? Won't k/test-infra/kubetest run the older branches' local-cluster-up.sh and expect a hyperkube? |
AFAIK, in terms of our CI, not if the appropriate versioned kubekins image (that hosts kubetest) is used. but this change might break someone that tries to use a standalone recent (e.g. build from test-infra master) kubetest against an older branch of k/k. |
my suggestion on the mailing list was - announce hyperkube deprecation and give it at least 1 release for removal. but given hyperkube is used in production we should probably apply the GA deprecation rules on it. |
/retest |
/override pull-kubernetes-local-e2e |
@dims: dims unauthorized: /override is restricted to repo administrators In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/test pull-kubernetes-local-e2e |
@dims: The following test failed, say
Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
/hold cancel |
We should just use the regular binaries used everywhere else and drop
hyperkube. Only the local-up-cluster related CI jobs use hyperkube at
the moment, so this will help deprecate and eliminate hyperkube at least
from the main kubernetes repository.
Change-Id: Ic53a4b56420565f546ac75be6cc23d214766c057
What type of PR is this?
/kind cleanup
What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:
Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.: