Skip to content

Commit

Permalink
Increased default reconciliation timeout to 3 minutes. (gardener#8085)
Browse files Browse the repository at this point in the history
The default reconciliation timeout is used as a safety net to prevent reconciliation
operations to hang forever. Therefore, it should be long enough that all usual
operations succeed, but short enough so that the waiting time is not endless in case
of issues/hanging operations.
With larger amount of resources being applied for example as managed resources the
default reconciliation timeout needs to be increased. An example for such a resource
is the istio ingress gateway in case the seed uses multiple availability zones as
the managed resource will then include all entities for all availability zones.
Another contributing factor may be if the control plane of a seed is latency-wise
fairly distant to its data plane causing operations such as applying resources to
be rather slow.
  • Loading branch information
ScheererJ authored Jun 16, 2023
1 parent 626a814 commit fe3dcb6
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion pkg/controllerutils/miscellaneous.go
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ import (
)

// DefaultReconciliationTimeout is the default timeout for the context of reconciliation functions.
const DefaultReconciliationTimeout = 1 * time.Minute
const DefaultReconciliationTimeout = 3 * time.Minute

const separator = ","

Expand Down

0 comments on commit fe3dcb6

Please sign in to comment.