From 38a5c7e590ca84a526b703cf030da9b58f325b94 Mon Sep 17 00:00:00 2001
From: IceS2 <pjt1991@gmail.com>
Date: Mon, 26 Aug 2024 16:08:11 +0200
Subject: [PATCH] feat: Prepare Release 1.5.0 (#254)

* Prepare Release 1.5.0

* Fix typo

* Fix typo

* avoid adding templates as an attribute ot the chart

* Update charts/openmetadata/templates/_helpers.tpl

Co-authored-by: Matias Puerta <matias@getcollate.io>

---------

Co-authored-by: Matias Puerta <matias@getcollate.io>
---
 charts/deps/Chart.yaml                     |  4 +--
 charts/deps/values.yaml                    |  2 +-
 charts/openmetadata/Chart.yaml             |  4 +--
 charts/openmetadata/README.md              |  2 +-
 charts/openmetadata/templates/_helpers.tpl | 16 +++++++++
 charts/openmetadata/templates/secrets.yaml |  9 +++++
 charts/openmetadata/values.schema.json     | 39 ++++++++++++++++++++++
 charts/openmetadata/values.yaml            | 20 +++++++++--
 8 files changed, 88 insertions(+), 8 deletions(-)

diff --git a/charts/deps/Chart.yaml b/charts/deps/Chart.yaml
index 8b07567..2720022 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.4.8
+version: 1.5.0
 
 # 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.4.8"
+appVersion: "1.5.0"
 
 home: https://open-metadata.org/
 
diff --git a/charts/deps/values.yaml b/charts/deps/values.yaml
index 4041342..8ea7e7e 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.4.8
+      tag: 1.5.0
       pullPolicy: "IfNotPresent"
     executor: "KubernetesExecutor"
     config:
diff --git a/charts/openmetadata/Chart.yaml b/charts/openmetadata/Chart.yaml
index b8bf31b..4269a6e 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.4.8
+version: 1.5.0
 
 # 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.4.8"
+appVersion: "1.5.0"
 
 home: https://open-metadata.org/
 
diff --git a/charts/openmetadata/README.md b/charts/openmetadata/README.md
index b2db36e..915b133 100644
--- a/charts/openmetadata/README.md
+++ b/charts/openmetadata/README.md
@@ -258,7 +258,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.4.8` |
+| image.tag | string | `1.5.0` |
 | imagePullSecrets | list | `[]` |
 | ingress.annotations | object | `{}` |
 | ingress.className | string | `""` |
diff --git a/charts/openmetadata/templates/_helpers.tpl b/charts/openmetadata/templates/_helpers.tpl
index dc5c1cd..0471087 100644
--- a/charts/openmetadata/templates/_helpers.tpl
+++ b/charts/openmetadata/templates/_helpers.tpl
@@ -172,6 +172,22 @@ OpenMetadata Configurations Azure Additional Parameters Environment Variables fo
 {{- end }}
 {{- end -}}
 
+
+{{/*
+OpenMetadata Configurations GCP Additional Parameters Environment Variables for Secret Manager
+*/}}
+{{- define "OpenMetadata.configs.secretManager.gcp.additionalParameters" -}}
+{{- with .Values.openmetadata.config.secretsManager.additionalParameters.projectId }}
+{{- if .secretRef }}
+- name: OM_SM_PROJECT_ID
+  valueFrom:
+    secretKeyRef:
+      name: {{ .secretRef }}
+      key: {{ .secretKey }}
+{{- end }}
+{{- end }}
+{{- end -}}
+
 {{/*
 OpenMetadata Configurations Environment Variables*/}}
 {{- define "OpenMetadata.configs" -}}
diff --git a/charts/openmetadata/templates/secrets.yaml b/charts/openmetadata/templates/secrets.yaml
index 9a5f671..ed1db61 100644
--- a/charts/openmetadata/templates/secrets.yaml
+++ b/charts/openmetadata/templates/secrets.yaml
@@ -28,6 +28,13 @@ data:
   OM_DATABASE: {{ .databaseName | b64enc }}
   DB_PARAMS: {{ .dbParams | b64enc | quote }}
   DB_USER: {{ .auth.username | b64enc }}
+  DB_CONNECTION_POOL_MAX_SIZE: {{ .maxSize | quote | b64enc }}
+  DB_CONNECTION_POOL_MIN_SIZE: {{ .minSize | quote | b64enc }}
+  DB_CONNECTION_POOL_INITIAL_SIZE: {{ .initialSize | quote | b64enc }}
+  DB_CONNECTION_CHECK_CONNECTION_WHILE_IDLE: {{ .checkConnectionWhileIdle | quote | b64enc }}
+  DB_CONNECTION_CHECK_CONNECTION_ON_BORROW: {{ .checkCOnnectionOnBorrow | quote | b64enc }}
+  DB_CONNECTION_EVICTION_INTERVAL: {{ .evictionInterval | quote | b64enc }}
+  DB_CONNECTION_MIN_IDLE_TIME: {{ .minIdleTime | quote | b64enc }}
 {{ end }}
 {{ end }}
 
@@ -201,6 +208,8 @@ data:
   WEB_CONF_REFERRER_POLICY_OPTION: {{ .referrerPolicy.option | quote | b64enc }}
   WEB_CONF_PERMISSION_POLICY_ENABLED: {{ .permissionPolicy.enabled | quote | b64enc }}
   WEB_CONF_PERMISSION_POLICY_OPTION: {{ .permissionPolicy.option | quote | b64enc }}
+  WEB_CONF_CACHE_CONTROL: {{ .cacheControl | quote | b64enc }}
+  WEB_CONF_PRAGMA: {{ .pragma | quote | b64enc }}
 {{ end }}
 {{ end }}
 
diff --git a/charts/openmetadata/values.schema.json b/charts/openmetadata/values.schema.json
index 0d7edc0..e6d2651 100644
--- a/charts/openmetadata/values.schema.json
+++ b/charts/openmetadata/values.schema.json
@@ -132,6 +132,12 @@
                                         }
                                     }
                                 },
+                                "cacheControl": {
+                                    "type": "string"
+                                },
+                                "pragma": {
+                                    "type": "string"
+                                },
                                 "xssProtection": {
                                     "type": "object",
                                     "additionalProperties": false,
@@ -747,6 +753,27 @@
                                 },
                                 "port": {
                                     "type": "integer"
+                                },
+                                "maxSize": {
+                                    "type": "integer"
+                                },
+                                "minSize": {
+                                    "type": "integer"
+                                },
+                                "initialSize": {
+                                    "type": "integer"
+                                },
+                                "checkConnectionWhileIdle": {
+                                    "type": "boolean"
+                                },
+                                "checkConnectionOnBorrow": {
+                                    "type": "boolean"
+                                },
+                                "evictionInterval": {
+                                    "type": "string"
+                                },
+                                "minIdleTime": {
+                                    "type": "string"
                                 }
                             }
                         },
@@ -1013,6 +1040,18 @@
                                                     "type": "string"
                                                 }
                                             }
+                                        },
+                                        "projectId": {
+                                            "type": "object",
+                                            "additionalProperties": false,
+                                            "properties": {
+                                                "secretKey": {
+                                                    "type": "string"
+                                                },
+                                                "secretRef": {
+                                                    "type": "string"
+                                                }
+                                            }
                                         }
                                     }
                                 },
diff --git a/charts/openmetadata/values.yaml b/charts/openmetadata/values.yaml
index 49a9dbb..05cd257 100644
--- a/charts/openmetadata/values.yaml
+++ b/charts/openmetadata/values.yaml
@@ -32,7 +32,7 @@ openmetadata:
       payLoadSize: 10485760
       connectionTimeoutSecs: 5
       socketTimeoutSecs: 60
-      batchSize: 10
+      batchSize: 100
       searchIndexMappingLanguage: "EN"
       keepAliveTimeoutSecs: 600
       trustStore:
@@ -60,6 +60,13 @@ openmetadata:
           secretRef: mysql-secrets
           secretKey: openmetadata-mysql-password
       dbParams: "allowPublicKeyRetrieval=true&useSSL=false&serverTimezone=UTC"
+      maxSize: 50
+      minSize: 10
+      initialSize: 10
+      checkConnectionWhileIdle: true
+      checkConnectionOnBorrow: true
+      evictionInterval: 5 minutes
+      minIdleTime: 1 minute
     pipelineServiceClientConfig:
       enabled: true
       className: "org.openmetadata.service.clients.pipeline.airflow.AirflowRESTClient"
@@ -254,7 +261,7 @@ openmetadata:
         secretKey: ""
     secretsManager:
       enabled: true
-      # Possible values are db, aws, aws-ssm, managed-aws, managed-aws-ssm, in-memory, managed-azure-kv, azure-kv
+      # Possible values are db, aws, aws-ssm, managed-aws, managed-aws-ssm, in-memory, managed-azure-kv, azure-kv, gcp
       provider: db
       # Define the secret key ID as /<prefix>/<clusterName>/<key> for AWS
       # Define the secret key ID as <prefix>-<clusterName>-<key> for Azure
@@ -302,6 +309,13 @@ openmetadata:
         # vaultName:
         #   secretRef: azure-vault-name-secret
         #   secretKey: azure-key-secret
+        # For GCP
+        projectId:
+          secretRef: ""
+          secretKey: ""
+        # projectId:
+        #   secretRef: gcp-project-id-secret
+        #   secretKey: gcp-key-secret
       # You can create Kubernetes secrets from AWS Credentials with the below command
       # kubectl create secret generic aws-key-secret \
       # --from-literal=aws-access-key-secret=<access_key_id_value> \
@@ -334,6 +348,8 @@ openmetadata:
       permissionPolicy:
         enabled: false
         option: ""
+      cacheControl: ""
+      pragma: ""
 
 networkPolicy:
   # If networkPolicy is true, following values can be set