Skip to content

Commit

Permalink
Make chains QPS and Burst configuration consistent (#1100)
Browse files Browse the repository at this point in the history
Earlier release of chains multiply default cfg by 2. In pipelines also,
we do this. Same in Results. This makes chains behaviour consistent with
all other controllers.
  • Loading branch information
khrm authored May 10, 2024
1 parent b8fb5f1 commit fd18dd5
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions cmd/controller/main.go
Original file line number Diff line number Diff line change
Expand Up @@ -53,6 +53,10 @@ func main() {
cfg.Burst = rest.DefaultBurst
}

// Multiply by number of controllers
cfg.QPS = 2 * cfg.QPS
cfg.Burst = 2 * cfg.Burst

flag.Parse()
ctx := injection.WithNamespaceScope(signals.NewContext(), *namespace)

Expand Down

0 comments on commit fd18dd5

Please sign in to comment.