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

Improve Service Node Time Synchronicity #1633

Open
KeeJef opened this issue Feb 20, 2023 · 4 comments
Open

Improve Service Node Time Synchronicity #1633

KeeJef opened this issue Feb 20, 2023 · 4 comments
Labels
enhancement New feature or request service nodes Relating to service nodes
Milestone

Comments

@KeeJef
Copy link
Collaborator

KeeJef commented Feb 20, 2023

Recently we've had some Service Nodes register with incorrectly synced clocks, this is an issue because it can cause users messages to fail to send if the Service Node thinks the users timestamp is out of bounds (because its own clock is out of bounds). These Service Nodes will be kicked from the network by other Service Nodes during testing, but they will bounce between decommissioned and registered until all uptime credits are used, exacerbating the time Session users can be affected.

Some options to reduce the prevalence of this issue

  • Warn during installation of Service Node package that local time may be out of sync and that this may result in deregistration
  • Require retesting clock before Service Node can go from Decommissioned -> Registered (prevents bouncing back and forth between decommissioned and registered)
@KeeJef KeeJef added enhancement New feature or request service nodes Relating to service nodes labels Feb 20, 2023
@KeeJef KeeJef added this to the Oxen11 milestone Feb 20, 2023
@javabudd
Copy link

javabudd commented Oct 8, 2024

@javabudd
Copy link

javabudd commented Oct 15, 2024

@KeeJef @jagerman Do you recommend we use NTP for this type of thing? If not, what is the preferred alternative?

@jagerman
Copy link
Member

These days I use the built-in systemd-timesync service; it's a lighter weight alternative than a full ntpd service for just keeping the clock synced.

But this depends quite a bit on the hardware and distribution in use: some (maybe most?) VPSes rely on time synchronization on the host system.

@javabudd
Copy link

javabudd commented Oct 15, 2024

Yea that's why I was thinking something like this would require us to reach out to a centralized time server to make sure we're always in sync. If we warn on package install it may be easier to get the appropriate dependencies installed, but if we want to do this in the code itself it might make sense to make requests to an ntp server(pool.ntp.org?).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request service nodes Relating to service nodes
Projects
None yet
Development

No branches or pull requests

3 participants