Create "Distributions" with kfctl + Kubeflow Operator #2492

rui-vas · 2021-02-08T18:03:26Z

PR to tackle issue #2489, as part of the Kubeflow docs restructuring project.

rui-vas · 2021-02-16T10:40:55Z

Hey @Bobgy could you please take a look into this PR?

This creates the concept of "Methods & Distributions" as discussed in the last community meetings, to clarify what docs belong to what methods or distro.

/assign @Bobgy

rui-vas · 2021-02-16T11:11:13Z

Hey @8bitmp3! Could you please take a look into this PR?

This creates the concept of "Methods & Distributions" as discussed in the last community meetings, to clarify what docs belong to what methods or distro.

content/en/docs/methods/_index.md

content/en/docs/methods/kfctl/_index.md

8bitmp3 · 2021-02-18T02:29:30Z

@RFMVasconcelos Catching up with PRs 🙏 See my comments above, LMKWYT. If you're happy with the links, I'm good and will LGTM

rui-vas · 2021-02-18T12:16:20Z

Hey @8bitmp3 !

Thank you for the review! Let me take a look :)

rui-vas · 2021-02-18T12:17:53Z

Nice catches! Accepted both suggestions :)

google-cla · 2021-03-10T23:07:43Z

All (the pull request submitter and all commit authors) CLAs are signed, but one or more commits were authored or co-authored by someone other than the pull request submitter.

We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that by leaving a comment that contains only @googlebot I consent. in this pull request.

Note to project maintainer: There may be cases where the author cannot leave a comment, or the comment is not properly detected as consent. In those cases, you can manually confirm consent of the commit author(s), and set the cla label to yes (if enabled on your project).

ℹ️ Googlers: Go here for more info.

google-cla · 2021-03-11T11:14:22Z

All (the pull request submitter and all commit authors) CLAs are signed, but one or more commits were authored or co-authored by someone other than the pull request submitter.

We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that by leaving a comment that contains only @googlebot I consent. in this pull request.

Note to project maintainer: There may be cases where the author cannot leave a comment, or the comment is not properly detected as consent. In those cases, you can manually confirm consent of the commit author(s), and set the cla label to yes (if enabled on your project).

ℹ️ Googlers: Go here for more info.

Update kfctl description Update content/en/docs/methods/_index.md

rui-vas · 2021-03-11T11:22:58Z

@thesuperzapper @8bitmp3 I squashed commits so no action required.

I think we're good to review and merge these. @joeliedtke @Bobgy would you be keen to take a look? :)

Bobgy

/lgtm

Thank you @RFMVasconcelos !

/assign @joeliedtke
to make the final decision

joeliedtke · 2021-03-16T23:00:01Z

Let's update the Netlify configuration to use the 1.2 docs branch (we may need to create this branch) so these changes can be made without affecting the published docs until the 1.3 release. Once that change is in place, I will remove the hold and approve this PR.

Bobgy · 2021-03-17T06:52:18Z

I've set up v1.2-branch in #2546.
/unhold

@RFMVasconcelos can you resolve the conflicts in this PR?

Bobgy · 2021-03-17T06:52:42Z

/approve

google-oss-robot · 2021-03-17T06:52:59Z

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: Bobgy, RFMVasconcelos

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:

~~OWNERS~~ [Bobgy]

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

rui-vas · 2021-03-17T12:16:32Z

I've resolved the conflict @Bobgy, need a new LGTM now as I made changes.

Bobgy · 2021-03-17T13:37:45Z

/lgtm

* Create methods folder == section * Move /operator under /methods * Update links on Operator * Add 'kfctl' folder == section * mv kfctl specific minikube docs under /kfctl * Update links on minikube * mv kustomize from other-guides to /kfctl * fix links for kustomize change * delete outdated redirect * move istio-dex-auth to /kfctl + rename to multi-user * fix links after name change * move kfctl install under /kfctl + rename to deployment * fix links after move * Add OWNERS for accountability Update kfctl description Update content/en/docs/methods/_index.md * Add redirects for Operator * Add redirects for kfctl * Rename "methods" to "distributions" * update redirects to distributions as folder name (cherry picked from commit c792fe5)

…rent `master` v2 (#2612) * Create "Distributions" with kfctl + Kubeflow Operator (#2492) * Create methods folder == section * Move /operator under /methods * Update links on Operator * Add 'kfctl' folder == section * mv kfctl specific minikube docs under /kfctl * Update links on minikube * mv kustomize from other-guides to /kfctl * fix links for kustomize change * delete outdated redirect * move istio-dex-auth to /kfctl + rename to multi-user * fix links after name change * move kfctl install under /kfctl + rename to deployment * fix links after move * Add OWNERS for accountability Update kfctl description Update content/en/docs/methods/_index.md * Add redirects for Operator * Add redirects for kfctl * Rename "methods" to "distributions" * update redirects to distributions as folder name * doc: Add instructions to access cluster with IBM Cloud vpc-gen2. (#2530) * doc, Add instructions to access cluster with IBM Cloud vpc-gen2. * added extra steps. * Improved formatting * Added details for creating cluster against existing VPC * Apply suggestions from code review Co-authored-by: 8bitmp3 <19637339+8bitmp3@users.noreply.github.com> * Apply suggestions from code review Co-authored-by: 8bitmp3 <19637339+8bitmp3@users.noreply.github.com> * Formatting fixes, as per the review. * Added a note about security. * Choose between a classic or vpc-gen2 provider. * added a note * formatting fixes * Apply suggestions from code review Co-authored-by: 8bitmp3 <19637339+8bitmp3@users.noreply.github.com> * Document split up. * Cleanup. * Apply suggestions from code review Co-authored-by: 8bitmp3 <19637339+8bitmp3@users.noreply.github.com> * Apply suggestions from code review Co-authored-by: Tommy Li <Tommy.chaoping.li@ibm.com> * Formatting improvements and cleanup. * format fixes * Apply suggestions from code review Co-authored-by: 8bitmp3 <19637339+8bitmp3@users.noreply.github.com> * Apply suggestions from code review Co-authored-by: 8bitmp3 <19637339+8bitmp3@users.noreply.github.com> Co-authored-by: 8bitmp3 <19637339+8bitmp3@users.noreply.github.com> Co-authored-by: Tommy Li <Tommy.chaoping.li@ibm.com> * Add RFMVasconcelos to OWNERS/approvers (#2539) * Deletes old redirects - pages do not exist anymore (#2552) * Move `AWS` platform under /distributions (#2551) * move /aws under /distributions * fix AWS redirects + add catch-all * update broken link (#2557) * UPDATE fix broken links to tensoflorw serving (#2558) * Move `Google` platform under /distributions (#2547) * move /gke folder to under /distributions * update redirects * Move `Azure` platform under /distributions (#2548) * mv /azure to /distributions * add catch-all azure to redirects * KFP - Update Python function-based component doc with param naming rules (#2544) * Describe pipeline param naming Adds notes on how the KFP SDK updates param names to describe the data instead of the implementation. Updates passing data by value to indicate that users can pass lists and dictionaries. * Update auto-gen Markdown Updates python-function-components.md with changes to python-function-components.ipynb. * Move `Openshift` platform under /distributions (#2550) * move /openshift to under /distributions * add openshift catch-all to redirects * Move `IBM` platform under /distributions (#2549) * move /ibm to under /distributions * Add IBM catch-all to redirects * [IBM] Update openshift kubeflow installation (#2560) * Make kfctl first distribution (#2562) * Move getting started on K8s page to under kfctl distribution (#2569) * mv overview to under kfctl * delete empty getting started with k8s section * Add redirect to catch traffic * Update GCP distribution OWNERS (#2574) * Update KFP shortcodes OWNERS (#2575) * Move MicroK8s to distributions (#2577) * create microk8s folder in distributions * move microk8s docs to distributions * update title * Add redirect for MicroK8s move - missed on #2577 (#2579) * Add Charmed Kubeflow Operators to list of available Kubeflow distributions (#2578) * Uplevel clouds for a level playing field * Add Owners + Index of Charmed Kubeflow * Add install page to Charmed Kubeflow distribution * Link to Charmed Kubeflow docs * Naming corrections * Update content/en/docs/distributions/charmed/install-kubeflow.md Co-authored-by: 8bitmp3 <19637339+8bitmp3@users.noreply.github.com> * Update content/en/docs/distributions/charmed/install-kubeflow.md Co-authored-by: 8bitmp3 <19637339+8bitmp3@users.noreply.github.com> * Update content/en/docs/distributions/charmed/install-kubeflow.md Co-authored-by: 8bitmp3 <19637339+8bitmp3@users.noreply.github.com> * Update content/en/docs/distributions/charmed/install-kubeflow.md Co-authored-by: 8bitmp3 <19637339+8bitmp3@users.noreply.github.com> * Update content/en/docs/distributions/charmed/install-kubeflow.md Co-authored-by: 8bitmp3 <19637339+8bitmp3@users.noreply.github.com> * Update content/en/docs/distributions/charmed/install-kubeflow.md Co-authored-by: 8bitmp3 <19637339+8bitmp3@users.noreply.github.com> * Update content/en/docs/distributions/charmed/install-kubeflow.md Co-authored-by: 8bitmp3 <19637339+8bitmp3@users.noreply.github.com> * Update content/en/docs/distributions/charmed/install-kubeflow.md Co-authored-by: 8bitmp3 <19637339+8bitmp3@users.noreply.github.com> * Update content/en/docs/distributions/charmed/install-kubeflow.md Co-authored-by: 8bitmp3 <19637339+8bitmp3@users.noreply.github.com> * Update content/en/docs/distributions/charmed/install-kubeflow.md Co-authored-by: 8bitmp3 <19637339+8bitmp3@users.noreply.github.com> * Update content/en/docs/distributions/charmed/install-kubeflow.md Co-authored-by: 8bitmp3 <19637339+8bitmp3@users.noreply.github.com> * Update content/en/docs/distributions/charmed/install-kubeflow.md Co-authored-by: 8bitmp3 <19637339+8bitmp3@users.noreply.github.com> * Update content/en/docs/distributions/charmed/install-kubeflow.md Co-authored-by: 8bitmp3 <19637339+8bitmp3@users.noreply.github.com> * Update content/en/docs/distributions/charmed/install-kubeflow.md Co-authored-by: 8bitmp3 <19637339+8bitmp3@users.noreply.github.com> * Update content/en/docs/distributions/charmed/install-kubeflow.md Co-authored-by: 8bitmp3 <19637339+8bitmp3@users.noreply.github.com> * final fixes Co-authored-by: 8bitmp3 <19637339+8bitmp3@users.noreply.github.com> * Fix broken link (#2581) * IBM Cloud docs: update pipelines SDK setup for single-user (#2571) Made the following changes to the instructions for setting up the pipelines SDK for single-user. * append '/pipeline' to the host string * add client.list_experiments to make sure the setup is working, consistent with the multi-user example in section 2 * add a note about KUBEFLOW_PUBLIC_ENDPOINT_URL since the user may or may not have exposed the endpoint as a LoadBalancer Signed-off-by: Chin Huang <chhuang@us.ibm.com> * update broken links / tweak names (#2583) * Move MiniKF to distributions (#2576) * create minikf folde + index * move minikf docs to minikf folder * Add redirects for external links * Change naming according to request * update description minikf * Clean up "Frameworks for training" + rename to "Training Operators" (#2584) * Remove outdated banners from Pytorch and TF * delete chainer * order TF and pyT up * rename "Frameworks for training" to "Training operators" * Fix broken link (#2580) * Remove "outdated" banners from MPI + MXnet operators (#2585) * docs: Update MPI and MXNet operator pages (#2586) Signed-off-by: terrytangyuan <terrytangyuan@gmail.com> * Pin the version of kustomize, v4 is not supported. (#2572) * Pin the version of kustomize, v4 is not supported. There are issues installing Kubeflow with version v4. Note: #2570 https://github.com/kubeflow/kubeflow/issues/5755 * Add refrence to manifest repo version. * Default to 3.2.0 * Update gke/anthos.md (#2591) * fix broken link (#2603) Co-authored-by: Prashant Sharma <prashsh1@in.ibm.com> Co-authored-by: 8bitmp3 <19637339+8bitmp3@users.noreply.github.com> Co-authored-by: Tommy Li <Tommy.chaoping.li@ibm.com> Co-authored-by: Mathew Wicks <thesuperzapper@users.noreply.github.com> Co-authored-by: JohanWork <39947546+JohanWork@users.noreply.github.com> Co-authored-by: Joe Liedtke <joeliedtke@google.com> Co-authored-by: Mofizur Rahman <moficodes@gmail.com> Co-authored-by: Yuan (Bob) Gong <4957653+Bobgy@users.noreply.github.com> Co-authored-by: Chin Huang <chhuang@us.ibm.com> Co-authored-by: brett koonce <koonce@gmail.com> Co-authored-by: Yuan Tang <terrytangyuan@gmail.com> Co-authored-by: drPytho <filip@voiapp.io> Co-authored-by: Ihor Sychevskyi <arhell333@gmail.com>

k8s-ci-robot added the do-not-merge/work-in-progress label Feb 8, 2021

k8s-ci-robot requested review from alfsuse and jinchihe February 8, 2021 18:03

google-cla bot added the cla: yes label Feb 8, 2021

k8s-ci-robot added the size/M label Feb 8, 2021

rui-vas force-pushed the methods branch from 36fd422 to e0eb687 Compare February 8, 2021 18:06

k8s-ci-robot added size/S and removed size/M labels Feb 8, 2021

k8s-ci-robot added size/M and removed size/S labels Feb 8, 2021

rui-vas changed the title ~~[WIP] - Create concept of "Methods & Distributions"~~ Create concept of "Methods & Distributions" Feb 11, 2021

k8s-ci-robot removed the do-not-merge/work-in-progress label Feb 11, 2021

rui-vas changed the title ~~Create concept of "Methods & Distributions"~~ Create "Methods & Distributions" with kfctl + Kubeflow Operator Feb 16, 2021

k8s-ci-robot assigned Bobgy Feb 16, 2021

rui-vas force-pushed the methods branch from b000304 to 118166d Compare February 16, 2021 11:06

k8s-ci-robot added size/L and removed size/M labels Feb 16, 2021

rui-vas force-pushed the methods branch from 118166d to 62bd6d5 Compare February 16, 2021 11:16

k8s-ci-robot added size/M and removed size/L labels Feb 16, 2021

rui-vas force-pushed the methods branch from 62bd6d5 to c190b69 Compare February 16, 2021 15:49

8bitmp3 reviewed Feb 18, 2021

View reviewed changes

content/en/docs/methods/_index.md Outdated Show resolved Hide resolved

8bitmp3 reviewed Feb 18, 2021

View reviewed changes

content/en/docs/methods/kfctl/_index.md Outdated Show resolved Hide resolved

rui-vas force-pushed the methods branch from 6c23bdd to e1a1b85 Compare March 11, 2021 11:14

rui-vas added 5 commits March 11, 2021 11:15

Add OWNERS for accountability

8f816ca

Update kfctl description Update content/en/docs/methods/_index.md

Add redirects for Operator

65a036d

Add redirects for kfctl

106079b

Rename "methods" to "distributions"

e8ead8e

update redirects to distributions as folder name

6e05d53

rui-vas force-pushed the methods branch from e1a1b85 to 6e05d53 Compare March 11, 2021 11:16

google-cla bot added cla: yes and removed cla: no labels Mar 11, 2021

Bobgy reviewed Mar 12, 2021

View reviewed changes

google-oss-robot added the lgtm label Mar 12, 2021

google-oss-robot removed the do-not-merge/hold label Mar 17, 2021

google-oss-robot added the approved label Mar 17, 2021

Merge branch 'master' into methods

dbbf10e

google-oss-robot removed the lgtm label Mar 17, 2021

google-oss-robot added the lgtm label Mar 17, 2021

google-oss-robot merged commit c792fe5 into kubeflow:master Mar 17, 2021

thesuperzapper mentioned this pull request Apr 9, 2021

Refactor the "Getting Started" "Installing Kubeflow" page #2590

Closed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Create "Distributions" with kfctl + Kubeflow Operator #2492

Create "Distributions" with kfctl + Kubeflow Operator #2492

rui-vas commented Feb 8, 2021

rui-vas commented Feb 16, 2021

rui-vas commented Feb 16, 2021

8bitmp3 commented Feb 18, 2021

rui-vas commented Feb 18, 2021

rui-vas commented Feb 18, 2021

google-cla bot commented Mar 10, 2021

google-cla bot commented Mar 11, 2021

rui-vas commented Mar 11, 2021

Bobgy left a comment

joeliedtke commented Mar 16, 2021

Bobgy commented Mar 17, 2021

Bobgy commented Mar 17, 2021

google-oss-robot commented Mar 17, 2021

rui-vas commented Mar 17, 2021

Bobgy commented Mar 17, 2021

Create "Distributions" with kfctl + Kubeflow Operator #2492

Create "Distributions" with kfctl + Kubeflow Operator #2492

Conversation

rui-vas commented Feb 8, 2021

rui-vas commented Feb 16, 2021

rui-vas commented Feb 16, 2021

8bitmp3 commented Feb 18, 2021

rui-vas commented Feb 18, 2021

rui-vas commented Feb 18, 2021

google-cla bot commented Mar 10, 2021

google-cla bot commented Mar 11, 2021

rui-vas commented Mar 11, 2021

Bobgy left a comment

Choose a reason for hiding this comment

joeliedtke commented Mar 16, 2021

Bobgy commented Mar 17, 2021

Bobgy commented Mar 17, 2021

google-oss-robot commented Mar 17, 2021

rui-vas commented Mar 17, 2021

Bobgy commented Mar 17, 2021