Skip to content

Commit dbf780a

Browse files
committed
authn-trust: don't react to target CM updates
If kube-apiservers run at a different version (during upgrades) and each has different opinion on what the `extension-apiserver-authentication` config map should look like, they would start dueling with config map writes. This commit removes handling of the update events of the target CM in order to reduce the amount of such dueling to once a minute, as the controller is guaranteed to run at least once a minute anyway. The idea is that a cluster state with two different versions of kube-apiserver should never be permanent, and so a reduced amount of dueling for that period is tolerable.
1 parent 9d9e1af commit dbf780a

File tree

1 file changed

+0
-3
lines changed

1 file changed

+0
-3
lines changed

pkg/controlplane/controller/clusterauthenticationtrust/cluster_authentication_trust_controller.go

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -128,9 +128,6 @@ func NewClusterAuthenticationTrustController(requiredAuthenticationData ClusterA
128128
AddFunc: func(obj interface{}) {
129129
c.queue.Add(keyFn())
130130
},
131-
UpdateFunc: func(oldObj, newObj interface{}) {
132-
c.queue.Add(keyFn())
133-
},
134131
DeleteFunc: func(obj interface{}) {
135132
c.queue.Add(keyFn())
136133
},

0 commit comments

Comments
 (0)