You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Currently, the kamaji Helm Chart is optionally installing its own datastore based on etcd. At same time, we developed the kamaji-etcd Helm Chart to provide a multi-tenant datastore for Kamaji based on etcd . To avoid duplication in maintenance of the charts, it would be useful to remove the native etcd datastore option in Kamaji Helm Chart and use the kamaji-etcd instead as dependancy sub-chart.
The text was updated successfully, but these errors were encountered:
This would be really useful. I just got confused by this trying Kamaji for the first time. The helm chart is also more configurable than the kamaji chart installation. For example I cannot add annotations to the serviceAccount for etcd in the kamaji chart.
Edit: It's even worse. The ETCD statefulset is hard-coded to use default serviceAccount.
The ETCD statefulset is hard-coded to use default serviceAccount
Although this should be discussed in the kamaji-etcd chart, there's a rationale behind that, mostly related to the fact etcd is not expecting to interact with the Kubernetes API Server: the actions performed there, such as backups, are triggere with a temporary ServiceAccount deployed thanks to Helm hooks (source).
@prometherion I appreciate the opportunity, but I'm afraid I don't have very much insight into how Kamaji uses etcd under the hood, nor exactly how helm sub charts function, so I'd be very worried about causing loss of data for existing users of the charts.
Currently, the kamaji Helm Chart is optionally installing its own datastore based on
etcd
. At same time, we developed the kamaji-etcd Helm Chart to provide a multi-tenant datastore for Kamaji based onetcd
. To avoid duplication in maintenance of the charts, it would be useful to remove the nativeetcd
datastore option in Kamaji Helm Chart and use the kamaji-etcd instead as dependancy sub-chart.The text was updated successfully, but these errors were encountered: