Skip to content

feat(CLI): add a check requiring the gateway API#15409

Merged
adleong merged 7 commits into
mainfrom
alex/gw-check
Jul 14, 2026
Merged

feat(CLI): add a check requiring the gateway API#15409
adleong merged 7 commits into
mainfrom
alex/gw-check

Conversation

@adleong

@adleong adleong commented Jun 24, 2026

Copy link
Copy Markdown
Member

When Linkerd is installed with the CLI, we first check to ensure the Gateway API CRDs are installed, and abort if the Gateway API is not present. However, if Helm is used to the install Linkerd, no such check is performed. This can lead to an invalid installation because HTTPRoute is a required CRD for Linkerd.

We add a check to linkerd check that validates that the Gateway API is installed.

But note the prior art in: #12917

In that PR we removed a check for the Gateway API. This is because the Gateway API consists of many different CRDs, most of which are optional for Linkerd; only HTTPRoute is required. Therefore, we don't want to require that all Gateway API CRDs are installed. For this new check, the HTTPRoute CRD alone is sufficient to make it pass. This matches the behavior of the condition checked by the linkerd install command.

Signed-off-by: Alex Leong <alex@buoyant.io>
@adleong adleong requested a review from a team as a code owner June 24, 2026 20:24
adleong added 5 commits July 2, 2026 19:04
Signed-off-by: Alex Leong <alex@buoyant.io>
Signed-off-by: Alex Leong <alex@buoyant.io>
Signed-off-by: Alex Leong <alex@buoyant.io>
Signed-off-by: Alex Leong <alex@buoyant.io>
Signed-off-by: Alex Leong <alex@buoyant.io>
Comment thread pkg/healthcheck/healthcheck.go Outdated
[]Checker{
{
description: "Gateway API CRDs are installed",
hintAnchor: "l5d-existence-crd",

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I think we should use a different anchor so the troubleshooting docs can speak more specifically about the gateway api.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

good point

Signed-off-by: Alex Leong <alex@buoyant.io>
@adleong adleong merged commit 6ed4384 into main Jul 14, 2026
121 of 128 checks passed
@adleong adleong deleted the alex/gw-check branch July 14, 2026 21:48
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