Skip to content

Added missing resources for InitContainer#202

Open
WMP wants to merge 1 commit into
qdrant:mainfrom
WMP:patch-1
Open

Added missing resources for InitContainer#202
WMP wants to merge 1 commit into
qdrant:mainfrom
WMP:patch-1

Conversation

@WMP

@WMP WMP commented Jul 4, 2024

Copy link
Copy Markdown

Maybe you need to make other section for InitContainer Resources.

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

Copy link
Copy Markdown

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

@killmeplz

Copy link
Copy Markdown

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

@killmeplz

Copy link
Copy Markdown

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