From 84b666d42b357e6504e490423ffde311090f1555 Mon Sep 17 00:00:00 2001 From: Rahul Sharma Date: Wed, 6 Nov 2024 15:27:15 +0000 Subject: [PATCH] fix readme and helm chart --- README.md | 2 +- deploy/chart/templates/daemonset.yaml | 4 ++-- main.go | 2 +- 3 files changed, 4 insertions(+), 4 deletions(-) diff --git a/README.md b/README.md index 957de726..93aefe00 100644 --- a/README.md +++ b/README.md @@ -176,7 +176,7 @@ When running k8s clusters within VPC, node specific podCIDRs need to be allowed ##### Example usage in values.yaml ```yaml routeController: - vpcName: + vpcNames: clusterCIDR: 10.0.0.0/8 configureCloudRoutes: true ``` diff --git a/deploy/chart/templates/daemonset.yaml b/deploy/chart/templates/daemonset.yaml index fe812f0a..fcfff9d9 100644 --- a/deploy/chart/templates/daemonset.yaml +++ b/deploy/chart/templates/daemonset.yaml @@ -39,10 +39,10 @@ spec: {{- if .Values.routeController }} - --enable-route-controller=true {{- if .Values.routeController.vpcName }} - - --vpc-name={{ "A valid .Values.routeController.vpcName is required" .Values.routeController.vpcName }} + - --vpc-name={{ .Values.routeController.vpcName }} {{- end }} {{- if .Values.routeController.vpcNames }} - - --vpc-names={{ "A valid .Values.routeController.vpcNames is required" .Values.routeController.vpcNames }} + - --vpc-names={{ .Values.routeController.vpcNames }} {{- end }} - --configure-cloud-routes={{ default true .Values.routeController.configureCloudRoutes }} - --cluster-cidr={{ required "A valid .Values.routeController.clusterCIDR is required" .Values.routeController.clusterCIDR }} diff --git a/main.go b/main.go index 455c9475..2ef42d02 100644 --- a/main.go +++ b/main.go @@ -81,7 +81,7 @@ func main() { // Add Linode-specific flags command.Flags().BoolVar(&linode.Options.LinodeGoDebug, "linodego-debug", false, "enables debug output for the LinodeAPI wrapper") command.Flags().BoolVar(&linode.Options.EnableRouteController, "enable-route-controller", false, "enables route_controller for ccm") - command.Flags().StringVar(&linode.Options.VPCName, "vpc-name", "", "[deprecated] vpc name whose routes will be managed by route-controller") + command.Flags().StringVar(&linode.Options.VPCName, "vpc-name", "", "[deprecated: use vpc-names instead] vpc name whose routes will be managed by route-controller") command.Flags().StringVar(&linode.Options.VPCNames, "vpc-names", "", "comma separated vpc names whose routes will be managed by route-controller") command.Flags().StringVar(&linode.Options.LoadBalancerType, "load-balancer-type", "nodebalancer", "configures which type of load-balancing to use for LoadBalancer Services (options: nodebalancer, cilium-bgp)") command.Flags().StringVar(&linode.Options.BGPNodeSelector, "bgp-node-selector", "", "node selector to use to perform shared IP fail-over with BGP (e.g. cilium-bgp-peering=true")