Skip to content

Commit

Permalink
Update openmetadata helm chart to address issue #209 and #210
Browse files Browse the repository at this point in the history
  • Loading branch information
preetsshah committed Jan 9, 2024
1 parent 0f85170 commit 13205b2
Show file tree
Hide file tree
Showing 5 changed files with 31 additions and 6 deletions.
6 changes: 3 additions & 3 deletions charts/openmetadata/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -16,13 +16,13 @@ type: application
# to the chart and its templates, including the app version.
# Versions are expected to follow Semantic Versioning (https://semver.org/)

version: 1.2.6
version: 1.2.7

# This is the version number of the application being deployed. This version number should be
# incremented each time you make changes to the application. Versions are not expected to
# follow Semantic Versioning. They should reflect the version the application is using.
# It is recommended to use it with quotes.
appVersion: "1.2.4"
appVersion: "1.2.5"

home: https://open-metadata.org/

Expand Down Expand Up @@ -55,4 +55,4 @@ maintainers:
- name: OpenMetadata
email: support@open-metadata.org

icon: https://open-metadata.org/images/favicon.png
icon: https://open-metadata.org/images/favicon.png

Check failure on line 58 in charts/openmetadata/Chart.yaml

View workflow job for this annotation

GitHub Actions / lint-test

[new-line-at-end-of-file] no new line character at the end of file
7 changes: 6 additions & 1 deletion charts/openmetadata/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -240,6 +240,10 @@ helm install openmetadata open-metadata/openmetadata --values <<path-to-values-f
| readinessProbe.httpGet.port | string | `http` |
| replicaCount | int | `1` |
| resources | object | `{}` |
| initContainerResources.limits.cpu | string | `1` |
| initContainerResources.limits.memory | string | `2048Mi` |
| initContainerResources.requests.cpu | string | `500m` |
| initContainerResources.requests.memory | string | `256Mi` |
| securityContext | object | `{}` |
| service.adminPort | string | `8586` |
| service.annotations | object | `{}` |
Expand All @@ -248,6 +252,7 @@ helm install openmetadata open-metadata/openmetadata --values <<path-to-values-f
| serviceAccount.annotations | object | `{}` |
| serviceAccount.create | bool | `true` |
| serviceAccount.name | string | `nil` |
| automountServiceAccountToken| bool | `true` |
| serviceMonitor.annotations | object | `{}` |
| serviceMonitor.enabled | bool | `false` |
| serviceMonitor.interval | string | `30s` |
Expand All @@ -262,4 +267,4 @@ helm install openmetadata open-metadata/openmetadata --values <<path-to-values-f
| networkPolicy.enabled | bool |`false` |
| podDisruptionBudget.enabled | bool | `false` |
| podDisruptionBudget.config.maxUnavailable | String | `1` |
| podDisruptionBudget.config.minAvailable | String | `1` |
| podDisruptionBudget.config.minAvailable | String | `1` |
5 changes: 5 additions & 0 deletions charts/openmetadata/templates/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,9 @@ spec:
{{- toYaml . | nindent 8 }}
{{- end }}
serviceAccountName: {{ include "OpenMetadata.serviceAccountName" . }}
{{- if not (.Values.automountServiceAccountToken) }}
automountServiceAccountToken: {{ .Values.automountServiceAccountToken }}
{{- end }}
securityContext:
{{- toYaml .Values.podSecurityContext | nindent 8 }}
initContainers:
Expand All @@ -36,6 +39,8 @@ spec:
{{- with .Values.extraVolumeMounts }}
{{- toYaml . | nindent 10 }}
{{- end }}
resources:
{{- toYaml .Values.initContainerResources | nindent 10 }}
envFrom:
- secretRef:
name: {{ include "OpenMetadata.fullname" . }}-config-secret
Expand Down
8 changes: 7 additions & 1 deletion charts/openmetadata/values.schema.json
Original file line number Diff line number Diff line change
Expand Up @@ -1014,6 +1014,12 @@
"replicaCount": {
"type": "integer"
},
"automountServiceAccountToken": {
"type": "boolean"
},
"initContainerResources": {
"type": "object"
},
"resources": {
"type": "object"
},
Expand Down Expand Up @@ -1121,4 +1127,4 @@
}
}
}
}
}
11 changes: 10 additions & 1 deletion charts/openmetadata/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -305,6 +305,7 @@ serviceAccount:
# The name of the service account to use.
# If not set and create is true, a name is generated using the fullname template
name: ""
automountServiceAccountToken: true
podSecurityContext: {}
# fsGroup: 2000
securityContext: {}
Expand Down Expand Up @@ -362,6 +363,14 @@ extraVolumeMounts: []

extraInitContainers: []

initContainerResources:
limits:
cpu: 1
memory: 2048Mi
requests:
cpu: 500m
memory: 256Mi

resources: {}
# We usually recommend not to specify default resources and to leave this as a conscious
# choice for the user. This also increases chances charts run on environments with little
Expand Down Expand Up @@ -409,4 +418,4 @@ podDisruptionBudget:
minAvailable: "1"

commonLabels: {}
podAnnotations: {}
podAnnotations: {}

0 comments on commit 13205b2

Please sign in to comment.