NCEAS / metacatui

MetacatUI: A client-side web interface for DataONE data repositories
https://nceas.github.io/metacatui
Apache License 2.0
42 stars 24 forks source link

Feature 2368 helm k8s #2427

Open artntek opened 1 month ago

artntek commented 1 month ago

Metacatui Helm Chart:

robyngit commented 1 month ago

Thanks for working on this @artntek !! Besides the in-line comments above, here are some general questions and thoughts that I came up looking through this:

artntek commented 4 weeks ago

It seems like the functionality is pretty independent of metacatui. I wonder if we should re-consider making it it's own repo (e.g. metacatui-helm) ? One advantage is that it would facilitate updating the helm logic without needing to do a MetacatUI release.

Thanks, @robyngit. I'm not opposed to having the helm chart be in its own repo, although the convention for all our other helm-deployed applications (metacat, metadig, indexer, bookkeeper) is to have the helm chart co-located with the code in the same repo.

We were discussing chart releases/versioning with @mbjones yesterday, and concluded they can be decoupled from the software releases/versioning, at least initially until chart development stabilizes. (i.e. we would not need an "official" metacatui release each time we update the chart, and vice-versa). We decided that we can sync the versions & releases at a later date, if we think it's appropriate.

We recognized that having different chart and software version numbering is confusing, but this follows the "industry standard" approach used by bitnami charts, and seems to be the only viable solution when charts are changing at a different rate than the software they are deploying.

7/1/24:

Latest version of metacatui helm chart addresses @robyngit's comments above, regarding themes: