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 ambassador and statsd to 0.30.1 as part of 0.1 release #513

Merged
merged 2 commits into from Mar 27, 2018
Merged

Update ambassador and statsd to 0.30.1 as part of 0.1 release #513

merged 2 commits into from Mar 27, 2018

Conversation

ankushagarwal
Copy link
Contributor

@ankushagarwal ankushagarwal commented Mar 27, 2018

Deployed kubeflow on a new cluster and tested new ambassador by
deploying an inception model and routing traffic through ambassador

Related to #506


This change is Reviewable

Deployed kubeflow on a new cluster and tested new ambassador by
deploying an inception model and routing traffic through ambassador

Related to #506
@ankushagarwal
Copy link
Contributor Author

/cc @jlewi

@k8s-ci-robot k8s-ci-robot requested a review from jlewi March 27, 2018 04:12
@jlewi
Copy link
Contributor

jlewi commented Mar 27, 2018

Looks like there was a jsonnet failure.

@jlewi
Copy link
Contributor

jlewi commented Mar 27, 2018

/lgtm
/approve

@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: jlewi

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 0fac52a into kubeflow:master Mar 27, 2018
@ankushagarwal ankushagarwal deleted the ambassador-version branch March 29, 2018 02:18
yanniszark pushed a commit to arrikto/kubeflow that referenced this pull request Nov 1, 2019
…ents (kubeflow#513)

* On GCP use BackendConfig to set the backend timeout and other improvements.

* As of GKE 1.13 we can now set the backend timeout using a BackendConfig
  https://cloud.google.com/kubernetes-engine/docs/how-to/configure-backend-service

* BasicAuth doesn't have a backendconfig so we need to add one.

  * In BasicAuth we are still using Ambassador to handle the auth.

* Improve the setup_backend.sh script for IAP

  * This script is only updating the JWT based on IAP; there's no reason
    the script should be checking if the backend timeout is set.

  * Make the script run for ever to avoid confusing container restarts

* Related to kubeflow#4269 backend timeout not set
* Related to kubeflow#4043 crash looping of iap enabler and backend_updater pods.

* Update tests.

* Update Ambassador test.
saffaalvi pushed a commit to StatCan/kubeflow that referenced this pull request Feb 11, 2021
…ow#513)

* Update ambassador and statsd to 0.30.1 as part of 0.1 release

Deployed kubeflow on a new cluster and tested new ambassador by
deploying an inception model and routing traffic through ambassador

Related to kubeflow#506

* Fix jsonnet test
elenzio9 pushed a commit to arrikto/kubeflow that referenced this pull request Oct 31, 2022
Co-authored-by: hackerboy01 <penglei031303@gmail.com>
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.

None yet

3 participants