Merge pull request #247 from broadinstitute/se/DR-3413-remove-elastic…

…search-module

[DR-3413] Remove de-elasticsearch chart

.github/test-config.yaml

@@ -12,7 +12,6 @@ excluded-charts:
   - charts/oidc-proxy
   - charts/gcloud-sqlproxy
   - charts/create-secret-manager-secret
-  - charts/de-elasticsearch
 charts:
   - charts/backend-config
   - charts/datarepo

.github/workflows/releasedr.yaml

@@ -24,7 +24,6 @@ on:
       - 'charts/gcloud-sqlproxy/**'
       - 'charts/oidc-proxy/**'
       - 'charts/gcp-managed-cert/**'
-      - 'charts/de-elasticsearch/**'
 
 jobs:
   release_new_umbrella_dr:

.github/workflows/releasenondr.yaml

@@ -15,7 +15,6 @@ on:
       - '!charts/create-secret-manager-secret/**'
       - '!charts/gcloud-sqlproxy/**'
       - '!charts/oidc-proxy/**'
-      - '!charts/de-elasticsearch/**'
 
 jobs:
   release_non_dr_chart:

charts/datarepo/Chart.lock

@@ -14,11 +14,8 @@ dependencies:
 - name: create-secret-manager-secret
   repository: https://broadinstitute.github.io/datarepo-helm/
   version: 0.0.6
-- name: de-elasticsearch
-  repository: https://broadinstitute.github.io/datarepo-helm/
-  version: 0.1.4
 - name: postgres
   repository: https://terra-helm.storage.googleapis.com/
   version: 0.26.0
-digest: sha256:abbabe591136d12e44e97bd6e538fb07b50676ad43a4d75dd9c0b73cad291399
-generated: "2024-01-17T16:51:44.429599987Z"
+digest: sha256:98f06426708ce763118547e1a62b2b060b387b12944800d809de105c6a03aef8
+generated: "2024-01-18T13:25:16.141926-05:00"

charts/datarepo/Chart.yaml

@@ -13,8 +13,8 @@ type: application
 # This is the chart version. This version number should be incremented each time you make changes
 # to the chart and its templates, including the app version
 
-version: 0.1.834
-appVersion: 0.1.834
+version: 0.2.0
+appVersion: 0.2.0
 keywords:
 - google
 - cloud
@@ -47,10 +47,6 @@ dependencies:
   version: 0.0.6
   repository: https://broadinstitute.github.io/datarepo-helm/
   condition: create-secret-manager-secret.enabled
-- name: de-elasticsearch
-  version: 0.1.4
-  repository: https://broadinstitute.github.io/datarepo-helm/
-  condition: de-elasticsearch.enabled
 - name: postgres
   version: 0.26.0
   repository: https://terra-helm.storage.googleapis.com/

charts/datarepo/README.md

@@ -8,7 +8,6 @@ This chart will do the following:
   - [datarepo-ui](https://github.com/broadinstitute/datarepo-helm/tree/master/charts/datarepo-ui)
   - [oidc-proxy](https://github.com/broadinstitute/datarepo-helm/tree/master/charts/oidc-proxy)
   - [gcloud-sqlproxy](https://github.com/broadinstitute/datarepo-helm/tree/master/charts/gcloud-sqlproxy)
-  - [de-elasticsearch](https://github.com/broadinstitute/datarepo-helm/tree/master/charts/de-elasticsearch)  
   - [Postgresql](https://github.com/helm/charts/tree/master/stable/postgresql)
 
 ## Prerequisite

charts/datarepo/ci/testing-values.yaml

@@ -101,7 +101,3 @@ oidc-proxy:
   rbac:
     create: true
     pspEnabled: true
-
-de-elasticsearch:
-  enabled: true
-  fullnameOverride: test-de-elasticsearch

charts/datarepo/values.yaml

@@ -86,9 +86,6 @@ datarepo-ui:
   ##  rbac:
   ##    create: true
 
-de-elasticsearch:
-  enabled: false
-
 oidc-proxy:
   enabled: false ## if true uncomment below
   ##  env:
@@ -121,7 +118,7 @@ oidc-proxy:
   ##      -----END CERTIFICATE-----
 
 # enable local postgres chart deployment (rather than external cloudsql)
-postgres: 
+postgres:
   enabled: false
 
 ## https://github.com/helm/charts/tree/master/stable/postgresql