From 159f6b02674228d5dc84a1c9d4d8b79194995358 Mon Sep 17 00:00:00 2001
From: Akash Jain <15995028+akash-jain-10@users.noreply.github.com>
Date: Fri, 10 Nov 2023 15:18:59 +0530
Subject: [PATCH 1/5] feat: Migrate to OpenSearch 2.7 as Chart's default search
 dependency (#194)

* feat: Add OS Deps; update mysql charts dep version

* fix(docs): Update README.md
---
 charts/deps/Chart.lock          | 14 +++++++-------
 charts/deps/Chart.yaml          | 12 ++++++------
 charts/deps/values.yaml         | 28 ++++++++++++++--------------
 charts/openmetadata/README.md   |  4 ++--
 charts/openmetadata/values.yaml |  4 ++--
 5 files changed, 31 insertions(+), 31 deletions(-)

diff --git a/charts/deps/Chart.lock b/charts/deps/Chart.lock
index eab4b4c3..69d7c7d1 100644
--- a/charts/deps/Chart.lock
+++ b/charts/deps/Chart.lock
@@ -1,12 +1,12 @@
 dependencies:
 - name: mysql
-  repository: https://raw.githubusercontent.com/bitnami/charts/archive-full-index/bitnami
-  version: 9.7.1
-- name: elasticsearch
-  repository: https://helm.elastic.co
-  version: 8.5.1
+  repository: https://charts.bitnami.com/bitnami
+  version: 9.7.2
 - name: airflow
   repository: https://airflow-helm.github.io/charts
   version: 8.8.0
-digest: sha256:c4a68ed6f7480bda021d99bb39daf90ed686edeae3605389158b2fc5b9d05f7c
-generated: "2023-10-26T15:20:32.348887+05:30"
+- name: opensearch
+  repository: https://opensearch-project.github.io/helm-charts/
+  version: 2.12.1
+digest: sha256:ef5c9862e7107bdf2ba81ffb30b9da74b6072dec41800a80723eb08bcdac6022
+generated: "2023-11-07T16:06:28.907894+05:30"
diff --git a/charts/deps/Chart.yaml b/charts/deps/Chart.yaml
index 2f1fd3b8..a5860515 100644
--- a/charts/deps/Chart.yaml
+++ b/charts/deps/Chart.yaml
@@ -60,14 +60,14 @@ icon: https://open-metadata.org/images/favicon.png
 # Add Dependencies of other charts
 dependencies:
 - name: mysql
-  version: "9.7.1"
-  repository: "https://raw.githubusercontent.com/bitnami/charts/archive-full-index/bitnami"
+  version: "9.7.2"
+  repository: "https://charts.bitnami.com/bitnami"
   condition: mysql.enabled
-- name: elasticsearch
-  version: "8.5.1"
-  repository: "https://helm.elastic.co"
-  condition: elasticsearch.enabled
 - name: airflow
   version: "8.8.0"
   repository: "https://airflow-helm.github.io/charts"
   condition: airflow.enabled
+- name: opensearch
+  version: "2.12.1"
+  repository: "https://opensearch-project.github.io/helm-charts/"
+  condition: opensearch.enabled
diff --git a/charts/deps/values.yaml b/charts/deps/values.yaml
index c00c96b2..d8b715db 100644
--- a/charts/deps/values.yaml
+++ b/charts/deps/values.yaml
@@ -30,22 +30,22 @@ mysql:
       GRANT ALL PRIVILEGES ON airflow_db.* TO 'airflow_user'@'%' WITH GRANT OPTION;
       commit;
 
-# ElasticSearch configurations for helm dependency
-# you can find more details about this here https://artifacthub.io/packages/helm/elastic/elasticsearch
-elasticsearch:
+# OpenSearch Helm Dependency
+# you can find more details about this here https://artifacthub.io/packages/helm/opensearch-project-helm-charts/opensearch/2.12.2
+opensearch:
   enabled: true
-  replicas: 1
-  minimumMasterNodes: 1
-  fullnameOverride: "elasticsearch"
-  imagePullPolicy: "Always"
-  esJavaOpts: "-Xmx1024m -Xms1024m"
+  clusterName: opensearch
+  fullnameOverride: opensearch
+  nodeGroup: ""
+  imagePullPolicy: Always
+  opensearchJavaOpts: "-Xmx1g -Xms1g"
+  persistence:
+    size: 30Gi
   protocol: http
-  createCert: false
-  esConfig:
-    elasticsearch.yml: |
-      xpack.security.enabled: false
-  maxUnavailable: 0
-  clusterHealthCheckParams: "wait_for_status=yellow&timeout=20s"
+  config:
+    opensearch.yml: |
+      plugins.security.disabled: true
+  singleNode: true
   resources:
     requests:
       cpu: "100m"
diff --git a/charts/openmetadata/README.md b/charts/openmetadata/README.md
index d5947474..b53bebad 100644
--- a/charts/openmetadata/README.md
+++ b/charts/openmetadata/README.md
@@ -117,10 +117,10 @@ helm install openmetadata open-metadata/openmetadata --values <<path-to-values-f
 | openmetadata.config.elasticsearch.auth.username | string | `elasticsearch` | ELASTICSEARCH_USER |
 | openmetadata.config.elasticsearch.auth.password.secretRef | string | `elasticsearch-secrets` | ELASTICSEARCH_PASSWORD |
 | openmetadata.config.elasticsearch.auth.password.secretKey | string | `openmetadata-elasticsearch-password` | ELASTICSEARCH_PASSWORD |
-| openmetadata.config.elasticsearch.host | string | `elasticsearch` | ELASTICSEARCH_HOST |
+| openmetadata.config.elasticsearch.host | string | `opensearch` | ELASTICSEARCH_HOST |
 | openmetadata.config.elasticsearch.keepAliveTimeoutSecs | int | `600` | ELASTICSEARCH_KEEP_ALIVE_TIMEOUT_SECS |
 | openmetadata.config.elasticsearch.port | int | 9200 | ELASTICSEARCH_PORT |
-| openmetadata.config.elasticsearch.searchType | string | `elasticsearch` | SEARCH_TYPE |
+| openmetadata.config.elasticsearch.searchType | string | `opensearch` | SEARCH_TYPE |
 | openmetadata.config.elasticsearch.scheme | string | `http` | ELASTICSEARCH_SCHEME |
 | openmetadata.config.elasticsearch.searchIndexMappingLanguage | string | `EN`| ELASTICSEARCH_INDEX_MAPPING_LANG |
 | openmetadata.config.elasticsearch.trustStore.enabled | bool | `false` | |
diff --git a/charts/openmetadata/values.yaml b/charts/openmetadata/values.yaml
index a93b5502..8591c489 100644
--- a/charts/openmetadata/values.yaml
+++ b/charts/openmetadata/values.yaml
@@ -23,8 +23,8 @@ openmetadata:
       port: 8585
       adminPort: 8586
     elasticsearch:
-      host: elasticsearch
-      searchType: elasticsearch
+      host: opensearch
+      searchType: opensearch
       port: 9200
       scheme: http
       connectionTimeoutSecs: 5

From eba3bf4bcfbb340bf0acbfaf2c8e372668fb5eb4 Mon Sep 17 00:00:00 2001
From: Akash Jain <15995028+akash-jain-10@users.noreply.github.com>
Date: Mon, 13 Nov 2023 17:40:01 +0530
Subject: [PATCH 2/5] feat: Prepare 1.2.1 Application Release (#195)

* fix: remove `MIGRATION_EXTENSION_PATH`

* fix: remove basic login extended configurations

* chore: bump versions

* fix(CI): Update helm repo add step with the changes

* fix(ci): Chart lint fixes
---
 .github/workflows/lint-test.yaml           |  4 ++--
 .github/workflows/release.yaml             |  4 ++--
 charts/deps/Chart.yaml                     |  4 ++--
 charts/deps/values.yaml                    |  2 +-
 charts/openmetadata/Chart.yaml             |  4 ++--
 charts/openmetadata/README.md              |  5 +----
 charts/openmetadata/templates/_helpers.tpl |  6 ------
 charts/openmetadata/values.schema.json     | 21 ---------------------
 charts/openmetadata/values.yaml            |  6 ------
 9 files changed, 10 insertions(+), 46 deletions(-)

diff --git a/.github/workflows/lint-test.yaml b/.github/workflows/lint-test.yaml
index 7010cacb..363193c8 100644
--- a/.github/workflows/lint-test.yaml
+++ b/.github/workflows/lint-test.yaml
@@ -35,8 +35,8 @@ jobs:
       
       - name: Add Helm repos
         run: |
-          helm repo add elasticsearch https://helm.elastic.co
-          helm repo add mysql https://raw.githubusercontent.com/bitnami/charts/archive-full-index/bitnami
+          helm repo add opensearch https://opensearch-project.github.io/helm-charts/
+          helm repo add mysql https://charts.bitnami.com/bitnami
           helm repo add airflow https://airflow-helm.github.io/charts
 
       - uses: actions/setup-python@v2
diff --git a/.github/workflows/release.yaml b/.github/workflows/release.yaml
index ae6c0332..33243829 100644
--- a/.github/workflows/release.yaml
+++ b/.github/workflows/release.yaml
@@ -24,8 +24,8 @@ jobs:
       
       - name: Add Helm repos
         run: |
-          helm repo add elasticsearch https://helm.elastic.co
-          helm repo add mysql https://raw.githubusercontent.com/bitnami/charts/archive-full-index/bitnami
+          helm repo add opensearch https://opensearch-project.github.io/helm-charts/
+          helm repo add mysql https://charts.bitnami.com/bitnami
           helm repo add airflow https://airflow-helm.github.io/charts
 
       - name: Run chart-releaser
diff --git a/charts/deps/Chart.yaml b/charts/deps/Chart.yaml
index a5860515..88a338ac 100644
--- a/charts/deps/Chart.yaml
+++ b/charts/deps/Chart.yaml
@@ -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.1
+version: 1.2.2
 
 # 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.0"
+appVersion: "1.2.1"
 
 home: https://open-metadata.org/
 
diff --git a/charts/deps/values.yaml b/charts/deps/values.yaml
index d8b715db..dc76902c 100644
--- a/charts/deps/values.yaml
+++ b/charts/deps/values.yaml
@@ -61,7 +61,7 @@ airflow:
   airflow:
     image:
       repository: docker.getcollate.io/openmetadata/ingestion
-      tag: 1.2.0
+      tag: 1.2.1
       pullPolicy: "IfNotPresent"
     executor: "KubernetesExecutor"
     config:
diff --git a/charts/openmetadata/Chart.yaml b/charts/openmetadata/Chart.yaml
index 30f62bb5..c3a693d8 100644
--- a/charts/openmetadata/Chart.yaml
+++ b/charts/openmetadata/Chart.yaml
@@ -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.1
+version: 1.2.2
 
 # 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.0"
+appVersion: "1.2.1"
 
 home: https://open-metadata.org/
 
diff --git a/charts/openmetadata/README.md b/charts/openmetadata/README.md
index b53bebad..607c0109 100644
--- a/charts/openmetadata/README.md
+++ b/charts/openmetadata/README.md
@@ -101,8 +101,6 @@ helm install openmetadata open-metadata/openmetadata --values <<path-to-values-f
 | openmetadata.config.airflow.openmetadata.serverHostApiUrl | string | `http://openmetadata:8585/api` | SERVER_HOST_API_URL |
 | openmetadata.config.airflow.sslCertificatePath | string | `/no/path` | PIPELINE_SERVICE_CLIENT_SSL_CERT_PATH |
 | openmetadata.config.airflow.verifySsl | string | `no-ssl` | PIPELINE_SERVICE_CLIENT_VERIFY_SSL |
-| openmetadata.config.basicLogin.maxLoginFailAttempts | int | 3 | OM_MAX_FAILED_LOGIN_ATTEMPTS |
-| openmetadata.config.basicLogin.accessBlockTime | int | 600 | OM_LOGIN_ACCESS_BLOCK_TIME |
 | openmetadata.config.clusterName | string | `openmetadata` | OPENMETADATA_CLUSTER_NAME |
 | openmetadata.config.database.auth.password.secretRef | string | `mysql-secrets` | DB_USER_PASSWORD |
 | openmetadata.config.database.auth.password.secretKey | string | `openmetadata-mysql-password` | DB_USER_PASSWORD |
@@ -141,7 +139,6 @@ helm install openmetadata open-metadata/openmetadata --values <<path-to-values-f
 | openmetadata.config.jwtTokenConfiguration.keyId | string | `Gb389a-9f76-gdjs-a92j-0242bk94356` | JWT_KEY_ID |
 | openmetadata.config.logLevel | string | `INFO` | LOG_LEVEL |
 | openmetadata.config.maskPasswordsApi | bool | `false` | MASK_PASSWORDS_API |
-| openmetadata.config.migrationConfigs.extensionPath | string | `Empty string` | MIGRATION_EXTENSION_PATH |
 | openmetadata.config.openmetadata.adminPort | int | 8586 | SERVER_ADMIN_PORT |
 | openmetadata.config.openmetadata.host | string | `openmetadata` | OPENMETADATA_SERVER_URL |
 | openmetadata.config.openmetadata.port | int | 8585 | SERVER_PORT |
@@ -209,7 +206,7 @@ helm install openmetadata open-metadata/openmetadata --values <<path-to-values-f
 | fullnameOverride | string | `"openmetadata"` |
 | image.pullPolicy | string | `"Always"` |
 | image.repository | string | `"docker.getcollate.io/openmetadata/server"` |
-| image.tag | string | `1.2.0` |
+| image.tag | string | `1.2.1` |
 | imagePullSecrets | list | `[]` |
 | ingress.annotations | object | `{}` |
 | ingress.className | string | `""` |
diff --git a/charts/openmetadata/templates/_helpers.tpl b/charts/openmetadata/templates/_helpers.tpl
index 435f6044..c840c4d8 100644
--- a/charts/openmetadata/templates/_helpers.tpl
+++ b/charts/openmetadata/templates/_helpers.tpl
@@ -110,8 +110,6 @@ OpenMetadata Configurations Environment Variables*/}}
       name: {{ include "OpenMetadata.fullname" . }}-secret 
       key: FERNET_KEY
 {{- end }}
-- name: MIGRATION_EXTENSION_PATH
-  value: "{{ .Values.openmetadata.config.migrationConfigs.extensionPath }}"
 - name: EVENT_MONITOR
   value: "{{ .Values.openmetadata.config.eventMonitor.type }}"
 - name: EVENT_MONITOR_BATCH_SIZE
@@ -148,10 +146,6 @@ OpenMetadata Configurations Environment Variables*/}}
   value: '[{{ include "OpenMetadata.commaJoinedQuotedList" (dict "value" .Values.openmetadata.config.authentication.jwtPrincipalClaims) }}]'
 - name: AUTHENTICATION_ENABLE_SELF_SIGNUP
   value: "{{ .Values.openmetadata.config.authentication.enableSelfSignup }}"
-- name: OM_MAX_FAILED_LOGIN_ATTEMPTS
-  value: "{{ .Values.openmetadata.config.basicLogin.maxLoginFailAttempts }}"
-- name: OM_LOGIN_ACCESS_BLOCK_TIME
-  value: "{{ .Values.openmetadata.config.basicLogin.accessBlockTime }}"
 - name: AUTHORIZER_CLASS_NAME
   value: "{{ .Values.openmetadata.config.authorizer.className }}"
 - name: AUTHORIZER_REQUEST_FILTER
