Skip to content

[chart/v1-2x-test] Improve API server HTTPRoute configuration (#68552)#69579

Open
steveahnahn wants to merge 1 commit into
apache:chart/v1-2x-testfrom
steveahnahn:backport-68552-to-chart-v1-2x
Open

[chart/v1-2x-test] Improve API server HTTPRoute configuration (#68552)#69579
steveahnahn wants to merge 1 commit into
apache:chart/v1-2x-testfrom
steveahnahn:backport-68552-to-chart-v1-2x

Conversation

@steveahnahn

@steveahnahn steveahnahn commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Backport of #68552

  • Add config guards for the API server HTTPRoute: mutual exclusivity with the API server Ingress, and required parentRefs enforced via schema minItems
  • Move httpRoute configuration under apiServer

One conflict in chart/templates/api-server/api-server-ingress.yaml: kept this branch's Ingress enablement condition (which retains the semverCompare / global ingress.enabled handling) and applied only the new HTTPRoute mutual-exclusion guard. Verified with the full apiserver helm-test group (125 passed) on this branch.


Was generative AI tooling used to co-author this PR?
  • Yes (please specify the tool below)
    • Claude Code (Opus 4.8)

Generated-by: Claude Code (Opus 4.8) following the guidelines

@potiuk

potiuk commented Jul 8, 2026

Copy link
Copy Markdown
Member

Closed/re-opened to re-trigger the jobs after #69626

* Move API server HTTPRoute under apiServer and add config guards

* Require HTTPRoute parentRefs and document API server Ingress mutual exclusivity

* Fail fast on empty HTTPRoute parentRefs via schema minItems

* Apply pretty-format-json formatting to values.schema.json

(cherry picked from commit d2d9f4c)
@potiuk potiuk force-pushed the backport-68552-to-chart-v1-2x branch from 9a2cce2 to 59459de Compare July 8, 2026 21:40

@Miretpl Miretpl left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks!

@Miretpl

Miretpl commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

I will wait with merge till the CI finishes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants