Skip to content
Snippets Groups Projects
Unverified Commit 8e5bf00c authored by Matthias Loibl's avatar Matthias Loibl Committed by GitHub
Browse files

Merge pull request #984 from paulfantom/am_resources

jsonnet/alertmanager: add default alertmanager resource requirements
parents 5f13d381 d7531691
No related branches found
No related tags found
No related merge requests found
...@@ -3,7 +3,10 @@ local defaults = { ...@@ -3,7 +3,10 @@ local defaults = {
namespace: error 'must provide namespace', namespace: error 'must provide namespace',
image: error 'must provide image', image: error 'must provide image',
version: error 'must provide version', version: error 'must provide version',
resources: {}, resources: {
limits: { cpu: '100m', memory: '100Mi' },
requests: { cpu: '4m', memory: '100Mi' },
},
commonLabels:: { commonLabels:: {
'app.kubernetes.io/name': 'alertmanager', 'app.kubernetes.io/name': 'alertmanager',
'app.kubernetes.io/version': defaults.version, 'app.kubernetes.io/version': defaults.version,
......
...@@ -20,7 +20,13 @@ spec: ...@@ -20,7 +20,13 @@ spec:
app.kubernetes.io/part-of: kube-prometheus app.kubernetes.io/part-of: kube-prometheus
app.kubernetes.io/version: 0.21.0 app.kubernetes.io/version: 0.21.0
replicas: 3 replicas: 3
resources: {} resources:
limits:
cpu: 100m
memory: 100Mi
requests:
cpu: 4m
memory: 100Mi
securityContext: securityContext:
fsGroup: 2000 fsGroup: 2000
runAsNonRoot: true runAsNonRoot: true
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment