Skip to content

Added GRPCRoutes, TCPRoutes, additional fixes.#221

Merged
gyoza merged 15 commits into
mainfrom
develop
May 19, 2026
Merged

Added GRPCRoutes, TCPRoutes, additional fixes.#221
gyoza merged 15 commits into
mainfrom
develop

Conversation

@gyoza
Copy link
Copy Markdown
Contributor

@gyoza gyoza commented May 19, 2026

Added GRPCRoutes, TCPRoutes, additional fixes.

gyoza added 15 commits April 10, 2026 14:33
…sources

moving chart to correct location
…sources

Cli 54049/add gateway api spec resources
…sources

Cli 54049/add gateway api spec resources
…ndTLSPolicy

Adds templates for GRPCRoute, TCPRoute, TLSRoute, UDPRoute, and
BackendTLSPolicy. All rendered automatically via renderIngress when
global.gatewayAPI.enabled is true. Fixes nil pointer safety across
all gateway templates by switching to dig-based access.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

AI-Session-Id: 4f9cdbba-9363-442e-a473-42572bf70f96
AI-Tool: claude-code
AI-Model: unknown
…is true

Bare "*" is rejected by Gateway API CRD hostname validation (regex requires
"*.<domain>" or specific hostname). Omitting the field lets HTTPRoute inherit
from the parent Gateway listener.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

AI-Session-Id: 4f9cdbba-9363-442e-a473-42572bf70f96
AI-Tool: claude-code
AI-Model: unknown
Gateway API CRD hostname validation rejects bare "*". Filter it out of
hosts, global additionalHostnames, and per-route additionalHostnames.
When the resulting list is empty, omit the hostnames field so the route
inherits from the parent Gateway listener.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

AI-Session-Id: 4f9cdbba-9363-442e-a473-42572bf70f96
AI-Tool: claude-code
AI-Model: unknown
Adding GRPCroute and TCProute due to future need from pipeline team.
…sources

Cli 54049/add gateway api spec resources
@ebn-harness
Copy link
Copy Markdown
Contributor

Validated across all of HarnessCore the previous issue involving host * . Will only affect services if enabled and defualts to false.

Copy link
Copy Markdown
Contributor

@ebn-harness ebn-harness left a comment

Choose a reason for hiding this comment

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

LGTM

@gyoza gyoza merged commit c883357 into main May 19, 2026
5 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