Skip to content

Commit

Permalink
fix: increase default memoryRequest for helm-sync (#895)
Browse files Browse the repository at this point in the history
The helm-sync container is failing regularly on autopilot due to
OOMKill. Autopilot sets the limits equal to the requests, and the
current value is too low for all of the e2e tests to pass. Standard
allows bursting while autopilot does not. We may be able to revert this
increase after an upcoming change which will allow separate defaults for
standard and autopilot.
  • Loading branch information
sdowell authored Sep 22, 2023
1 parent 108eaa9 commit 199db6d
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,7 @@ func ReconcilerContainerResourceDefaults() map[string]v1beta1.ContainerResources
reconcilermanager.HelmSync: {
ContainerName: reconcilermanager.HelmSync,
CPURequest: resource.MustParse("50m"),
MemoryRequest: resource.MustParse("200Mi"),
MemoryRequest: resource.MustParse("256Mi"),
},
reconcilermanager.GitSync: {
ContainerName: reconcilermanager.GitSync,
Expand Down

0 comments on commit 199db6d

Please sign in to comment.