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.

Sign up for GitHub

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

Jump to bottom

remove "no services to aggregate for telemetry" warning #1641

Closed
larkin opened this issue Jul 5, 2017 · 5 comments
Closed

remove "no services to aggregate for telemetry" warning #1641

larkin opened this issue Jul 5, 2017 · 5 comments
Labels
help_wanted Help the Open MCT project!

Comments

@larkin
Copy link
Contributor

larkin commented Jul 5, 2017

With the new Telemetry API, warnings are thrown in the console because there are no services to aggregate for telemetry. We should disable this warning so that it isn't a false flag for developers.

reported here: nasa/openmct-tutorial#15

Also reported in other places.
@larkin
Copy link
Contributor Author

larkin commented Jul 5, 2017

Also remove the "Telemetry service unavailable" warning.

@larkin larkin added the help_wanted Help the Open MCT project! label Jul 5, 2017
@larkin
Copy link
Contributor Author

larkin commented Jul 5, 2017

To reproduce: run the openmct tutorial and view a telemetry point. You'll see these warnings in console.

@larkin larkin mentioned this issue Jul 5, 2017
Closed
@luisschubert
Copy link
Contributor

@larkin were you thinking to just comment out/ delete the warnings entirely?

Telemetry Service Unavailable:
platform/telemetry/src/TelemetryCapability.js Line 120

No services to aggregate for telemetry:
platform/framework/src/register/ServiceCompositor.js Line 169 - 175

@larkin
Copy link
Contributor Author

larkin commented Jul 6, 2017

@luisschubert remove, or change their severity to info, which should be below our default log level.

@luisschubert luisschubert mentioned this issue Jul 6, 2017
Merged
@larkin
Copy link
Contributor Author

larkin commented Jul 27, 2017

Closed in #1643.

@larkin larkin closed this as completed Jul 27, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
help_wanted Help the Open MCT project!
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@larkin @luisschubert