Install fails on non-posthog namespace about charts-clickhouse HOT 6 CLOSED

To fix, change clickhouseOperator.namespace: "main-tools-staging" in values.yaml

cc @fuziontech @tiina303 - having this option still seems slightly weird. Perhaps it should just default to the same namespace?

from charts-clickhouse.

Thanks @tiina303 -- we'll leave in clickhouseOperator.namespace for now then!

Re: the wrapped config values, it's an internal convention to "wrap" charts as dependencies in custom charts of our own. It turns out thaat using posthog instead of posthog-clickhouse for this actually fixes the problems (so our Chart.yaml has name: posthog and all values are under the posthog key).

Closing this; thanks for your help!

from charts-clickhouse.

cc @fuziontech @tiina303 - having this option still seems slightly weird. Perhaps it should just default to the same namespace?

Yeah this is for legacy compatibility and can be nuked now. #79 addresses it

from charts-clickhouse.

The error still appears without setting clickhouseOperator.namespace (I actually only changed that in an attempt to work around the bug). Sorry, could've been clearer about that!

from charts-clickhouse.

The error still appears without setting clickhouseOperator.namespace (I actually only changed that in an attempt to work around the bug). Sorry, could've been clearer about that!

Sorry this is inconvenient. Currently you need to set that value or have "posthog" namespace. I agree that this is confusing and annoying that we need to define it, I'll look into removing that.

Why you're running into the error with your initial values that you provided: I'm not sure how your values are used as normally they would be at the top level, i.e.
instead of

posthog-clickhouse:
  cloud: "gcp"
  ...

you'd have

cloud: "gcp"
...

from charts-clickhouse.

Looked into the clickhouseOperator.namespace and by default it would otherwise install into kube-system namespace (https://github.com/Altinity/clickhouse-operator/blob/master/docs/operator_installation_details.md), however I think it's better if we keep all Posthog things in the same namespace and I couldn't find a way to use the current namespace.

from charts-clickhouse.