Skip to content

Commit

Permalink
Merge pull request #24 from brancz/remove-limits
Browse files Browse the repository at this point in the history
*: Remove limits
  • Loading branch information
openshift-merge-robot committed Apr 17, 2019
2 parents 9a3ca6e + 4c38a8f commit 73ed419
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 21 deletions.
6 changes: 0 additions & 6 deletions pkg/alertmanager/statefulset.go
Expand Up @@ -460,12 +460,6 @@ func makeStatefulSetSpec(a *monitoringv1.Alertmanager, config Config) (*appsv1.S
MountPath: alertmanagerConfDir,
},
},
Resources: v1.ResourceRequirements{
Limits: v1.ResourceList{
v1.ResourceCPU: resource.MustParse("50m"),
v1.ResourceMemory: resource.MustParse("10Mi"),
},
},
},
}, a.Spec.Containers...),
Volumes: volumes,
Expand Down
18 changes: 3 additions & 15 deletions pkg/prometheus/statefulset.go
Expand Up @@ -594,12 +594,6 @@ func makeStatefulSetSpec(p monitoringv1.Prometheus, c *Config, ruleConfigMapName
fmt.Sprintf("--webhook-url=%s", localReloadURL),
},
VolumeMounts: []v1.VolumeMount{},
Resources: v1.ResourceRequirements{
Limits: v1.ResourceList{
v1.ResourceCPU: resource.MustParse("25m"),
v1.ResourceMemory: resource.MustParse("10Mi"),
},
},
}

for _, name := range ruleConfigMapNames {
Expand Down Expand Up @@ -816,22 +810,16 @@ func makeStatefulSetSpec(p monitoringv1.Prometheus, c *Config, ruleConfigMapName
Command: []string{"/bin/prometheus-config-reloader"},
Args: configReloadArgs,
VolumeMounts: configReloadVolumeMounts,
Resources: v1.ResourceRequirements{
Limits: v1.ResourceList{
v1.ResourceCPU: resource.MustParse("50m"),
v1.ResourceMemory: resource.MustParse("50Mi"),
},
},
},
}, additionalContainers...),
SecurityContext: securityContext,
ServiceAccountName: p.Spec.ServiceAccountName,
NodeSelector: p.Spec.NodeSelector,
PriorityClassName: p.Spec.PriorityClassName,
TerminationGracePeriodSeconds: &terminationGracePeriod,
Volumes: volumes,
Tolerations: p.Spec.Tolerations,
Affinity: p.Spec.Affinity,
Volumes: volumes,
Tolerations: p.Spec.Tolerations,
Affinity: p.Spec.Affinity,
},
},
}, nil
Expand Down

0 comments on commit 73ed419

Please sign in to comment.