-
Notifications
You must be signed in to change notification settings - Fork 4.6k
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
Clear append admission plugins before inserting flags to kube-apiserver #7182
Conversation
/test pull-kops-e2e-kubernetes-aws |
6009a06
to
4d5ce12
Compare
@robinpercy now the behaviour is what we need, and it works! It is not added to k8s apiserver manifest but the behaviour otherwise works as should. |
/test pull-kops-e2e-kubernetes-aws |
@zetaab looks great. Have you been able to test to see why the e2e is failing? |
@robinpercy it is timing out, like many other PRs currently @justinsb could we do something to this e2e aws test? It is really difficult to merge anything because its failing all the time |
/test pull-kops-e2e-kubernetes-aws |
2 similar comments
/test pull-kops-e2e-kubernetes-aws |
/test pull-kops-e2e-kubernetes-aws |
Thanks @zetaab /approve |
@justinsb this needs to be backported to release-1.13 and release-1.14. Is there some easy way to do that or do I have to do PRs? |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: justinsb, zetaab 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 |
Cherry pick of #7182 onto release-1.13
Cherry pick of #7182 onto release-1.14
do not append admissionplugins, cherrypick #7182 to 1.14
Currently when using appendAdmissionPlugins flag in kops configuration - it will also append that as flag to kube-apiserver. That will lead to result:
{"log":"error: unknown flag: --append-admission-plugins\n","stream":"stdout","time":"2019-06-24T10:21:40.176147035Z"}
(and kube-apiserver does not start)
This PR will remove --append-admission-plugins flag if defined. This PR needs to be cherry-picked to release-1.14 and release-1.13
/kind bug