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

[WIP] Uber cluster controller #20733

Closed
wants to merge 4 commits into from
Closed

[WIP] Uber cluster controller #20733

wants to merge 4 commits into from

Conversation

jianhuiz
Copy link
Contributor

@jianhuiz jianhuiz commented Feb 5, 2016

This is initial working on API server and cluster controller of Ubernetes. It added the cluster objects to the existing API server and created a skeleton of cluster controller that watches the cluster objects.
Please review the code for fundamental flaws, we will keep working on the details and new functions in the mean while.

@quinton-hoole, @deepak-vij, @XiaoningDing, @alfred-huangjian
#19313

@googlebot
Copy link

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed, please reply here (e.g. I signed it!) and we'll verify. Thanks.


  • If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address. Check your existing CLA data and verify that your email is set on your git commits.
  • If you signed the CLA as a corporation, please let us know the company's name.

@k8s-bot
Copy link

k8s-bot commented Feb 5, 2016

Can one of the admins verify that this patch is reasonable to test? (reply "ok to test", or if you trust the user, reply "add to whitelist")

If this message is too spammy, please complain to ixdy.

@k8s-github-robot k8s-github-robot added kind/api-change Categorizes issue or PR as related to adding, removing, or otherwise changing an API kind/new-api labels Feb 5, 2016
@k8s-github-robot
Copy link

Labelling this PR as size/XL

@k8s-github-robot k8s-github-robot added the size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. label Feb 5, 2016
@k8s-bot
Copy link

k8s-bot commented Feb 5, 2016

Can one of the admins verify that this patch is reasonable to test? (reply "ok to test", or if you trust the user, reply "add to whitelist")

If this message is too spammy, please complain to ixdy.

@jianhuiz jianhuiz changed the title Uber cluster controller [WIP] Uber cluster controller Feb 6, 2016
@jianhuiz
Copy link
Contributor Author

jianhuiz commented Feb 6, 2016

i signed it!

@googlebot
Copy link

We found a Contributor License Agreement for you (the sender of this pull request), but were unable to find agreements for the commit author(s). If you authored these, maybe you used a different email address in the git commits than was used to sign the CLA (login here to double check)? If these were authored by someone else, then they will need to sign a CLA as well, and confirm that they're okay with these being contributed to Google.

@ghost ghost added the team/control-plane label Feb 8, 2016
@ghost ghost added this to the next-candidate milestone Feb 8, 2016
@ghost ghost assigned ghost and unassigned smarterclayton Feb 8, 2016
@ghost ghost added the priority/important-soon Must be staffed and worked on either currently, or very soon, ideally in time for the next release. label Feb 8, 2016
@lavalamp
Copy link
Member

lavalamp commented Feb 9, 2016

I'm recommending you copy /cmd/kube-apiserver and /pkg/master/

I or @nikhiljindal can answer questions.

@k8s-github-robot
Copy link

Labelling this PR as size/XXL

@k8s-github-robot k8s-github-robot added size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. and removed size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. labels Feb 9, 2016
@jianhuiz jianhuiz closed this Feb 12, 2016
@jianhuiz jianhuiz deleted the uber-cluster-controller branch February 12, 2016 23:20
@jianhuiz
Copy link
Contributor Author

uber api server in new PR due to "copy /cmd/kube-apiserver and /pkg/master/" and CLA issue
#21190

client for clusters api in PR
TBD

cluster controller in PR
TBD

@jianhuiz jianhuiz mentioned this pull request Feb 12, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/api-change Categorizes issue or PR as related to adding, removing, or otherwise changing an API priority/important-soon Must be staffed and worked on either currently, or very soon, ideally in time for the next release. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants