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

Make Argo UI available publicly at testing-argo.kubeflow.ui #132

Merged
merged 4 commits into from Jan 19, 2018

Conversation

jlewi
Copy link
Contributor

@jlewi jlewi commented Jan 17, 2018

* We use Argo to run our E2E tests so the UI is very useful for debugging tests.
* Add an ingress with a static IP to expose it publicly.
@jlewi jlewi merged commit 698cc67 into kubeflow:master Jan 19, 2018
jlewi added a commit to jlewi/kubeflow that referenced this pull request Jan 29, 2018
commit f10c384
Author: Jeremy Lewi <jeremy@lewi.us>
Date:   Thu Jan 25 17:11:18 2018 -0800

    Need to set authenticator param.

commit 705067f
Author: Jeremy Lewi <jeremy@lewi.us>
Date:   Thu Jan 25 15:53:08 2018 -0800

    Fix some bugs and start a user guide.

commit cbf67a3
Merge: c60c245 fed20a1
Author: Jeremy Lewi <jeremy@lewi.us>
Date:   Thu Jan 25 14:31:46 2018 -0800

    Sync to head.

commit c60c245
Author: Jeremy Lewi <jeremy@lewi.us>
Date:   Thu Jan 25 14:20:47 2018 -0800

    * Use Envoy as a reverse proxy and for JWT verification.
    * Don't run Envoy as a side car in the JupyterHub pod. This is cleaner
      and allows us to have a single reverse proxy for all services.

commit fed20a1
Author: Elson Rodriguez <elson.rodriguez@gmail.com>
Date:   Thu Jan 25 06:28:22 2018 -0800

    Changing Jupyter service to ClusterIP by default. (kubeflow#139)

    * Changing default Service type for Jupyterhub to Cluster IP
       * Exposing services publicly is a security risk so we want to avoid recommending that since people may not understand the implications
    * Updated documentation to reflect ClusterIP change for Jupyter.

commit 8aaa393
Author: lluunn <lunkai@google.com>
Date:   Thu Jan 25 06:21:55 2018 -0800

    fix guide typo (kubeflow#144)

commit 7bb642c
Author: Robert Wilkins III <1147229+genome21@users.noreply.github.com>
Date:   Tue Jan 23 23:05:10 2018 -0600

    Update README.md (kubeflow#142)

    Revised grammer and punctuation changes to the document as detailed below:

    Added (ML) after "Machine Learning" on line 3 to avoid ambiguation over the "ML" reference on line 18.

    Changed "best of breed" to "best-of-breed" on line 3 to match language used in Kubernetes/Kubernetes.

    Removed the comma after GPUs on line 6 to avoid treating a regular sentence as a run-on sentence.

    Changed "it is" to "it's" on line 23.  There's no need to get formal here if the rest of the document has a light conversational feel.

    Removed an unnecessary comma from Line 23 after (within reason).

    Added dashes in "easy to use" on line 25 to match formatting from line 3.

    Added a colon after "using Kubeflow if" on line 30 to match the format previously established in the previous section called "The Kubeflow Mission".

    Capitalized Kubeflow on line 34 to match casing from line 30.

    Changed the sentence structure and added a period for line 36 to make more sense rather than a strange run-on sentence.

    Removed unnecessary newline between lines 36 and 37 to match format established in line 34.

    Added a period after (see below) on line 39.

    Added a comma after "GPUs" to avoid a run-on sentence on line 47.

    Changed the ending comma on line 58 to a colon to follow the established format used previously in this document.

    Added a comma after Kubeflow on line 83 and added a period to finish the sentence.

    Moved the comma after "using" to after "GKE" in line 109for the sentence to make sense.

    Added a dash for "in depth" on line to follow the document's established format.

commit 80cb162
Author: Neeraj Kashyap <nkash@google.com>
Date:   Mon Jan 22 05:31:32 2018 -0800

    Client script for inception model server (kubeflow#92)

    Added a script that allows users to run the hosted inception model on
    images on their local filesystems or on Google Cloud Storage.

    This is, with only very slight modifications for readability, the same
    as the client provided by TensorFlow Serving -
    https://github.com/tensorflow/serving/blob/master/tensorflow_serving/example/inception_client.py

    As such, I am completely okay with us just linking to their script.

    My initial intention was to make this as a notebook, but the problem is
    that the tensorflow-serving-api Python package is only available for
    Python 2 and the kubeflow-core environment only offers a Python 3
    backend for the Jupyter notebook.

    This is therefore a stopgap until I can introduce an appropriate image
    in place of the one used by kubeflow-core.

    * Changed model serving service type to ClusterIP from LoadBalancer

    * Added instructions for exposing service IP

commit bde2ddc
Author: Putra Manggala <pmangg@users.noreply.github.com>
Date:   Fri Jan 19 17:34:59 2018 -0500

    Fix datascientists typo (kubeflow#137)

commit 698cc67
Author: Jeremy Lewi <jeremy@lewi.us>
Date:   Thu Jan 18 20:45:49 2018 -0800

    Make Argo UI available publicly at testing-argo.kubeflow.ui (kubeflow#132)

    * We use Argo to run our E2E tests so the UI is very useful for debugging tests.

    * Add an ingress with a static IP to expose it publicly.

    * Fix kubeflow#131

commit 55c220d
Merge: 11d989c ca95a0d
Author: Jeremy Lewi <jeremy@lewi.us>
Date:   Wed Jan 17 21:24:58 2018 -0800

    Merge remote-tracking branch 'github/iap' into iap

commit 11d989c
Merge: 8e6fb87 4c9217d
Author: Jeremy Lewi <jeremy@lewi.us>
Date:   Wed Jan 17 21:24:34 2018 -0800

    Resolve conflicts.

commit 4c9217d
Author: Jeremy Lewi <jeremy@lewi.us>
Date:   Wed Jan 17 20:50:24 2018 -0800

    Fix TfJob operator roles and TfCNN prototype (kubeflow#130)

    * Fix the TFCNN prototype; the termination policy wasn't being properly set

    * Create service accounts and role bindings for the TfJob operator and UI

    * Fix kubeflow#129 TfCnn template doesn't set termination policy correctly

    * Fix kubeflow#125 Missing roles for tf-job operator

    * Fix kubeflow#95; presubmits/postsubmits need to use the code at the commit we checked out

    *We do this by replacing the directory in vendor with a symbolic link to where we checked out the source.
       * It looks like using "--as" with ksonnet leads to strange errors about the server not being able to create the config map
       * If we don't use "--as" need to fetch credentials a second time or else we get RBAC issues creating the cluster
elenzio9 pushed a commit to arrikto/kubeflow that referenced this pull request Oct 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants