qdrant-helm icon indicating copy to clipboard operation
qdrant-helm copied to clipboard

Added missing resources for InitContainer

Open WMP opened this issue 1 year ago • 3 comments

Maybe you need to make other section for InitContainer Resources.

WMP avatar Jul 04 '24 08:07 WMP

+1 When you have resourcequota on a namespace this is obligatory.

killmeplz avatar Mar 01 '25 16:03 killmeplz

Maybe it's better to create additional values: Like .Values.initContainers.resources: {}

killmeplz avatar Mar 01 '25 16:03 killmeplz

If someone is using kyverno there is a workaround

apiVersion: kyverno.io/v1
kind: Policy
metadata:
  name: require-container-resources
spec:
  validationFailureAction: Audit
  background: true
  rules:
  - name: validate-resources
    match:
      resources:
        kinds:
        - Deployment
        - StatefulSet
    mutate:
      foreach:
      - list: "request.object.spec.template.spec.containers"
        patchStrategicMerge:
          spec:
            template:
              spec:
                containers:
                - name: "{{ element.name }}"
                  resources:
                    limits:
                      cpu: "{{ element.resources.limits.cpu || '500m' }}"
                      memory: "{{ element.resources.limits.memory || '512Mi' }}"
                    requests:
                      cpu: "{{ element.resources.requests.cpu || '250m' }}"
                      memory: "{{ element.resources.requests.memory || '256Mi' }}"
      - list: "request.object.spec.template.spec.initContainers || []"
        patchStrategicMerge:
          spec:
            template:
              spec:
                initContainers:
                - name: "{{ element.name }}"
                  resources:
                    limits:
                      cpu: "{{ element.resources.limits.cpu || '500m' }}"
                      memory: "{{ element.resources.limits.memory || '512Mi' }}"
                    requests:
                      cpu: "{{ element.resources.requests.cpu || '250m' }}"
                      memory: "{{ element.resources.requests.memory || '256Mi' }}"

killmeplz avatar Mar 01 '25 18:03 killmeplz