Skip to content
This repository has been archived by the owner on Jun 29, 2022. It is now read-only.

Dex convert to helm chart and update to v2.25.0 #962

Merged
merged 2 commits into from
Sep 15, 2020

Conversation

knrt10
Copy link
Member

@knrt10 knrt10 commented Sep 14, 2020

closes: #950 #948
Signed-off-by: knrt10 kautilya@kinvolk.io

@knrt10 knrt10 linked an issue Sep 14, 2020 that may be closed by this pull request
@knrt10 knrt10 force-pushed the knrt10/dex-convert-to-helm branch 2 times, most recently from 8953e0d to 11c34eb Compare September 14, 2020 09:36
assets/charts/components/dex/templates/00-common.yaml Outdated Show resolved Hide resolved
assets/charts/components/dex/templates/04-deployment.yaml Outdated Show resolved Hide resolved
assets/charts/components/dex/templates/05-ingress.yaml Outdated Show resolved Hide resolved
assets/charts/components/dex/templates/05-ingress.yaml Outdated Show resolved Hide resolved
assets/charts/components/dex/templates/05-ingress.yaml Outdated Show resolved Hide resolved
assets/charts/components/dex/values.yaml Outdated Show resolved Hide resolved
assets/charts/components/dex/Chart.yaml Show resolved Hide resolved
pkg/components/dex/manifest.go Outdated Show resolved Hide resolved
pkg/components/dex/component_test.go Outdated Show resolved Hide resolved
closes: #950 #948
Signed-off-by: knrt10 <kautilya@kinvolk.io>
@knrt10 knrt10 force-pushed the knrt10/dex-convert-to-helm branch 3 times, most recently from 7ded411 to de9c695 Compare September 14, 2020 10:20
invidian
invidian previously approved these changes Sep 15, 2020
Copy link
Member

@invidian invidian left a comment

Choose a reason for hiding this comment

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

Cool, LGTM

surajssd
surajssd previously approved these changes Sep 15, 2020
Copy link
Member

@surajssd surajssd left a comment

Choose a reason for hiding this comment

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

LGTM

pkg/components/dex/component_test.go Outdated Show resolved Hide resolved
Signed-off-by: knrt10 <kautilya@kinvolk.io>
Copy link
Member

@surajssd surajssd left a comment

Choose a reason for hiding this comment

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

LGTM

@knrt10 knrt10 merged commit bf777cd into master Sep 15, 2020
@knrt10 knrt10 deleted the knrt10/dex-convert-to-helm branch September 15, 2020 10:22
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

dex: Convert to helm chart Update dex to v2.25.0
3 participants