Skip to content

[prometheus-yet-another-cloudwatch-exporter] Add Gateway API HTTPRoute support - #7030

Open
somaz94 wants to merge 1 commit into
prometheus-community:mainfrom
somaz94:feat/prometheus-yet-another-cloudwatch-exporter-httproute
Open

[prometheus-yet-another-cloudwatch-exporter] Add Gateway API HTTPRoute support#7030
somaz94 wants to merge 1 commit into
prometheus-community:mainfrom
somaz94:feat/prometheus-yet-another-cloudwatch-exporter-httproute

Conversation

@somaz94

@somaz94 somaz94 commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

Add Gateway API HTTPRoute support to the prometheus-yet-another-cloudwatch-exporter chart, mirroring the pattern already merged for the prometheus, prometheus-pushgateway, alertmanager, prometheus-blackbox-exporter, and prometheus-json-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 (numeric backend port, parentRefs, hostnames, filters, additionalRules)

  • httpsRedirect=true renders only the RequestRedirect filter

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

  • helm install --dry-run=server against a kind cluster with Gateway API CRDs validates the rendered HTTPRoute against the live CRD schema

  • DCO signed

@somaz94
somaz94 marked this pull request as ready for review June 24, 2026 05:09
…e support

Signed-off-by: somaz <genius5711@gmail.com>
@somaz94
somaz94 force-pushed the feat/prometheus-yet-another-cloudwatch-exporter-httproute branch from 7793ed1 to 82ec9cc Compare July 28, 2026 02:47
@somaz94

somaz94 commented Aug 3, 2026

Copy link
Copy Markdown
Contributor Author

Gentle ping @cristiangreco @tristanburgess @thomaspeitz @andriikushch — this has been open since Jun 24 without a review yet.

Everything the repo asks for is in place: Chart Testing, the Integration Tests against ci/httproute-values.yaml, Super Linter, Check Generated Files and Check Title all pass, DCO is signed, and the chart version is bumped (0.47.0 → 0.48.0, still ahead of main).

The change is opt-in and off by default — with httpRoute.enabled: false the rendered output is identical to today's, so it carries no risk for existing users. Happy to rebase onto main if that helps; the branch is a bit behind but merges cleanly.

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.

1 participant