vmalertmanager: operator always requests memory

[uhthomas]

The operator will always apply a memory request of 200Mi to the Alertmanager pod, even if limits are already set. This is problematic if the limit is lower than the request, or if the user does not want an explicit request. I usually only set limits, as the Kubernetes API will automatically set the requests to match those limits and the QOS class will then be considered "Guaranteed". However, the QOS class is considered "Burstable" because of the default memory request. This should just be removed.

operator/controllers/factory/alertmanager.go

Lines 124 to 126 in aff55e6

	if _, ok := cr.Spec.Resources.Requests[v1.ResourceMemory]; !ok {
	cr.Spec.Resources.Requests[v1.ResourceMemory] = resource.MustParse("200Mi")
	}

[Amper]

Added at v0.36.0 release