Skip to content
This repository has been archived by the owner on Aug 30, 2022. It is now read-only.

Added support for setting tracer tags via the configuration #284

Merged
merged 2 commits into from
Sep 13, 2021
Merged

Added support for setting tracer tags via the configuration #284

merged 2 commits into from
Sep 13, 2021

Conversation

tobiasstadler
Copy link
Contributor

Which problem is this PR solving?

At the moment setting the tracer tags is only possible via an environment variable. It should also be possible to set them via the configuration (file).

Short description of the changes

  • I enhanced the parse the parse method to read a new configuration key for the tracer tags

@tobiasstadler
Copy link
Contributor Author

@yurishkuro would mind doing a review please?

@AppVeyorBot
Copy link

@yurishkuro yurishkuro enabled auto-merge (squash) September 4, 2021 14:38
@AppVeyorBot
Copy link

@tobiasstadler
Copy link
Contributor Author

@yurishkuro The build failure in macOS seems to be unrelated to my change.

@tobiasstadler
Copy link
Contributor Author

@yurishkuro Could you rerun the build please

@yurishkuro
Copy link
Member

restarted

@yurishkuro yurishkuro merged commit 703028f into jaegertracing:master Sep 13, 2021
@tobiasstadler
Copy link
Contributor Author

Thank you! Would you be so kind and do a release please?

@yurishkuro
Copy link
Member

Released in 0.8.0

@tobiasstadler
Copy link
Contributor Author

Thank You!

@miry
Copy link

miry commented Sep 14, 2021

@tobiasstadler Is it working only with Yaml config?

Nevermind. Tested and it works. global_tag

image

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

Successfully merging this pull request may close these issues.

4 participants