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

Add support to configure telemetry file and enable JMX port and hostname #26

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

GrigorievNick
Copy link

  • give ability to specify content of dremio-telemetry.yaml
  • add POD_ID to have ability enable jmx -Djava.rmi.server.hostname=$(POD_IP)
  • add all JMX necessary java options(-D)
  • add flag to enable disable this feautre

@GrigorievNick GrigorievNick force-pushed the helm_chart_v2_metrics_support branch 2 times, most recently from aa48ffb to eb8d08c Compare May 12, 2022 19:02
@GrigorievNick GrigorievNick force-pushed the helm_chart_v2_metrics_support branch from eb8d08c to 81abb1e Compare May 16, 2022 19:03
@treyhendon
Copy link

Hey, thanks for this! We're getting our Dremio Prod up and going in k8s and want to have metrics (and use this chart as-is). Any chance this will get resolved and merged?

@GrigorievNick
Copy link
Author

@treyhendon As far as I understand official dremio-tool does not support contribution.
You can check our fork.
https://github.com/AppsFlyer/dremio-cloud-tools

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants