check if opensearch plugin exists before removing

Signed-off-by: Yulong Ruan <[email protected]>
opensearch-project · Dec 25, 2024 · 09b7496 · 09b7496
1 parent 704bc9d
commit 09b7496
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 3 deletions.
diff --git a/charts/opensearch-dashboards/templates/deployment.yaml b/charts/opensearch-dashboards/templates/deployment.yaml
@@ -170,7 +170,9 @@ spec:
             #!/usr/bin/bash
             set -e
             {{- range $plugin := .Values.plugins.removeList }}
-            ./bin/opensearch-dashboards-plugin remove {{ $plugin }}
+            if ./bin/opensearch-dashboards-plugin list | grep -q {{ $plugin }}; then
+              ./bin/opensearch-dashboards-plugin remove {{ $plugin }}
+            fi
             {{- end }}
             {{- range $plugin := .Values.plugins.installList }}
             ./bin/opensearch-dashboards-plugin install {{ $plugin }}

diff --git a/charts/opensearch-dashboards/values.yaml b/charts/opensearch-dashboards/values.yaml
@@ -279,7 +279,7 @@ plugins:
   installList: []
   # - example-fake-plugin-downloadable-url
   removeList: []
-  # - securityDashboards
+  # - examplePluginName
 
 # ServiceMonitor Configuration for Prometheus
 # Enabling this option will create a ServiceMonitor resource that allows Prometheus to scrape metrics from the OpenSearch service.

diff --git a/charts/opensearch/values.yaml b/charts/opensearch/values.yaml
@@ -492,7 +492,7 @@ plugins:
   installList: []
   # - example-fake-plugin
   removeList: []
-  # - opensearch-ml
+  # - example-fake-plugin
 
 # -- Array of extra K8s manifests to deploy
 extraObjects: []