Skip to content

Added validator NTP instructions#2542

Merged
anupsdf merged 2 commits into
stellar:mainfrom
SirTyson:ntp-sync-validators
Jul 6, 2026
Merged

Added validator NTP instructions#2542
anupsdf merged 2 commits into
stellar:mainfrom
SirTyson:ntp-sync-validators

Conversation

@SirTyson

@SirTyson SirTyson commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

Add instructions for validators to run NTP sync

Copilot AI review requested due to automatic review settings July 1, 2026 22:02
@SirTyson SirTyson requested a review from anupsdf July 1, 2026 22:03

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds validator-facing documentation that explicitly calls out NTP clock synchronization as a prerequisite for running Stellar Core, and links that guidance from Tier 1 onboarding checklists and environment preparation.

Changes:

  • Adds a new “Clock Synchronization” section to the validator prerequisites guide (with chrony install/verification steps).
  • Updates validator operational checklists to include enabling NTP clock synchronization.
  • Adds an environment-prep reminder to verify clock sync via NTP.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File Description
docs/validators/tier-1-orgs.mdx Adds an explicit checklist item linking to clock synchronization prerequisites.
docs/validators/admin-guide/prerequisites.mdx Introduces a clock synchronization section and notes outbound NTP requirements.
docs/validators/admin-guide/environment-preparation.mdx Adds a preparation bullet to ensure NTP-based clock synchronization is configured.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread docs/validators/admin-guide/prerequisites.mdx Outdated
Comment thread docs/validators/admin-guide/prerequisites.mdx Outdated
@SirTyson SirTyson force-pushed the ntp-sync-validators branch from 5c1adde to beed51f Compare July 1, 2026 22:07
@stellar-jenkins-ci

Copy link
Copy Markdown

@anupsdf anupsdf requested a review from jacekn July 1, 2026 22:14
@anupsdf

anupsdf commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

@jacekn , Added you to get your inputs on this PR. We need synchronized clocks for stellar-core features. Do you see any issue with requiring NTP server sync?

@stellar-jenkins-ci

Copy link
Copy Markdown

1 similar comment
@stellar-jenkins-ci

Copy link
Copy Markdown

@ElliotFriend

Copy link
Copy Markdown
Contributor

@SirTyson and @anupsdf I brought in the changes from main and resolved the conflicts (the ones i caused, sorry). let me know as soon as you're ready, and i can merge this. thanks!!

@stellar-jenkins-ci

Copy link
Copy Markdown

@anupsdf

anupsdf commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

@SirTyson and @anupsdf I brought in the changes from main and resolved the conflicts (the ones i caused, sorry). let me know as soon as you're ready, and i can merge this. thanks!!

I am okay if we merge this now. I was waiting to know @jacekn 's thoughts on this but we can make modifications later.

@anupsdf anupsdf merged commit ad388cc into stellar:main Jul 6, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants