Email setup from the PostHog app about charts-clickhouse HOT 9 CLOSED

@tiina303 can you add more info about this? Which secret are we trying to create?

from charts-clickhouse.

@paolodamico has been working on this, not sure if there was an issue tracking it already?

from charts-clickhouse.

@hazzadous I'm inclined to close this as part of #286. What do you think?

from charts-clickhouse.

I think this is actually a different issue @guidoiaquinti , assuming that the other linked PR is relevant. i.e. the desire is to setup email via the app UI(?)

from charts-clickhouse.

Yes but I think it was to avoid passing email settings via env variables, right @tiina303 ?

If yes, this issue should be in posthog/posthog

from charts-clickhouse.

I already have a PR that accomplishes this almost ready to go. Just need to do some final cleanup.

from charts-clickhouse.

In order to not need to pass email via envs here we needed it to be in the app. Not sure if we should keep the email/pass env option in the chart - it would be nice to be able to put them there for folks who setup multiple instances via automation but having a password as raw text might not be what we want to be doing. Happy to leave this for @guidoiaquinti & @hazzadous to decide.

from charts-clickhouse.

@tiina303 it's great for reproducible environments, which is the reason I put in the fix. The fix I've put in is actually to make it such that we don't need to expose CI for deploy at least to these secrets, and rather provision a k8s secret separately

Having these passed as env vars to the app I think is ok, but maybe there's a better way?

from charts-clickhouse.

I also think we should keep the ability to configure email settings via ENV variables as it’s good for several use cases.

My main question is if:

Ability to set the email config in the PostHog app.

is something belonging to this repo or not? My take is that it doesn’t as the solution described in this ticket is a PostHog app feature. In that case, we should simply close this issue.

from charts-clickhouse.