Seagate / cortx-k8s

CORTX Kubernetes Orchestration Repository
https://github.com/Seagate/cortx
Apache License 2.0
6 stars 47 forks source link

Let Helm manage dependent Chart install #329

Closed keithpine closed 2 years ago

keithpine commented 2 years ago

Description

Helm can update and install Chart dependencies by using the install --dependency-update flag. It isn't necessary to manually install the repositories and then build the dependencies.

If the local chart files exist already on the next deployment, Helm won't go out and attempt to update them. So this is a slight optimization in that it's less likely to require contacting the remote Chart repos every deployment.

In the future, we can avoid this completely by creating a CORTX Chart artifact (e.g. helm package) and bundle the dependencies with it, and update instructions to use the released Chart (then later, create a Helm repo).

Type of change

How was this tested?

  1. Fresh install with no Helm repos installed and a clean "charts" directory. Deploy script adds repos and downloads the charts automatically
  2. A subsequent deployment with charts already downloaded. Deploy script does not update repos or download anything.
  3. A subsequent deployment with one of the chart .gz files deleted. Deploy script updates repost and downloads the charts automatically.

Additional information

Checklist

If this change requires newer CORTX or third party image versions:

If this change addresses a CORTX Jira issue:


View rendered charts/cortx/README.md View rendered charts/cortx/README.md.gotmpl