Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Update default config. #1003

Closed
MariusBrill opened this issue Feb 16, 2021 · 0 comments · Fixed by #1004
Closed

Update default config. #1003

MariusBrill opened this issue Feb 16, 2021 · 0 comments · Fixed by #1004
Assignees
@MariusBrill
Labels
area/eum-server

Comments

@MariusBrill
Copy link
Member

MariusBrill commented Feb 16, 2021 •
edited
Loading

exporters.metrics.influx: url property should be added but commented out.

exporters.tracing.jaeger: grpc property should be added but commented out.

Add new lines between the properties to increase readbility.

tracing.jaeger.url is deprecated and should be marked as @Depcrecated in the respective settings.

Add logs to TraceExportersConfiguration.
@MariusBrill MariusBrill self-assigned this Feb 16, 2021
@MariusBrill MariusBrill linked a pull request Feb 16, 2021 that will close this issue
Closes #1003 - Updated default config. #1004
Merged
@MariusBrill MariusBrill changed the title Influx Exporter does not create Database when create-database ist true. InfluxDB URL and Jaeger URL is missing in Default Config. Feb 16, 2021
@MariusBrill MariusBrill changed the title InfluxDB URL and Jaeger URL is missing in Default Config. InfluxDB URL and Jaeger URL are missing in Default Config. Feb 16, 2021
@MariusBrill MariusBrill changed the title InfluxDB URL and Jaeger URL are missing in Default Config. Update default config. Feb 16, 2021
Patrick-Eichhorn pushed a commit that referenced this issue Feb 17, 2021
@MariusBrill
Closes #1003 - Updated default config. (#1004)
b6a47ed
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees

@MariusBrill MariusBrill

Labels
area/eum-server
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Closes #1003 - Updated default config. MariusBrill/inspectit-ocelot
1 participant
@MariusBrill