Skip to content

Conversation

WMP
Copy link

@WMP WMP commented Jul 4, 2024

Maybe you need to make other section for InitContainer Resources.

Maybe you need to make other section for InitContainer Resources.
@killmeplz
Copy link

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

@killmeplz
Copy link

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

@killmeplz
Copy link

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' }}"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants