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

Telemetry should be working even when the api server is disabled. #12447

Closed
4 tasks
robert-zaremba opened this issue Jul 5, 2022 · 0 comments · Fixed by #12448
Closed
4 tasks

Telemetry should be working even when the api server is disabled. #12447

robert-zaremba opened this issue Jul 5, 2022 · 0 comments · Fixed by #12448
Assignees
Labels
C: telemetry Issues and features pertaining to SDK telemetry. T:Bug

Comments

@robert-zaremba
Copy link
Collaborator

robert-zaremba commented Jul 5, 2022

Summary of Bug

Telemetry sinks are only initialized when both api.enabled = true and telemetry.enabled = true and only in startInProcess mode (with tendermint in process). See the flow:

Expected: Telemetry should be correctly initialized when telemetry.enabled = true even when api.enabled = false

Version

3283617, v0.46-rc1

Steps to Reproduce

Start an app with telemetry enabled and api disabled. Add some logs to check that telemetry is not enabled.


For Admin Use

  • Not duplicate issue
  • Appropriate labels applied
  • Appropriate contributors tagged
  • Contributor assigned/self-assigned
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C: telemetry Issues and features pertaining to SDK telemetry. T:Bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant