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
open-metadata · Nov 13, 2023 · eba3bf4 · eba3bf4
1 parent 159f6b0
commit eba3bf4
Show file tree

Hide file tree

Showing 9 changed files with 10 additions and 46 deletions.
diff --git 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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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