diff --git a/charts/openmetadata/values.schema.json b/charts/openmetadata/values.schema.json
index f547ca5b..bb04ced4 100644
--- a/charts/openmetadata/values.schema.json
+++ b/charts/openmetadata/values.schema.json
@@ -39,15 +39,6 @@
                         "errorMessage": "Global keyword has been replaced by openmetadata.config"
                     },
                     "properties": {
-                        "migrationConfigs": {
-                            "type": "object",
-                            "additionalProperties": false,
-                            "properties": {
-                                "extensionPath": {
-                                    "type": "string"
-                                }
-                            }
-                        },
                         "web": {
                             "type": "object",
                             "additionalProperties": false,
@@ -549,18 +540,6 @@
                                 }
                             }
                         },
-                        "basicLogin": {
-                            "type": "object",
-                            "additionalProperties": false,
-                            "properties": {
-                                "accessBlockTime": {
-                                    "type": "integer"
-                                },
-                                "maxLoginFailAttempts": {
-                                    "type": "integer"
-                                }
-                            }
-                        },
                         "clusterName": {
                             "type": "string"
                         },
diff --git a/charts/openmetadata/values.yaml b/charts/openmetadata/values.yaml
index 8591c489..194057a6 100644
--- a/charts/openmetadata/values.yaml
+++ b/charts/openmetadata/values.yaml
@@ -7,8 +7,6 @@ replicaCount: 1
 # Below are defaults as per openmetadata-dependencies Helm Chart Values
 openmetadata:
   config:
