update QPS default value (#761)

Azure · Apr 17, 2024 · db2e653 · db2e653
1 parent 913d128
commit db2e653
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/charts/hub-agent/README.md b/charts/hub-agent/README.md
@@ -34,7 +34,7 @@ _See [helm install](https://helm.sh/docs/helm/helm_install/) for command documen
 | logVerbosity                  | Log level. Uses V logs (klog)                                                                                                                                | `5`                                              |
 | enableV1Alpha1APIs            | If set, the agents will watch for the v1alpha1 APIs.                                                                                                         | `false`                                          |
 | enableV1Beta1APIs             | If set, the agents will watch for the v1beta1 APIs.                                                                                                          | `true`                                           |
-| hubAPIQPS                     | QPS to use while talking with fleet-apiserver. Doesn't cover events and node heartbeat apis which rate limiting is controlled by a different set of flags.   | `1000`                                           |
+| hubAPIQPS                     | QPS to use while talking with fleet-apiserver. Doesn't cover events and node heartbeat apis which rate limiting is controlled by a different set of flags.   | `250`                                            |
 | hubAPIBurst                   | Burst to use while talking with fleet-apiserver. Doesn't cover events and node heartbeat apis which rate limiting is controlled by a different set of flags. | `1000`                                           |
 | MaxConcurrentClusterPlacement | The max number of clusterResourcePlacement to run concurrently this fleet supports.                                                                          | `100`                                            |
 | ConcurrentResourceChangeSyncs | The number of resourceChange reconcilers that are allowed to run concurrently.                                                                               | `20`                                             |

diff --git a/charts/hub-agent/values.yaml b/charts/hub-agent/values.yaml
@@ -35,7 +35,7 @@ affinity: {}
 enableV1Alpha1APIs: false
 enableV1Beta1APIs: true
 
-hubAPIQPS: 1000
+hubAPIQPS: 250
 hubAPIBurst: 1000
 MaxConcurrentClusterPlacement: 100
 ConcurrentResourceChangeSyncs: 20

diff --git a/cmd/hubagent/options/options.go b/cmd/hubagent/options/options.go
@@ -125,7 +125,7 @@ func (o *Options) AddFlags(flags *flag.FlagSet) {
 		"<group>/<version>/<kind>,<kind> for skip one or more specific resource(e.g. networking.k8s.io/v1beta1/Ingress,IngressClass) where the kinds are case-insensitive.")
 	flags.StringVar(&o.SkippedPropagatingNamespaces, "skipped-propagating-namespaces", "",
 		"Comma-separated namespaces that should be skipped from propagating in addition to the default skipped namespaces(fleet-system, namespaces prefixed by kube- and fleet-work-).")
-	flags.Float64Var(&o.HubQPS, "hub-api-qps", 1000, "QPS to use while talking with fleet-apiserver. Doesn't cover events and node heartbeat apis which rate limiting is controlled by a different set of flags.")
+	flags.Float64Var(&o.HubQPS, "hub-api-qps", 250, "QPS to use while talking with fleet-apiserver. Doesn't cover events and node heartbeat apis which rate limiting is controlled by a different set of flags.")
 	flags.IntVar(&o.HubBurst, "hub-api-burst", 1000, "Burst to use while talking with fleet-apiserver. Doesn't cover events and node heartbeat apis which rate limiting is controlled by a different set of flags.")
 	flags.DurationVar(&o.ResyncPeriod.Duration, "resync-period", 300*time.Second, "Base frequency the informers are resynced.")
 	flags.IntVar(&o.MaxConcurrentClusterPlacement, "max-concurrent-cluster-placement", 100, "The max number of concurrent cluster placement to run concurrently.")