From 60e2904e34dccf7e9c81696a646c38051652342a Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Julian=20T=C3=B6lle?= <julian.toelle@hetzner-cloud.de>
Date: Fri, 17 Mar 2023 13:40:04 +0100
Subject: [PATCH] fix: many API requests from Routes controller (#403)

Reduce the number of API requests coming from the routes controller by
reducing the reconciliation interval from 10s to 30s.

This is only a temporary fix until we can properly refactor the routes
controller to only reconcile when necessary.

Related to #395
---
 chart/values.yaml        | 4 ++++
 deploy/ccm-networks.yaml | 1 +
 deploy/ccm.yaml          | 1 +
 3 files changed, 6 insertions(+)

diff --git a/chart/values.yaml b/chart/values.yaml
index 045e0557..46d1c38c 100644
--- a/chart/values.yaml
+++ b/chart/values.yaml
@@ -5,6 +5,10 @@ args:
   leader-elect: "false"
   allow-untagged-cloud: ""
 
+  # Read issue #395 to understand how changes to this value affect you.
+  # https://github.com/hetznercloud/hcloud-cloud-controller-manager/issues/395
+  route-reconciliation-period: 30s
+
 image:
   repository: hetznercloud/hcloud-cloud-controller-manager
   tag: '{{ $.Chart.Version }}'
diff --git a/deploy/ccm-networks.yaml b/deploy/ccm-networks.yaml
index d185bd1f..02d0d27e 100644
--- a/deploy/ccm-networks.yaml
+++ b/deploy/ccm-networks.yaml
@@ -65,6 +65,7 @@ spec:
             - "--allow-untagged-cloud"
             - "--cloud-provider=hcloud"
             - "--leader-elect=false"
+            - "--route-reconciliation-period=30s"
             - "--allocate-node-cidrs=true"
             - "--cluster-cidr=10.244.0.0/16"
           env:
diff --git a/deploy/ccm.yaml b/deploy/ccm.yaml
index f65ce94a..b36a5f7f 100644
--- a/deploy/ccm.yaml
+++ b/deploy/ccm.yaml
@@ -64,6 +64,7 @@ spec:
             - "--allow-untagged-cloud"
             - "--cloud-provider=hcloud"
             - "--leader-elect=false"
+            - "--route-reconciliation-period=30s"
           env:
             - name: NODE_NAME
               valueFrom:
-- 
GitLab