-    migrationConfigs:
-      extensionPath: ""
     upgradeMigrationConfigs:
       force: false
       migrationLimitParam: 1200
@@ -162,10 +160,6 @@ openmetadata:
             secretRef: ""
             secretKey: ""
 
-    basicLogin:
-      maxLoginFailAttempts: 3
-      # in seconds
-      accessBlockTime: 600
     jwtTokenConfiguration:
       enabled: true
       # File Path on Airflow Container

From 9469bacbacb54a1adda644e2fcd9b5ee6edc0028 Mon Sep 17 00:00:00 2001
From: Akash Jain <15995028+akash-jain-10@users.noreply.github.com>
Date: Mon, 20 Nov 2023 17:30:15 +0530
Subject: [PATCH 3/5] chore: Version Bump to 1.2.2 Application Release (#198)

---
 charts/deps/Chart.yaml         | 4 ++--
 charts/deps/values.yaml        | 2 +-
 charts/openmetadata/Chart.yaml | 4 ++--
 charts/openmetadata/README.md  | 2 +-
 4 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/charts/deps/Chart.yaml b/charts/deps/Chart.yaml
index 88a338ac..7cdb45cd 100644
--- a/charts/deps/Chart.yaml
+++ b/charts/deps/Chart.yaml
@@ -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.2
+version: 1.2.3
 
 # 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.1"
+appVersion: "1.2.2"
 
 home: https://open-metadata.org/
 
diff --git a/charts/deps/values.yaml b/charts/deps/values.yaml
index dc76902c..e2d7634e 100644
--- a/charts/deps/values.yaml
+++ b/charts/deps/values.yaml
@@ -61,7 +61,7 @@ airflow:
   airflow:
     image:
       repository: docker.getcollate.io/openmetadata/ingestion
-      tag: 1.2.1
+      tag: 1.2.2
       pullPolicy: "IfNotPresent"
     executor: "KubernetesExecutor"
     config:
diff --git a/charts/openmetadata/Chart.yaml b/charts/openmetadata/Chart.yaml
index c3a693d8..8b2a188b 100644
--- a/charts/openmetadata/Chart.yaml
+++ b/charts/openmetadata/Chart.yaml
@@ -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.2
+version: 1.2.3
 
 # 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.1"
+appVersion: "1.2.2"
 
 home: https://open-metadata.org/
 
diff --git a/charts/openmetadata/README.md b/charts/openmetadata/README.md
index 607c0109..c0b3aed3 100644
--- a/charts/openmetadata/README.md
+++ b/charts/openmetadata/README.md
@@ -206,7 +206,7 @@ helm install openmetadata open-metadata/openmetadata --values <<path-to-values-f
 | fullnameOverride | string | `"openmetadata"` |
 | image.pullPolicy | string | `"Always"` |
 | image.repository | string | `"docker.getcollate.io/openmetadata/server"` |
-| image.tag | string | `1.2.1` |
+| image.tag | string | `1.2.2` |
 | imagePullSecrets | list | `[]` |
 | ingress.annotations | object | `{}` |
 | ingress.className | string | `""` |

From 345a01dff01fb81a96cf4683ff73341a617b2032 Mon Sep 17 00:00:00 2001
From: Preet Shah <48907607+preetsshah@users.noreply.github.com>
Date: Tue, 21 Nov 2023 10:53:31 +0530
Subject: [PATCH 4/5] Introduction of poddisruptionbudget for openmetadata
 server (#197)

* Introduction of poddisruptionbudget for openmetadata server

* Updated PR as suggested in previous commit
---
 .../templates/poddisruptionbudget.yaml        | 20 +++++++++++++++++++
 charts/openmetadata/values.schema.json        | 17 ++++++++++++++++
 charts/openmetadata/values.yaml               |  8 +++++++-
 3 files changed, 44 insertions(+), 1 deletion(-)
 create mode 100644 charts/openmetadata/templates/poddisruptionbudget.yaml

diff --git a/charts/openmetadata/templates/poddisruptionbudget.yaml b/charts/openmetadata/templates/poddisruptionbudget.yaml
new file mode 100644
index 00000000..f0f16681
--- /dev/null
+++ b/charts/openmetadata/templates/poddisruptionbudget.yaml
@@ -0,0 +1,20 @@
+{{- if .Values.podDisruptionBudget.enabled }}
+apiVersion: policy/v1
+kind: PodDisruptionBudget
+metadata:
+  name: {{ include "OpenMetadata.fullname" . }}-poddisruptionbudget
+  labels:
+    {{- include "OpenMetadata.labels" . | indent 4 }}
+spec:
+  {{- with .Values.podDisruptionBudget.config }}
+  {{- if .minAvailable }}
+  minAvailable: {{ .minAvailable }}
+  {{- end }}
+  {{- if .maxUnavailable }}
+  maxUnavailable: {{ .maxUnavailable }}
+  {{- end }}
+  {{- end }}
+  selector:
+    matchLabels:
+      {{- include "OpenMetadata.selectorLabels" . | nindent 6 }}
+{{- end }}
\ No newline at end of file
diff --git a/charts/openmetadata/values.schema.json b/charts/openmetadata/values.schema.json
index bb04ced4..ffb84ac4 100644
--- a/charts/openmetadata/values.schema.json
+++ b/charts/openmetadata/values.schema.json
@@ -1052,6 +1052,23 @@
         "commonLabels": {
             "type": "object"
         },
+        "podDisruptionBudget": {
+            "type": "object",
+            "additionalProperties": false,
+            "properties": {
+                "config": {
+                    "type": "object",
+                    "properties": {
+                        "maxUnavailable": {
+                            "type": "string"
+                        },
+                        "minAvailable": {
+                            "type": "string"
+                        }
+                    }
+                }
+            }
+        },
         "podAnnotations": {
             "type": "object"
         },
diff --git a/charts/openmetadata/values.yaml b/charts/openmetadata/values.yaml
index 194057a6..7c791e14 100644
--- a/charts/openmetadata/values.yaml
+++ b/charts/openmetadata/values.yaml
@@ -389,5 +389,11 @@ startupProbe:
     path: /healthcheck
     port: http-admin
 
+podDisruptionBudget:
+  enabled: false
+  config:
+    maxUnavailable: 1
+    minAvailable: 1
+
 commonLabels: {}
-podAnnotations: {}
+podAnnotations: {}
\ No newline at end of file

From 3d830a45aa84917592ad1e61a85ff8190fbd8d7a Mon Sep 17 00:00:00 2001
From: Dhruvin Maniar <dhruvinmaniar123@users.noreply.github.com>
Date: Tue, 21 Nov 2023 19:20:17 +0530
Subject: [PATCH 5/5] [Fix] Airflow Auth (#199)

* [Fix] Airflow Auth

* [Fix] Airflow Auth

* [Fix] Airflow Auth

* [Fix] Airflow Auth

* [Fix] Updating charts

* [Fix] Updating charts
---
 charts/openmetadata/Chart.yaml             | 2 +-
 charts/openmetadata/templates/_helpers.tpl | 2 ++
 charts/openmetadata/values.schema.json     | 8 +++++++-
 charts/openmetadata/values.yaml            | 7 ++++---
 4 files changed, 14 insertions(+), 5 deletions(-)

diff --git a/charts/openmetadata/Chart.yaml b/charts/openmetadata/Chart.yaml
index 8b2a188b..12aa43fe 100644
--- a/charts/openmetadata/Chart.yaml
+++ b/charts/openmetadata/Chart.yaml
@@ -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
diff --git a/charts/openmetadata/templates/_helpers.tpl b/charts/openmetadata/templates/_helpers.tpl
index c840c4d8..23b79447 100644
--- a/charts/openmetadata/templates/_helpers.tpl
+++ b/charts/openmetadata/templates/_helpers.tpl
@@ -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 }}"
@@ -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
diff --git a/charts/openmetadata/values.schema.json b/charts/openmetadata/values.schema.json
index ffb84ac4..3169a681 100644
--- a/charts/openmetadata/values.schema.json
+++ b/charts/openmetadata/values.schema.json
@@ -166,6 +166,9 @@
                                     "type": "object",
                                     "additionalProperties": false,
                                     "properties": {
+                                        "enabled" : {
+                                            "type": "boolean"
+                                        },
                                         "password": {
                                             "type": "object",
                                             "additionalProperties": false,
@@ -1056,6 +1059,9 @@
             "type": "object",
             "additionalProperties": false,
             "properties": {
+                "enabled" : {
+                    "type": "boolean"
+                },
                 "config": {
                     "type": "object",
                     "properties": {
@@ -1082,4 +1088,4 @@
             }
         }
     }
-}
\ No newline at end of file
+}
diff --git a/charts/openmetadata/values.yaml b/charts/openmetadata/values.yaml
index 7c791e14..3e8b6d53 100644
--- a/charts/openmetadata/values.yaml
+++ b/charts/openmetadata/values.yaml
@@ -70,6 +70,7 @@ openmetadata:
       # local path in Airflow Pod
       sslCertificatePath: "/no/path"
       auth:
+        enabled: true
         username: admin
         password:
           secretRef: airflow-secrets
@@ -392,8 +393,8 @@ startupProbe:
 podDisruptionBudget:
   enabled: false
   config:
-    maxUnavailable: 1
-    minAvailable: 1
+    maxUnavailable: "1"
+    minAvailable: "1"
 
 commonLabels: {}
-podAnnotations: {}
\ No newline at end of file
+podAnnotations: {}