Skip to content

Commit

Permalink
Bump up Consul server statefulset memory defaults too
Browse files Browse the repository at this point in the history
  • Loading branch information
Andrew Stucki committed Jun 2, 2023
1 parent b2c31ba commit c6b2adf
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 5 deletions.
2 changes: 1 addition & 1 deletion charts/consul/test/unit/server-statefulset.bats
Original file line number Diff line number Diff line change
Expand Up @@ -99,7 +99,7 @@ load _helpers
-s templates/server-statefulset.yaml \
. | tee /dev/stderr |
yq -rc '.spec.template.spec.containers[0].resources' | tee /dev/stderr)
[ "${actual}" = '{"limits":{"cpu":"100m","memory":"100Mi"},"requests":{"cpu":"100m","memory":"100Mi"}}' ]
[ "${actual}" = '{"limits":{"cpu":"100m","memory":"200Mi"},"requests":{"cpu":"100m","memory":"200Mi"}}' ]
}

@test "server/StatefulSet: resources can be overridden" {
Expand Down
8 changes: 4 additions & 4 deletions charts/consul/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -808,21 +808,21 @@ server:
# ```yaml
# resources:
# requests:
# memory: '100Mi'
# memory: '200Mi'
# cpu: '100m'
# limits:
# memory: '100Mi'
# memory: '200Mi'
# cpu: '100m'
# ```
#
# @recurse: false
# @type: map
resources:
requests:
memory: "100Mi"
memory: "200Mi"
cpu: "100m"
limits:
memory: "100Mi"
memory: "200Mi"
cpu: "100m"

# The security context for the server pods. This should be a YAML map corresponding to a
Expand Down

0 comments on commit c6b2adf

Please sign in to comment.