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

POC: Hydra + Oathkeeper #3878

Merged
merged 25 commits into from
May 7, 2019
Merged

Conversation

jakkab
Copy link
Contributor

@jakkab jakkab commented Apr 26, 2019

Description

Changes proposed in this pull request:

  • Add Hydra chart

Related issue(s)
#3399

@jakkab jakkab added kind/feature Categorizes issue or PR as related to a new feature. area/security Issues or PRs related to security area/service-mesh Issues or PRs related to service-mesh labels Apr 26, 2019
@jakkab jakkab added this to the Sprint_Goat_11 milestone Apr 26, 2019
@jakkab jakkab changed the title Hydra oathkeeper poc POC: Hydra + Oathkeeper Apr 26, 2019
@strekm
Copy link
Contributor

strekm commented May 6, 2019

/test pre-master-kyma-integration

Copy link

@piotrmsc piotrmsc left a comment

Choose a reason for hiding this comment

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

First bunch of review

resources/ory/charts/hydra/templates/oauth2-svc.yaml Outdated Show resolved Hide resolved
resources/ory/charts/hydra/templates/login-consent.yaml Outdated Show resolved Hide resolved
resources/ory/charts/hydra/values.yaml Outdated Show resolved Hide resolved
resources/ory/charts/hydra/values.yaml Outdated Show resolved Hide resolved
resources/ory/charts/hydra/templates/oauth2.yaml Outdated Show resolved Hide resolved
resources/ory/charts/hydra/templates/oauth2.yaml Outdated Show resolved Hide resolved
resources/ory/charts/hydra/values.yaml Outdated Show resolved Hide resolved
resources/ory/charts/hydra/values.yaml Outdated Show resolved Hide resolved
resources/ory/charts/oathkeeper/templates/api-server.yaml Outdated Show resolved Hide resolved
@piotrmsc
Copy link

piotrmsc commented May 7, 2019

In the first iteration let's leave oathkeeper apiserver exposed like it is right now, we will think of using only one host in the future

@jakkab
Copy link
Contributor Author

jakkab commented May 7, 2019

/test pre-master-kyma-gke-upgrade

@jakkab jakkab merged commit 8864d76 into kyma-project:master May 7, 2019
@jakkab jakkab deleted the hydra-oathkeeper-poc branch May 7, 2019 13:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/security Issues or PRs related to security area/service-mesh Issues or PRs related to service-mesh kind/feature Categorizes issue or PR as related to a new feature.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants