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

separate survey collection from transport bandwidth logging collection #1657

Open
0pcom opened this issue Sep 7, 2023 · 1 comment
Open
Labels
bug Something isn't working telemetrics transport bandwidth & survey collection

Comments

@0pcom
Copy link
Collaborator

0pcom commented Sep 7, 2023

Currently the survey and transport bandwidth logs are collected from only visors which met 75% uptime for the previous day.

This is ok for surveys, however we need to collect transport bandwidth logging for all visors, regardless of their having made uptime.

The transport bandwidth logging should be collected from any visor which is currently online and should be preceded by a check of the /health endpoint of the dmsghttp log server.

We have a framework in place already for skywire-cli log which can do this.

we only need to implement this change on the reward system server.

@0pcom 0pcom added the bug Something isn't working label Sep 7, 2023
@0pcom 0pcom added the telemetrics transport bandwidth & survey collection label Mar 23, 2024
@0pcom
Copy link
Collaborator Author

0pcom commented May 17, 2024

What we really need for this is an endpoint on the transport discovery to show transports which have existed in the past day. Then the transport bandwidth logging may be collected in a more targeted way from the specific visor public keys which actually had transports over the past day.

However, we actually should not do this.

Instead, what we should do is make the visor report the transport bandwidth logging to the transport discovery server periodically and on closure of the transport. Then, the reward system can fetch the logging from there and display it, obviating the need for it to be collected directly.

see issue #1758

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working telemetrics transport bandwidth & survey collection
Projects
None yet
Development

No branches or pull requests

1 participant