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.

Sign up for GitHub

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

Jump to bottom

Replace two envoy containers with one in the envoy pod #404

Closed
ankushagarwal opened this issue Mar 9, 2018 · 0 comments
Closed

Replace two envoy containers with one in the envoy pod #404

ankushagarwal opened this issue Mar 9, 2018 · 0 comments
Assignees
@ankushagarwal
Labels
priority/p2

Comments

@ankushagarwal
Copy link
Contributor

envoy now supports specifying paths on which we should use JWT validation. We can use this feature to do JWT validation on the non-healthcheck routes. This will allows us to replace the two envoy containers with a single one.
@ankushagarwal ankushagarwal self-assigned this Mar 9, 2018
@ankushagarwal ankushagarwal mentioned this issue Mar 9, 2018
Merged
k8s-ci-robot pushed a commit that referenced this issue Mar 9, 2018
@k8s-ci-robot
Replace two envoy containers with a single one (#407)
6702151 
* Replace two envoy containers with a single one

Use bypass_jwt field to bypass jwt validation for /healthz and
/noiap/whoami routes

Tested by disabling IAP - verified that /healthz and /noiap/whoami work
and other routes return an error - Required JWT token is missing

Fixes #404

* Autoformat iap.libsonnet
yanniszark added a commit to arrikto/kubeflow that referenced this issue Nov 1, 2019
@yanniszark @k8s-ci-robot
kfdef: existing_arrikto: remove metacontroller application (kubeflow#404
d48bb0c 
)

Remove the metacontroller application from the existing_arrikto
config as it is no longer used.

Signed-off-by: Yannis Zarkadas <yanniszark@arrikto.com>
yanniszark pushed a commit to arrikto/kubeflow that referenced this issue Feb 15, 2021
@DeeperMind @k8s-ci-robot
Add support for parallel studyjobs (kubeflow#404)
4d8c599 
* Add support for parallel studyjobs

* fix a typo

* Reorganize the program a little bit

* fix a typo

* fix a typo
elenzio9 pushed a commit to arrikto/kubeflow that referenced this issue Oct 31, 2022
Remove deprecated github teams (kubeflow#404)
98be0d4
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ankushagarwal ankushagarwal

Labels
priority/p2
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@ankushagarwal