Skip to content

Commit 105b61d

Browse files
authored
feat(gatewayAPI): Stop installing the gateway API by default (#14523)
We switch the default value of `installGatewayAPI` from true to false. This has the effect that for Helm installs and upgrades, we will no longer install the gateway API CRDs by default and will instead depend on them already being installed on the cluster. For Helm upgrades from 2.18, this is a graceful upgrade because 2.18 added the `helm.sh/resource-policy=keep` annotation to these CRDs. This means that even though this default is changing, upgrading from 2.18 will not cause Helm to delete these CRDs. This change has no effect on the Linkerd CLI which was already not installing gateway API CRDs by default. Signed-off-by: Alex Leong <[email protected]>
1 parent 810cc94 commit 105b61d

File tree

5 files changed

+24
-25836
lines changed

5 files changed

+24
-25836
lines changed

charts/linkerd-crds/values.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
# -- Controls if Linkerd should install the Gateway API CRDs. This value acts
22
# as a default and can be overridden by the more specific `enable*Routes`
33
# values.
4-
installGatewayAPI: true
4+
installGatewayAPI: false
55
# -- Controls if Linkerd should install the Gateway API HTTPRoutes and
66
# GRPCRoutes CRDs. If unspecified, the value of `installGatewayAPI` is used.
77
# enableHttpRoutes: false

cli/cmd/install_test.go

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,16 @@ const (
2222
installProxyVersion = "install-proxy-version"
2323
installControlPlaneVersion = "install-control-plane-version"
2424
installDebugVersion = "install-debug-version"
25+
26+
externalGatewayAPIManifest = `---
27+
apiVersion: apiextensions.k8s.io/v1
28+
kind: CustomResourceDefinition
29+
metadata:
30+
name: httproutes.gateway.networking.k8s.io
31+
spec:
32+
versions:
33+
- name: v1
34+
`
2535
)
2636

2737
func TestRender(t *testing.T) {
@@ -300,16 +310,6 @@ func TestIgnoreCluster(t *testing.T) {
300310
}
301311

302312
func TestGWApi(t *testing.T) {
303-
externalGatewayAPIManifest := `---
304-
apiVersion: apiextensions.k8s.io/v1
305-
kind: CustomResourceDefinition
306-
metadata:
307-
name: httproutes.gateway.networking.k8s.io
308-
spec:
309-
versions:
310-
- name: v1
311-
`
312-
313313
unsupportedGatewayAPIVersionManifest := `---
314314
apiVersion: apiextensions.k8s.io/v1
315315
kind: CustomResourceDefinition

0 commit comments

Comments
 (0)