Skip to content

feat: add environment config for monitoring agent container#6

Draft
ndbaker1 wants to merge 1 commit intomainfrom
env
Draft

feat: add environment config for monitoring agent container#6
ndbaker1 wants to merge 1 commit intomainfrom
env

Conversation

@ndbaker1
Copy link
Contributor

@ndbaker1 ndbaker1 commented May 13, 2025

Issue #, if available:

Description of changes:

Add environment variable options to monitoring agent helm chart values.

Testing Done:

helm template eks-node-monitoring-agent ./charts/eks-node-monitoring-agent --namespace kube-system --debug --set nodeAgent.env.foo=bar

...
      serviceAccountName: eks-node-monitoring-agent
      hostNetwork: true
      hostPID: true
      containers:
        - name: eks-node-monitoring-agent
          image: 602401143452.dkr.ecr.us-west-2.amazonaws.com/eks/eks-node-monitoring-agent:v1.2.0-eksbuild.1
          imagePullPolicy: IfNotPresent
          args:
            - --probe-address=:8002
            - --metrics-address=:8003
          env:
            - name: HOST_ROOT
              value: /host
            - name: MY_NODE_NAME
              valueFrom:
                fieldRef:
                  fieldPath: spec.nodeName
            - name: foo
              value: "bar"
          livenessProbe:
            httpGet:
              path: /healthz
              port: 8002
          resources:
            limits:
              cpu: 250m
              memory: 100Mi
            requests:
              cpu: 10m
              memory: 30Mi
          securityContext:
...

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

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.

1 participant