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

Usage Problem? #93

Closed
xuanyuanaosheng opened this issue Apr 6, 2021 · 3 comments
Closed

Usage Problem? #93

xuanyuanaosheng opened this issue Apr 6, 2021 · 3 comments
Labels

Comments

@xuanyuanaosheng
Copy link

xuanyuanaosheng commented Apr 6, 2021

  1. Could you please update the helm repo to v3 ?
  2. Could you please provide a grafana dashboard json ?
  3. Which version support the config reload ?
@xuanyuanaosheng xuanyuanaosheng changed the title Could you please update the helm repo to v3? Usage Problem? Apr 7, 2021
@braedon
Copy link
Owner

braedon commented Apr 10, 2021

Hi @xuanyuanaosheng, I'll answer your questions in order:

  1. As far as I'm aware, the existing helm repo works fine with helm v3 (that's what I'm running myself). Have you encountered a problem that requires updating it to v3 to work?

  2. I haven't developed a general purpose Grafana dashboard, and don't have any plans to currently. If you'd like to contribute one I'd consider adding it as an example in the repo, or linking to it from the README. Please raise a separate issue or PR if so.

  3. There's no support currently for reloading the config without restarting the process/container, and I don't have any plans to add it any time soon. I'm open to being convinced otherwise, however - see the existing issue on the topic if you'd like to argue for it. That issue also links to a workaround that could be useful to you if you're using k8s.

@xuanyuanaosheng
Copy link
Author

@braedon Thanks for your reply, I have test it, using the helm v3, it works fine.

@braedon
Copy link
Owner

braedon commented Apr 28, 2021

Great, I'll leave the repo as-is for now since it works with helm v3. I'll probably upgrade it next time I have to change the chart.

@braedon braedon closed this as completed Apr 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants