skupperproject / skupper

Skupper is an implementation of a Virtual Application Network, enabling rich hybrid cloud communication.
http://skupper.io
Apache License 2.0
597 stars 75 forks source link

[v2] Controller runs kube api discovery on each site reconcile #1668

Closed c-kruse closed 2 months ago

c-kruse commented 2 months ago

Describe the bug It looks like pkg/kube/site/resources.Apply (used in site reconcile) spends a lot of time doing superfluous stuff - querying the discovery API likely.

I believe it probably comes down to this line here where we are creating a fresh cache for each invocation of this deferred discovery gizmo.

It is my understanding that this is where all of these "Throttling" log lines are coming from: getting rate limited on our discovery api usage - querying apis we have no business with Throttling request took 5.399145702s, request: GET:https://172.30.0.1:443/apis/topolvm.io/v1?timeout=32s

grs commented 2 months ago

@c-kruse thanks for looking into this and highlighting the issue!