Skip to content

[prometheus-json-exporter] Add Gateway API HTTPRoute support - #7022

Merged
zanhsieh merged 3 commits into
prometheus-community:mainfrom
somaz94:feat/json-exporter-httproute
Aug 31, 2026
Merged

[prometheus-json-exporter] Add Gateway API HTTPRoute support#7022
zanhsieh merged 3 commits into
prometheus-community:mainfrom
somaz94:feat/json-exporter-httproute

Conversation

@somaz94

@somaz94 somaz94 commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

Add Gateway API HTTPRoute support to the prometheus-json-exporter chart, mirroring the pattern already merged for the prometheus, prometheus-pushgateway, alertmanager, and prometheus-blackbox-exporter charts.

The HTTPRoute is opt-in and disabled by default, so the default render is unchanged.

Validation:

  • helm template renders nothing when route.main.enabled=false (default)

  • helm template renders a valid HTTPRoute with ci/httproute-values.yaml (parentRefs, hostnames, filters, additionalRules)

  • httpsRedirect=true renders only the RequestRedirect filter

  • ct lint --config .github/linters/ct.yaml passes

  • ct install on a kind cluster (with Gateway API CRDs applied) installs both the default and httproute scenarios successfully

  • DCO signed

Signed-off-by: somaz <genius5711@gmail.com>
@somaz94
somaz94 marked this pull request as ready for review June 24, 2026 03:40
@somaz94

somaz94 commented Aug 4, 2026

Copy link
Copy Markdown
Contributor Author

Hi @zanhsieh — thanks for the approval. This one's been green and mergeable since, so just flagging it in case it slipped through. Happy to rebase or adjust anything if needed.

@somaz94

somaz94 commented Aug 18, 2026

Copy link
Copy Markdown
Contributor Author

@zanhsieh thanks for the review. Green and mergeable since early August, so this looks ready whenever someone has a moment to merge it.

@zanhsieh
zanhsieh merged commit 03f14d0 into prometheus-community:main Aug 31, 2026
10 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.

2 participants