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

Request: read pinpoint config from JVM System properties or environment variables #2904

Open
joshwand opened this issue May 5, 2017 · 3 comments
Labels

Comments

@joshwand
Copy link

joshwand commented May 5, 2017

While working out config for my application (deployed using docker to AWS ECS), having to modify the pinpoint.config file and redeploy the image is very cumbersome. It'd be great to be able to set the various pinpoint config properties via environment variables or java System properties, e.g.:

java -Dpinpoint.profiler.include=com.mycompany.* -jar myapp.jar.

or, Spring-Boot-style:

PINPOINT_PROFILER_INCLUDE=com.mycompany.* java -jar myapp.jar

Would make tweaking configs in a cloud environment much more developer-friendly.

(incidentally there are no good docs on best practices for setting up how much to include in profiler.include.. I keep ending up with too much or too little)

Thanks!

@Xylus Xylus added the proposal label May 6, 2017
@Xylus
Copy link
Contributor

Xylus commented May 6, 2017

Hi @joshwand
This has been in our todo list, but kept being pushed back.
Might be a good idea to look into it.
Thanks for the suggestion.

(pinging #2895)

@emeroad
Copy link
Member

emeroad commented May 8, 2017

We recommend -Dpinpoint.config=$PINPOINT_CONFIG_PATH/custom_pinpoint.config

@jiaqifeng
Copy link
Contributor

I modified the code to get the config from our cloud config repository. This is really a simple fix. The hardest part is there are no single solution for all environment.

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

4 participants