search

stardog-union / helm-charts

Stardog Helm Charts
Apache License 2.0
9 stars 12 forks source link

Allow namespace to be specified in values file #30

Closed pdmars closed 3 years ago

pdmars commented 3 years ago

Currently we deploy stardog into the helm release namespace. However, for users that have stardog as a dependency to another application and want it to run in its own namespace, they need to be able to override which namespace Stardog is deployed into so we should allow that via an option that can be set.

pdmars commented 3 years ago

The ZooKeeper chart defaults to the .Release.Namespace so that will need to be fixed downstream.

danielburrell commented 3 years ago

The ZooKeeper chart defaults to the .Release.Namespace so that will need to be fixed downstream.

That's fine, I see https://github.com/stardog-union/helm-charts/issues/21 is open, I'll follow up with bitnami about sub-chart namespaces.

danielburrell commented 3 years ago

Zookeeper PR is out. https://github.com/bitnami/charts/pull/3881 though I know at this current time you're not using bitnami yet