-
Notifications
You must be signed in to change notification settings - Fork 38.9k
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
[Federation][init-10c] Fix hyperkube binary path in the image and disable auth mechanisms we don't support for now. #36048
[Federation][init-10c] Fix hyperkube binary path in the image and disable auth mechanisms we don't support for now. #36048
Conversation
Jenkins GCI GKE smoke e2e failed for commit 7e5027aa5d9ffdfe65f1e74fe4ce788c454e52cc. Full PR test history. The magic incantation to run this job again is |
Jenkins Kubemark GCE e2e failed for commit 7e5027aa5d9ffdfe65f1e74fe4ce788c454e52cc. Full PR test history. The magic incantation to run this job again is |
Jenkins GCE Node e2e failed for commit 7e5027aa5d9ffdfe65f1e74fe4ce788c454e52cc. Full PR test history. The magic incantation to run this job again is |
Jenkins GCI GCE e2e failed for commit 7e5027aa5d9ffdfe65f1e74fe4ce788c454e52cc. Full PR test history. The magic incantation to run this job again is |
Jenkins GKE smoke e2e failed for commit 7e5027aa5d9ffdfe65f1e74fe4ce788c454e52cc. Full PR test history. The magic incantation to run this job again is |
Jenkins GCE e2e failed for commit 7e5027aa5d9ffdfe65f1e74fe4ce788c454e52cc. Full PR test history. The magic incantation to run this job again is |
Jenkins GCE etcd3 e2e failed for commit 7e5027aa5d9ffdfe65f1e74fe4ce788c454e52cc. Full PR test history. The magic incantation to run this job again is |
7e5027a
to
2ea2118
Compare
LGTM |
2ea2118
to
48853dd
Compare
48853dd
to
519af1f
Compare
Jenkins unit/integration failed for commit 48853dd4660af52bb31b45a1a8238baa5688ad4f. Full PR test history. The magic incantation to run this job again is |
…manager image names configurable. This enables testing non-release images.
519af1f
to
e2890db
Compare
Jenkins verification failed for commit e2890dba8035f9695de87fe71987ddae95a21b07. Full PR test history. The magic incantation to run this job again is |
e2890db
to
4618eed
Compare
@@ -389,8 +389,6 @@ func createAPIServer(clientset *client.Clientset, namespace, name, image, creden | |||
"--etcd-servers=http://localhost:2379", | |||
"--service-cluster-ip-range=10.0.0.0/16", | |||
"--secure-port=443", | |||
"--token-auth-file=/etc/federation/apiserver/known_tokens.csv", | |||
"--basic-auth-file=/etc/federation/apiserver/basic_auth.csv", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We do support basic auth. That is how I connect to federation apiserver on my browser. Did we break it?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oh sorry. We do not support these auth mechanisms in the new setup? That is fine for alpha. I assume you have plans to support it later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@k8s-bot test this [submit-queue is verifying that this PR is safe to merge] |
Automatic merge from submit-queue |
Automatic merge from submit-queue [Federation][init-10d] Use the right service names in controller manager. Please review only the last commit here. This is based on PR #36048 which will be reviewed independently. Design Doc: PR #34484 cc @kubernetes/sig-cluster-federation @nikhiljindal
Please review only the last commit here. This is based on PR #36047 which will be reviewed independently.
Design Doc: PR #34484
cc @kubernetes/sig-cluster-federation @nikhiljindal
This change is![Reviewable](https://camo.githubusercontent.com/2d899f4291d07d3cd2fa4aaae1e3b243f164c23fce87d30a589ace0d496a444c/68747470733a2f2f72657669657761626c652e6b756265726e657465732e696f2f7265766965775f627574746f6e2e737667)