Skip to content

Commit

Permalink
feat: support setting priority class name for datahub pods (#392)
Browse files Browse the repository at this point in the history
* feat: support for priorityClassName in datahub-actions
* feat:support for priorityClassName in datahub-frontend
* feat:support for priorityClassName in datahub-gms
* feat:support for priorityClassName in datahub-mae-consumer
* feat:support for priorityClassName in datahub-mce-consumer

---------

Co-authored-by: david-leifker <[email protected]>
  • Loading branch information
sachinsaju and david-leifker authored Nov 29, 2023
1 parent b3f33cf commit 6598b1d
Show file tree
Hide file tree
Showing 12 changed files with 28 additions and 4 deletions.
5 changes: 2 additions & 3 deletions charts/datahub/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,8 +3,7 @@ name: datahub
description: A Helm chart for LinkedIn DataHub
type: application
# This is the chart version. This version number should be incremented each time you make changes
# to the chart and its templates, including the app version.
version: 0.3.12
version: 0.3.13
# This is the version number of the application being deployed. This version number should be
# incremented each time you make changes to the application.
appVersion: 0.11.0
Expand All @@ -14,7 +13,7 @@ dependencies:
repository: file://./subcharts/datahub-gms
condition: datahub-gms.enabled
- name: datahub-frontend
version: 0.2.146
version: 0.2.148
repository: file://./subcharts/datahub-frontend
condition: datahub-frontend.enabled
- name: datahub-mae-consumer
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -50,6 +50,9 @@ spec:
{{- if .Values.extraVolumes }}
{{ toYaml .Values.extraVolumes | nindent 8 }}
{{- end }}
{{- if .Values.priorityClassName }}
priorityClassName: "{{ .Values.priorityClassName }}"
{{- end }}
initContainers:
{{- if .Values.extraInitContainers }}
{{- .Values.extraInitContainers | toYaml | nindent 6 }}
Expand Down
2 changes: 2 additions & 0 deletions charts/datahub/subcharts/acryl-datahub-actions/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -30,6 +30,8 @@ podSecurityContext: {}

securityContext: {}

priorityClassName:

service:
type: ClusterIP
port: 9093
Expand Down
2 changes: 1 addition & 1 deletion charts/datahub/subcharts/datahub-frontend/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ description: A Helm chart for Kubernetes
type: application
# This is the chart version. This version number should be incremented each time you make changes
# to the chart and its templates, including the app version.
version: 0.2.147
version: 0.2.148
# This is the version number of the application being deployed. This version number should be
# incremented each time you make changes to the application.
appVersion: v0.11.0
Original file line number Diff line number Diff line change
Expand Up @@ -54,6 +54,9 @@ spec:
{{- with .Values.extraVolumes }}
{{- toYaml . | nindent 8 }}
{{- end }}
{{- if .Values.priorityClassName }}
priorityClassName: "{{ .Values.priorityClassName }}"
{{- end }}
initContainers:
{{- with .Values.extraInitContainers }}
{{- toYaml . | nindent 8 }}
Expand Down
2 changes: 2 additions & 0 deletions charts/datahub/subcharts/datahub-frontend/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -38,6 +38,8 @@ securityContext: {}
# runAsNonRoot: true
# runAsUser: 1000

priorityClassName:

service:
type: LoadBalancer # ClusterIP or NodePort
port: 9002
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -58,6 +58,9 @@ spec:
{{- with .Values.extraVolumes }}
{{- toYaml . | nindent 8 }}
{{- end }}
{{- if .Values.priorityClassName }}
priorityClassName: "{{ .Values.priorityClassName }}"
{{- end }}
initContainers:
{{- with .Values.extraInitContainers }}
{{- toYaml . | nindent 8 }}
Expand Down
2 changes: 2 additions & 0 deletions charts/datahub/subcharts/datahub-gms/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -42,6 +42,8 @@ securityContext: {}
# runAsNonRoot: true
# runAsUser: 1000

priorityClassName:

service:
type: LoadBalancer # ClusterIP or NodePort
port: "8080"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -49,6 +49,9 @@ spec:
{{- with .Values.extraVolumes }}
{{- toYaml . | nindent 8 }}
{{- end }}
{{- if .Values.priorityClassName }}
priorityClassName: "{{ .Values.priorityClassName }}"
{{- end }}
initContainers:
{{- with .Values.extraInitContainers }}
{{- toYaml . | nindent 8 }}
Expand Down
2 changes: 2 additions & 0 deletions charts/datahub/subcharts/datahub-mae-consumer/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -41,6 +41,8 @@ securityContext: {}
# runAsNonRoot: true
# runAsUser: 1000

priorityClassName:

service:
type: ClusterIP # NodePort
port: "9091"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -53,6 +53,9 @@ spec:
{{- with .Values.extraVolumes }}
{{- toYaml . | nindent 8 }}
{{- end }}
{{- if .Values.priorityClassName }}
priorityClassName: "{{ .Values.priorityClassName }}"
{{- end }}
initContainers:
{{- with .Values.extraInitContainers }}
{{- toYaml . | nindent 8 }}
Expand Down
2 changes: 2 additions & 0 deletions charts/datahub/subcharts/datahub-mce-consumer/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,8 @@ securityContext: {}
# runAsNonRoot: true
# runAsUser: 1000

priorityClassName:

service:
type: ClusterIP # NodePort
port: "9090"
Expand Down

0 comments on commit 6598b1d

Please sign in to comment.