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

master.go: converting API_v1beta* functions to private since they are not used outside #3494

Merged
merged 1 commit into from
Jan 16, 2015
Merged

master.go: converting API_v1beta* functions to private since they are not used outside #3494

merged 1 commit into from
Jan 16, 2015

Conversation

nikhiljindal
Copy link
Contributor

No description provided.

@lavalamp
Copy link
Member

@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?

@erictune
Copy link
Member

no objection to the change. wait for clayton though.

@smarterclayton
Copy link
Contributor

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.

On Jan 14, 2015, at 7:38 PM, Daniel Smith notifications@github.com wrote:

@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?


Reply to this email directly or view it on GitHub.

@smarterclayton
Copy link
Contributor

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.

On Jan 14, 2015, at 7:43 PM, Daniel Smith notifications@github.com wrote:

Assigned #3494 to @smarterclayton.


Reply to this email directly or view it on GitHub.

@nikhiljindal
Copy link
Contributor Author

Thanks for the pointers clayton.
I do not see InstallSwaggerAPI being used in openshift.

I have updated the PR as per your comments.
Please take another look.

@nikhiljindal nikhiljindal changed the title master.go cleanup: Deleting unused public methods and converting rest to private master.go: converting API_v1beta* functions to private since they are not used outside Jan 15, 2015
@smarterclayton
Copy link
Contributor

LGTM

@nikhiljindal nikhiljindal added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Jan 15, 2015
@bgrant0607
Copy link
Member

@nikhiljindal Please rebase.

@nikhiljindal
Copy link
Contributor Author

Rebased. Its now ready to merge!

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.

6 participants