Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SMTP configuration #443

Open
pascal89 opened this issue May 30, 2024 · 4 comments
Open

SMTP configuration #443

pascal89 opened this issue May 30, 2024 · 4 comments
Assignees
Labels
chart:signoz Issue related to signoz helm chart enhancement New feature or request good first issue Good for newcomers

Comments

@pascal89
Copy link

Hi,
Currently, the Signoz Helm chart doesn't support providing the SMTP configuration explicitly. By looking into the Helm chart I already found out, I'll be able to configure this via the queryService.additionalEnvs chart variable, but that way I'll unfortunately have to provide my SMTP password in an unsecure way.

My suggestion is to update the Helm chart to be able to provide the SMTP configuration through Helm variables, where for the SMTP password we'll be able to provide this through an existing secret.

As I have experience with maintaining Helm charts, and I'm also willing to make a contribution for this. Please let me hear your thought on this.

@pascal89
Copy link
Author

Hi @prashant-shahi, do you have any comments on this? Thanks in advance :-)

@grandwizard28
Copy link
Contributor

Hi @pascal89,
That sound nice! Would you be willing to open a PR?

@grandwizard28 grandwizard28 added chart:signoz Issue related to signoz helm chart enhancement New feature or request good first issue Good for newcomers labels Sep 19, 2024
@pascal89
Copy link
Author

Sure, I'll add it to my planning. I'll probably be able to pick it up in a few weeks from now.

@prashant-shahi
Copy link
Member

@pascal89 That would be great. Looking forward for your contribution.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chart:signoz Issue related to signoz helm chart enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

3 participants