-
Notifications
You must be signed in to change notification settings - Fork 63
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
[mainnet-manager] Manager Node backend additions. #226
Comments
@evanlinjin note that uptime only requires the time elapsed since the last connection to the discovery (ap4y suggested implementing uptime tracking in transport discvory). The total uptime is calculated by the monitor, although we could improve on that. Could you elaborate on why you want to move the RPC implementation back to /pkg/node? |
For the first point, it's the transport discovery right? I think we need to call it For the second point, it makes more sense to have it there and leads to more confusion. |
small correction, I actually backtracked suggestion for using transport discovery for uptime, see issue 110 in the original repo. My idea was to use transports to track uptime, since they have statuses but as I mentioned in that issue it's not going to work. Having separate service makes the most for me but I think it was rejected as an idea. |
Description
This is a parent issue of multiple sub-issues/sub-PRs. This is provided here for a clear overview of tasks.
From @jdknives on Discord:
Tasks
/pkg/node
module.The text was updated successfully, but these errors were encountered: