Skip to content

Commit

Permalink
sqs: Increase CPU limit to avoid CFS throttling
Browse files Browse the repository at this point in the history
  • Loading branch information
antoineco committed Dec 4, 2020
1 parent 709353f commit c8f42c4
Showing 1 changed file with 5 additions and 3 deletions.
8 changes: 5 additions & 3 deletions pkg/reconciler/awssqssource/adapter.go
Original file line number Diff line number Diff line change
Expand Up @@ -75,13 +75,15 @@ func adapterDeploymentBuilder(src *v1alpha1.AWSSQSSource, cfg *adapterConfig) co
resource.EnvVars(common.MakeSecurityCredentialsEnvVars(src.Spec.Credentials)...),
resource.EnvVars(cfg.configs.ToEnvVars()...),

// CPU throttling can be observed below a limit of 1,
// although the CPU usage under load remains below 400m.
resource.Requests(
*kr.NewMilliQuantity(90, kr.DecimalSI), // 90m
*kr.NewQuantity(1024*1024*18, kr.BinarySI), // 18Mi
*kr.NewQuantity(1024*1024*30, kr.BinarySI), // 30Mi
),
resource.Limits(
*kr.NewMilliQuantity(120, kr.DecimalSI), // 120m
*kr.NewQuantity(1024*1024*30, kr.BinarySI), // 30Mi
*kr.NewMilliQuantity(1000, kr.DecimalSI), // 1
*kr.NewQuantity(1024*1024*45, kr.BinarySI), // 45Mi
),
)
}
Expand Down

0 comments on commit c8f42c4

Please sign in to comment.