Skip to content

feat: Allow to use HTTPRoute from Gateway API.#164

Merged
sagikazarmark merged 1 commit into
dexidp:masterfrom
mirozoe:master
May 28, 2026
Merged

feat: Allow to use HTTPRoute from Gateway API.#164
sagikazarmark merged 1 commit into
dexidp:masterfrom
mirozoe:master

Conversation

@mirozoe
Copy link
Copy Markdown
Contributor

@mirozoe mirozoe commented Apr 27, 2026

Overview

Support for kubernetes Gateway API, customer can use HTTPRoute instead of Ingress (all credits goes to @abelhoula).

What this PR does / why we need it

HTTPRoute manifest added.

Special notes for your reviewer

Checklist

  • [/] Change log updated in Chart.yaml (see the contributing guide for details)
  • [/] Chart version bumped in Chart.yaml (see the contributing guide for details)
  • [/] Documentation regenerated by running make docs

@mirozoe
Copy link
Copy Markdown
Contributor Author

mirozoe commented Apr 27, 2026

This is PR derived from this reflecting feedback from @cnvergence.

Comment thread charts/dex/templates/httproute.yaml Outdated
@@ -0,0 +1,41 @@
{{- if .Values.httproute.enabled -}}
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

httpRoute here

Comment thread charts/dex/README.md Outdated
| ingress.annotations | object | `{}` | Annotations to be added to the ingress. |
| ingress.hosts | list | See [values.yaml](values.yaml). | Ingress host configuration. |
| ingress.tls | list | See [values.yaml](values.yaml). | Ingress TLS configuration. |
| httproute.enabled | bool | `false` | Enable Gateway API HTTPRoute. Gateway API support is in EXPERIMENTAL status. Support depends on your Gateway controller implementation. See the [Gateway API documentation](https://gateway-api.sigs.k8s.io/) for details. |
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

same here :)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Thank you, overlooked it. Should be fine now.

Signed-off-by: Miroslav Hlavka <miroslav.hlavka@goto.com>
@cnvergence
Copy link
Copy Markdown

@sagikazarmark ptal :)

@mirozoe
Copy link
Copy Markdown
Contributor Author

mirozoe commented May 4, 2026

@sagikazarmark could I get your review please? 🙇

@mirozoe
Copy link
Copy Markdown
Contributor Author

mirozoe commented May 11, 2026

@cnvergence could you possibly recommend another maintainer? Thank you.

@cnvergence
Copy link
Copy Markdown

@mirozoe sorry, I am not familiar with the structure here, you could try to reach out at Dex slack channels

@sagikazarmark
Copy link
Copy Markdown
Member

Thanks! Sorry it took so long.

As a rule of thumb: if I don't reply, it's not because I don't care. It's because I didn't get the notification. It's the unfortunate consequence of the world we live in.

So feel free to keep pinging me wherever you can get a hold of me. :)

@sagikazarmark sagikazarmark merged commit 205e521 into dexidp:master May 28, 2026
10 checks passed
@mirozoe
Copy link
Copy Markdown
Contributor Author

mirozoe commented May 28, 2026

Thank you @sagikazarmark , no worries at all ;)

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.

3 participants