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

Update KF pipeline deployment document #329

Merged
merged 11 commits into from
Dec 8, 2018
Merged

Conversation

IronPan
Copy link
Member

@IronPan IronPan commented Dec 5, 2018

Note - The v0.1.3 is not yet created. The link will be valid once the release is created.


This change is Reviewable

@IronPan
Copy link
Member Author

IronPan commented Dec 5, 2018

/assign @sarahmaddox @abhi-g

@IronPan
Copy link
Member Author

IronPan commented Dec 5, 2018

/assign @paveldournov @yebrahim @qimingj @vicaire

Copy link
Contributor

@yebrahim yebrahim left a comment

Choose a reason for hiding this comment

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

/lgtm

```bash
kubectl create -f bootstrapper.yaml
```
Starting from Kubeflow Pipeline [V0.1.3](https://github.com/kubeflow/pipelines/releases/tag/0.1.3), pipeline will be one of the Kubeflow core components. It will be deployed automatically when deploying Kubeflow. Please refer to [GKE setup](/docs/started/getting-started-gke/)" for setting up Kubeflow.
Copy link
Contributor

Choose a reason for hiding this comment

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

I'd use present tense here, people will likely be reading this after 0.1.3 is out.

Copy link
Member Author

Choose a reason for hiding this comment

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

good point done

Copy link
Member

Choose a reason for hiding this comment

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

For some reason, I don't see your update in the latest diff...

Copy link
Member Author

Choose a reason for hiding this comment

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

oh i failed to push the change. updated.

@k8s-ci-robot k8s-ci-robot removed the lgtm label Dec 5, 2018
```bash
kubectl create -f bootstrapper.yaml
```
Starting from [V0.1.3](https://github.com/kubeflow/pipelines/releases/tag/0.1.3), Kubeflow Pipeline becomes one of the Kubeflow core components. It's automatically deployed during Kubeflow deployment. Please refer to [GKE setup](/docs/started/getting-started-gke/)" for setting up Kubeflow.
Copy link
Contributor

Choose a reason for hiding this comment

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

s/Kubeflow Pipeline/Kubeflow Pipelines

Copy link
Contributor

Choose a reason for hiding this comment

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

Also remove the extra quotation mark " after the bracket - it's rendering on the page.

Copy link
Member Author

Choose a reason for hiding this comment

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

done. thanks!

```bash
kubectl create -f bootstrapper.yaml
```
Starting from [V0.1.3](https://github.com/kubeflow/pipelines/releases/tag/0.1.3), Kubeflow Pipeline becomes one of the Kubeflow core components. It's automatically deployed during Kubeflow deployment. Please refer to [GKE setup](/docs/started/getting-started-gke/)" for setting up Kubeflow.
Copy link
Contributor

Choose a reason for hiding this comment

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

We shouldn't refer only to the GKE setup, as Pipelines is available for other flavours of Kubeflow too. Instead, how about linking to the getting-started guide:

See the [getting-started guide](/docs/started/getting-started/) for help with setting up Kubeflow, or the [GKE setup](/docs/started/getting-started-gke/) for a specific guide to Kubeflow on GKE.

Copy link
Member Author

Choose a reason for hiding this comment

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

Thanks Sarah. Great you pointed this out. I link to the GKE deployment on propose.

Unfortunately some of the Pipeline features are relying on GCP(specifically GCS) and it's not available for non-GCP environment. See
kubeflow/pipelines#345
kubeflow/pipelines#337

Before they are resolved, I am not sure if we should link to the guide you mentioned. The rest of the features should work fine though. What do you think? Which link should we point user to?

Copy link
Member

Choose a reason for hiding this comment

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

Do you mind adding a line mentioning this? There are users interested in trying this on non-GCP platforms and it would help them to know before spending time and figuring this out after some pain.

Copy link
Member Author

Choose a reason for hiding this comment

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

done. thanks


Note: Due to
[kubeflow/pipelines#345](https://github.com/kubeflow/pipelines/issues/345) and
[kubeflow/pipelines#337](https://github.com/kubeflow/pipelines/issues/337), Kubeflow Pipelines depends on GCP services and some of the funcitonalities are not supported by non-GKE cluster.
Copy link
Member

Choose a reason for hiding this comment

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

spelling "functionalities".

Copy link
Member Author

Choose a reason for hiding this comment

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

done

Note: Due to
[kubeflow/pipelines#345](https://github.com/kubeflow/pipelines/issues/345) and
[kubeflow/pipelines#337](https://github.com/kubeflow/pipelines/issues/337), Kubeflow Pipelines depends on GCP services and some of the funcitonalities are not supported by non-GKE cluster.
For best user experience, please deploy Kubeflow through GKE. Please refer to [GKE setup](/docs/started/getting-started-gke/) for instruction.
Copy link
Member

Choose a reason for hiding this comment

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

Can we please reword this last sentence to:
You can try it currently with a Kubeflow deployment on GKE. Please refer to GKE setup for instructions.

Copy link
Contributor

Choose a reason for hiding this comment

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

Also as an FYI, tech writers avoid the use of the word "please" as it can have an ambiguous tone, especially for readers whose first language is not English. Instead, it's clearer just to tell people what to do:

You can try it currently with a Kubeflow deployment on GKE. See the [GKE setup guide](/docs/started/getting-started-gke/).

Copy link
Member Author

Choose a reason for hiding this comment

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

done

@abhi-g
Copy link
Member

abhi-g commented Dec 8, 2018

/lgtm
/approve

@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: abhi-g

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot merged commit f6412c4 into kubeflow:master Dec 8, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants