-
Notifications
You must be signed in to change notification settings - Fork 40.1k
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
master.go: converting API_v1beta* functions to private since they are not used outside #3494
master.go: converting API_v1beta* functions to private since they are not used outside #3494
Conversation
@erictune Is there a reason why the auth stuff should stay public? @smarterclayton Is there a reason why the API_x stuff should stay public? |
no objection to the change. wait for clayton though. |
We no longer need API_ public (our equiv of standalone.go is https://github.com/openshift/origin/blob/master/pkg/cmd/server/kubernetes/master.go). We may need the auth stuff to remain public - David is starting on the policy stuff and we may want to prototype it first.
|
We need handler container public and installswagger public. https://github.com/openshift/origin/blob/master/pkg/cmd/server/origin/master.go master really needs to be a factory with a default path, and the ability to reuse more code. It's kind of a black hole in terms of its influence. If I had to state a concrete use case, I would phrase it as "let people building on top of Kubernetes be able to reproduce the bulk of master functionality". Ie, someone wants to provide their own plugins, or wrap the Kube api with a simple golang filter should be able to. What's the best way to encode these things? Tests on master? The Kube service publish is a great example of reuse - I want to sync a running component into Kube, even though I'm not on Kube.
|
Thanks for the pointers clayton. I have updated the PR as per your comments. |
LGTM |
@nikhiljindal Please rebase. |
Rebased. Its now ready to merge! |
master.go: converting API_v1beta* functions to private since they are not used outside
No description provided.