dex-k8s-authenticator icon indicating copy to clipboard operation
dex-k8s-authenticator copied to clipboard

Dex chart should be removed from repository

Open solovyevt opened this issue 6 years ago • 8 comments

It is redundant to keep Dex Helm chart in this repository, we should leave dex-k8s-authenticator chart and a good explanation of how to integrate it with Dex chart taken from stable/Dex repository.

solovyevt avatar Oct 18 '18 14:10 solovyevt

#34

I tend to agree. I'll work on some docs.

nabadger avatar Oct 18 '18 19:10 nabadger

Moving forward with this:

  • Drop dex chart and current documentation for it in this repo
  • Add new documentation
    • Direct users to https://github.com/helm/charts/tree/master/stable/dex
    • Update examples/docs to show how you can use both

nickmintel avatar Mar 22 '19 07:03 nickmintel

#92

nabadger avatar Mar 22 '19 08:03 nabadger

See https://github.com/helm/charts/blob/master/CONTRIBUTING.md#technical-requirements

Some things to note:

  • dex-k8s-authenticator currently goes into crash state until it can resolve dex-idp
  • Add a requirements.yaml and make dexidp a dependency

nabadger avatar Mar 22 '19 08:03 nabadger

It would be a good excuse to get dex-k8s-authenticator listed in hub.helm.sh (https://github.com/helm/hub/blob/master/Repositories.md)!

desaintmartin avatar Apr 16 '19 07:04 desaintmartin

I was about to migrate to stable/dex chart but as of version 2.0.0 that chart only exposes subset of configuration.
I could not use frontend: subsection and that's a bummer. I'd like you to reconsider dropping dex chart from this repository. At least for a while.

kostyrev avatar Aug 22 '19 16:08 kostyrev

@kostyrev would you be able to create a feature-request on the upstream dex repo for this? I can then track it and base any decision from it too.

I haven't managed to find much time to progress this in anycase just yet as other things keep coming up :)

Keep in mind the existing tagged releases will still be around.

nabadger avatar Aug 22 '19 16:08 nabadger

you could use https://github.com/helm/charts/issues/16351

kostyrev avatar Aug 22 '19 18:08 kostyrev