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

Better link status #2777

Merged
merged 5 commits into from
Sep 10, 2021
Merged

Conversation

Fabien-B
Copy link
Contributor

@Fabien-B Fabien-B commented Sep 8, 2021

Improve TELEMETRY_STATUS when multiple (redundant) links are used.
Previously, there was no way to tell if one of the links was down.
Now, it should behave as expected:

  • if one of the link is down, it goes red in the Link tab. It goes orange in the strip (with correct number of links up).
  • if link combiner is down, all links are shown as down (link may still be up, but no message are received), and the indicator in the strip goes red.
  • if all links are lost, they all goes red, and the indicator in the strip goes red.

@Fabien-B
Copy link
Contributor Author

Fabien-B commented Sep 8, 2021

It should be ok now.

@gautierhattenberger gautierhattenberger merged commit 7715557 into paparazzi:master Sep 10, 2021
@gautierhattenberger gautierhattenberger deleted the better_link_status branch September 10, 2021 09:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants