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

Set cloud role name using Java system property #1684

Closed
paveljakov opened this issue May 14, 2021 · 3 comments · Fixed by #1813
Closed

Set cloud role name using Java system property #1684

paveljakov opened this issue May 14, 2021 · 3 comments · Fixed by #1813

Comments

@paveljakov
Copy link

Is your feature request related to a problem? Please describe.
Currently we use Application Insights e2e transaction logging on all our environments, even for local development. Now we have an issue (annoyance) that we have to provide cloud role name of application either using Environment Variable (does not work when you run multiple applications locally) or using JSON configuration file. It would be nice if it would be also possible to set application name (cloud role name) using Java system property, just like we can for configuration file path.

Describe the solution you would like
Ability to set cloud role name using Java system property, i.e. applicationinsights.role.name.

@trask
Copy link
Member

trask commented Jun 14, 2021

does not work when you run multiple applications locally

hey @paveljakov! are you able to set the environment variable in the startup script for each java process separately?

@paveljakov
Copy link
Author

Hi @trask,

As mentioned in #1685, we are pushing all developers to use AI before creating PRs to analyze how their changes impact the system overall and I would like to make their life as simple as possible. Some people open multiple services at the same time, some open single one each time. Asking to configure Environment Variables for each run configuration is not ideal and can lead to misconfigurations. Having ability to provide Cloud Role Name using Java system property would be simpler.

@trask
Copy link
Member

trask commented Aug 4, 2021

this is supported now in 3.2.0-BETA.2

@ghost ghost locked as resolved and limited conversation to collaborators Sep 4, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants