Skip to content

Commit

Permalink
[Fix] Airflow Auth (#199)
Browse files Browse the repository at this point in the history
* [Fix] Airflow Auth

* [Fix] Airflow Auth

* [Fix] Airflow Auth

* [Fix] Airflow Auth

* [Fix] Updating charts

* [Fix] Updating charts
  • Loading branch information
dhruvinmaniar123 authored Nov 21, 2023
1 parent 345a01d commit 3d830a4
Show file tree
Hide file tree
Showing 4 changed files with 14 additions and 5 deletions.
2 changes: 1 addition & 1 deletion charts/openmetadata/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ 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.3
version: 1.2.4

# 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
Expand Down
2 changes: 2 additions & 0 deletions charts/openmetadata/templates/_helpers.tpl
Original file line number Diff line number Diff line change
Expand Up @@ -359,6 +359,7 @@ OpenMetadata Configurations Environment Variables*/}}
value: "{{ .Values.openmetadata.config.pipelineServiceClientConfig.ingestionIpInfoEnabled }}"
- name: PIPELINE_SERVICE_CLIENT_ENDPOINT
value: "{{ .Values.openmetadata.config.pipelineServiceClientConfig.apiEndpoint }}"
{{ if .Values.openmetadata.config.pipelineServiceClientConfig.auth.enabled }}
{{- with .Values.openmetadata.config.pipelineServiceClientConfig.auth }}
- name: AIRFLOW_USERNAME
value: "{{ .username }}"
Expand All @@ -368,6 +369,7 @@ OpenMetadata Configurations Environment Variables*/}}
name: {{ .password.secretRef }}
key: {{ .password.secretKey }}
{{- end }}
{{- end }}
- name: PIPELINE_SERVICE_CLIENT_VERIFY_SSL
value: "{{ .Values.openmetadata.config.pipelineServiceClientConfig.verifySsl }}"
- name: PIPELINE_SERVICE_CLIENT_HOST_IP
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 @@ -166,6 +166,9 @@
"type": "object",
"additionalProperties": false,
"properties": {
"enabled" : {
"type": "boolean"
},
"password": {
"type": "object",
"additionalProperties": false,
Expand Down Expand Up @@ -1056,6 +1059,9 @@
"type": "object",
"additionalProperties": false,
"properties": {
"enabled" : {
"type": "boolean"
},
"config": {
"type": "object",
"properties": {
Expand All @@ -1082,4 +1088,4 @@
}
}
}
}
}
7 changes: 4 additions & 3 deletions charts/openmetadata/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -70,6 +70,7 @@ openmetadata:
# local path in Airflow Pod
sslCertificatePath: "/no/path"
auth:
enabled: true
username: admin
password:
secretRef: airflow-secrets
Expand Down Expand Up @@ -392,8 +393,8 @@ startupProbe:
podDisruptionBudget:
enabled: false
config:
maxUnavailable: 1
minAvailable: 1
maxUnavailable: "1"
minAvailable: "1"

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

0 comments on commit 3d830a4

Please sign in to comment.