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

Restore state from cache on startup when server is unreachable #316

Open
goodhoko opened this issue Jul 5, 2024 · 2 comments
Open

Restore state from cache on startup when server is unreachable #316

goodhoko opened this issue Jul 5, 2024 · 2 comments
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed

Comments

@goodhoko
Copy link
Member

goodhoko commented Jul 5, 2024

Innernet client is unable to setup the network interface when it can't reach innernet server on startup. This is a single-point of failure even when other peers are perfectly reachable.

At the same time innernet already caches peer information fetched from the server in /var/lib. We could use this cache on startup when we fail to read state from server.

cc @skywhale @strohel @bschwind

@goodhoko goodhoko added the enhancement New feature or request label Jul 5, 2024
@goodhoko goodhoko changed the title Restore from cache on startup when server is unreachable Restore state from cache on startup when server is unreachable Jul 5, 2024
@bschwind
Copy link
Member

bschwind commented Jul 5, 2024

Related to #296

@strohel strohel added good first issue Good for newcomers help wanted Extra attention is needed labels Jul 8, 2024
@strohel
Copy link
Member

strohel commented Jul 8, 2024

FWIW this has an interaction with the listen-port being fixed vs. random. If it is fixed, we have higher change of connecting to peers in case of unreachanble server, because peers with the previous (last published) endpoint will still have a chance to find us.

Nothing really actionable, there already is functionality to freeze the listen port, so mentioning just for the record.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

3 participants