Merge pull request #202 from xuzhu-591/fix-exec

fix: loosen rate limit of clientset in informers
horizoncd · Nov 24, 2023 · d9ebf5a · d9ebf5a
2 parents 74694df + 6da226e
commit d9ebf5a
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/pkg/cd/cd.go b/pkg/cd/cd.go
@@ -188,7 +188,7 @@ func (c *cd) DeleteCluster(ctx context.Context, params *DeleteClusterParams) (er
 
 func (c *cd) GetResourceTree(ctx context.Context,
 	params *GetResourceTreeParams) ([]ResourceNode, error) {
-	const op = "cd: get cluster status"
+	const op = "cd: get resource tree"
 	defer wlog.Start(ctx, op).StopPrint()
 
 	argo, err := c.factory.GetArgoCD(params.Environment)

diff --git a/pkg/regioninformers/regioninformers.go b/pkg/regioninformers/regioninformers.go
@@ -20,6 +20,7 @@ import (
 	"sync"
 	"time"
 
+	"github.com/horizoncd/horizon/pkg/util/kube"
 	"k8s.io/client-go/discovery"
 	"k8s.io/client-go/metadata"
 	"k8s.io/client-go/rest"
@@ -121,6 +122,8 @@ func (f *RegionInformers) NewRegionInformers(region *models.Region) error {
 		return err
 	}
 	restConfig = metadata.ConfigFor(restConfig)
+	restConfig.QPS = kube.K8sClientConfigQPS
+	restConfig.Burst = kube.K8sClientConfigBurst
 
 	clientSet, err := kubernetes.NewForConfig(restConfig)
 	if err != nil {