From 648e2f1179f66e9bfbe471c261af843369a9a352 Mon Sep 17 00:00:00 2001 From: Rahul Sharma Date: Fri, 8 Nov 2024 04:43:28 +0000 Subject: [PATCH] address review comments --- cloud/linode/cloud.go | 2 +- deploy/chart/templates/daemonset.yaml | 6 ++++++ 2 files changed, 7 insertions(+), 1 deletion(-) diff --git a/cloud/linode/cloud.go b/cloud/linode/cloud.go index 8d2b6cf2..f6ec2f7c 100644 --- a/cloud/linode/cloud.go +++ b/cloud/linode/cloud.go @@ -35,7 +35,7 @@ var Options struct { KubeconfigFlag *pflag.Flag LinodeGoDebug bool EnableRouteController bool - // deprecated: use VPCNames instead + // Deprecated: use VPCNames instead VPCName string VPCNames string LoadBalancerType string diff --git a/deploy/chart/templates/daemonset.yaml b/deploy/chart/templates/daemonset.yaml index fcfff9d9..0a46c517 100644 --- a/deploy/chart/templates/daemonset.yaml +++ b/deploy/chart/templates/daemonset.yaml @@ -38,6 +38,12 @@ spec: {{- end }} {{- if .Values.routeController }} - --enable-route-controller=true + {{- if and .Values.routeController.vpcName .Values.routeController.vpcNames }} + {{- fail "Both vpcName and vpcNames are set. Please use only vpcNames." }} + {{- end }} + {{- if not (or .Values.routeController.vpcName .Values.routeController.vpcNames) }} + {{- fail "Neither vpcName nor vpcNames is set. Please set one of them." }} + {{- end }} {{- if .Values.routeController.vpcName }} - --vpc-name={{ .Values.routeController.vpcName }} {{- end }}