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

Helm instructions #111

Merged
merged 7 commits into from
Nov 2, 2017
Merged

Helm instructions #111

merged 7 commits into from
Nov 2, 2017

Conversation

jlewi
Copy link
Contributor

@jlewi jlewi commented Nov 1, 2017

Add links describing how to configure service accounts for tiller if needed.

Fixes #106

jlewi added 6 commits October 30, 2017 13:52
* We have abandoned the name mlkube.io and moved the code into tensorflow
  so it makes sense to use tensorflow.org as the group.

* We downgrade the API version from v1beta1 to v1alpha1.
  We want to follow the guidelines used by K8s for alpha, beta, stable:
  https://github.com/kubernetes/community/blob/master/contributors/devel/api_changes.md#alpha-beta-and-stable-versions

  according to this definition we our closer to alpha since the API hasn't
  achieved stability.
* This will make it easier to fetch resources belonging to a specific job.
* Fixes kubeflow#72.
@jlewi
Copy link
Contributor Author

jlewi commented Nov 1, 2017

@wbuchwalter @sozercan can one of you please take a look at the bits about Azure.

README.md Outdated
1. Ensure helm is running on your cluster

* On GKE with K8s 1.8, follow these
[instructions](https://github.com/kubernetes/helm/blob/master/docs/service_accounts.md)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a better link instead which will be updated with the proper setup instructions: https://docs.helm.sh/using_helm/#tiller-namespaces-and-rbac

@jlewi
Copy link
Contributor Author

jlewi commented Nov 2, 2017

Thanks @bacongobbler

@jlewi jlewi merged commit 17f204a into kubeflow:master Nov 2, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants