Skip to content

Conversation

migmartri
Copy link
Member

@migmartri migmartri commented Jun 14, 2025

This PR apart of fixing the hardcoding DNS issue when the chart is deployed with another release name #1578, it allows using dex in a chart deployed in development mode #2107

It includes

  • Ingress support in dex subchart
  • Configuration update so it now takes into account that ingress and fixes the aforementioned issue.
  • Dex upgrade
  • Documentation in readme

To use it, just configure like this

dex:
  dex:
    # Point to the [controlplane http ingress]/auth/callback
    redirectURL: https://cp-miguel.k.chainloop.dev/auth/callback
    ingress:
      enabled: true
      tls: true
      hostname: dex-miguel.k.chainloop.dev

Fixes #1578 and #2107

Signed-off-by: Miguel Martinez <miguel@chainloop.dev>
Signed-off-by: Miguel Martinez <miguel@chainloop.dev>
Signed-off-by: Miguel Martinez <miguel@chainloop.dev>
Signed-off-by: Miguel Martinez <miguel@chainloop.dev>
@migmartri migmartri requested review from danlishka and javirln June 14, 2025 17:03
Signed-off-by: Miguel Martinez <miguel@chainloop.dev>
Signed-off-by: Miguel Martinez <miguel@chainloop.dev>
@migmartri migmartri requested a review from jiparis June 15, 2025 20:28
Signed-off-by: Miguel Martinez <miguel@chainloop.dev>
Signed-off-by: Miguel Martinez Trivino <miguel@chainloop.dev>
Signed-off-by: Miguel Martinez Trivino <miguel@chainloop.dev>
@migmartri migmartri merged commit f074544 into chainloop-dev:main Jun 16, 2025
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

dex dns seems to be hardcoded to chainloop
2 participants