Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

cu-86c0423hd Allow editing PriorityClass value in helm value #330

Merged
merged 1 commit into from
Aug 28, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions charts/komodor-agent/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -141,6 +141,7 @@ The command removes all the Kubernetes components associated with the chart and
| capabilities.telemetry.collectApiServerMetrics | bool | `false` | Collect metrics from the api server (Should only be used for debugging purposes) |
| components | object | See sub-values | Configure the agent components |
| components.komodorAgent | object | See sub-values | Configure the komodor agent components |
| components.komodorAgent.PriorityClassValue | int | `10000000` | Set the priority class value for the komodor agent deployment |
| components.komodorAgent.affinity | object | `{}` | Set node affinity for the komodor agent deployment |
| components.komodorAgent.annotations | object | `{}` | Set annotations for the komodor agent deployment |
| components.komodorAgent.labels | object | `{}` | Set custom labels |
Expand All @@ -159,6 +160,7 @@ The command removes all the Kubernetes components associated with the chart and
| components.komodorAgent.supervisor.extraEnvVars | list | `[]` | List of additional environment variables, Each entry is a key-value pair |
| components.komodorAgent.networkMapper.image | object | `{"name":"network-mapper","tag":"v1.0.3"}` | Override the komodor agent network mapper image name or tag. |
| components.komodorAgent.networkMapper.resources | object | `{}` | Set custom resources to the komodor agent network mapper container |
| components.komodorMetrics.PriorityClassValue | int | `10000000` | Set the priority class value for the komodor metrics agent deployment |
| components.komodorMetrics.affinity | object | `{}` | Set node affinity for the komodor metrics agent deployment |
| components.komodorMetrics.annotations | object | `{}` | Set annotations for the komodor metrics agent deployment |
| components.komodorMetrics.labels | object | `{}` | Set custom labels |
Expand All @@ -173,6 +175,7 @@ The command removes all the Kubernetes components associated with the chart and
| components.komodorMetrics.metrics.resources | object | `{"limits":{"cpu":1,"memory":"1Gi"},"requests":{"cpu":0.1,"memory":"384Mi"}}` | Set custom resources to the komodor agent metrics container |
| components.komodorMetrics.metrics.extraEnvVars | list | `[]` | List of additional environment variables, Each entry is a key-value pair |
| components.komodorDaemon | object | See sub-values | Configure the komodor agent components |
| components.komodorDaemon.PriorityClassValue | int | `10000000` | Set the priority class value for the komodor daemon deployment |
| components.komodorDaemon.affinity | object | `{}` | Set node affinity for the komodor agent daemon |
| components.komodorDaemon.annotations | object | `{}` | Adds custom annotations - Example: `--set annotations."app\.komodor\.com/app"="komodor-agent"` |
| components.komodorDaemon.labels | object | `{}` | Adds custom labels |
Expand Down
2 changes: 1 addition & 1 deletion charts/komodor-agent/templates/deployment_metrics.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ spec:
{{- include "komodorMetrics.selectorLabels" . | nindent 8 }}
{{- include "KomodorMetrics.user.labels" . | nindent 8 }}
spec:
priorityClassName: {{ .Release.Name }}-agent-high-priority
priorityClassName: {{ .Release.Name }}-metrics-high-priority
serviceAccountName: {{ include "komodorAgent.serviceAccountName" . }}
affinity:
{{- toYaml .Values.components.komodorMetrics.affinity | nindent 8 }}
Expand Down
14 changes: 11 additions & 3 deletions charts/komodor-agent/templates/priorityclasses.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ apiVersion: scheduling.k8s.io/v1
kind: PriorityClass
metadata:
name: {{ .Release.Name }}-daemon-high-priority
value: 10000000
value: {{ .Values.components.komodorDaemon.PriorityClassValue }}
globalDefault: false
description: "This priority class should be used for Komodor daemonset pods only."

Expand All @@ -12,6 +12,14 @@ apiVersion: scheduling.k8s.io/v1
kind: PriorityClass
metadata:
name: {{ .Release.Name }}-agent-high-priority
value: 10000000
value: {{ .Values.components.komodorAgent.PriorityClassValue }}
globalDefault: false
description: "This priority class should be used for Komodor agent pods only."
description: "This priority class should be used for Komodor agent pods only."
---
apiVersion: scheduling.k8s.io/v1
kind: PriorityClass
metadata:
name: {{ .Release.Name }}-metrics-high-priority
value: {{ .Values.components.komodorMetrics.PriorityClassValue }}
globalDefault: false
description: "This priority class should be used for Komodor pods only."
6 changes: 6 additions & 0 deletions charts/komodor-agent/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -119,6 +119,8 @@ components:
# components.komodorAgent -- Configure the komodor agent components
# @default -- See sub-values
komodorAgent:
# components.komodorAgent.PriorityClassValue -- Set the priority class value for the komodor agent deployment
PriorityClassValue: 10000000
# components.komodorAgent.affinity -- Set node affinity for the komodor agent deployment
affinity: {}
# components.komodorAgent.annotations -- Set annotations for the komodor agent deployment
Expand Down Expand Up @@ -182,6 +184,8 @@ components:
resources: {}

komodorMetrics:
# components.komodorMetrics.PriorityClassValue -- Set the priority class value for the komodor metrics agent deployment
PriorityClassValue: 10000000
# components.komodorMetrics.affinity -- Set node affinity for the komodor metrics agent deployment
affinity: { }
# components.komodorMetrics.annotations -- Set annotations for the komodor metrics agent deployment
Expand Down Expand Up @@ -227,6 +231,8 @@ components:
# components.komodorDaemon -- Configure the komodor agent components
# @default -- See sub-values
komodorDaemon:
# components.komodorDaemon.PriorityClassValue -- Set the priority class value for the komodor daemon deployment
PriorityClassValue: 10000000
# components.komodorDaemon.affinity -- Set node affinity for the komodor agent daemon
affinity: {}
# components.komodorDaemon.annotations -- Adds custom annotations - Example: `--set annotations."app\.komodor\.com/app"="komodor-agent"`
Expand Down