Skip to content

Sync time on raspberry pi before running setup script #3618

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.

Sign up for GitHub

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

Merged
StarrryNight merged 12 commits into from
Feb 28, 2026
Merged

Sync time on raspberry pi before running setup script #3618

Show file tree
Hide file tree
Changes from 4 commits
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
77ea2dc
create enable ntp block
Feb 20, 2026
55a3fb8
add to set_Up pi
Feb 20, 2026
9888d40
add sudo
StarrryNight Feb 21, 2026
b40cbc7
[pre-commit.ci lite] apply automatic fixes
pre-commit-ci-lite[bot] Feb 21, 2026
220988f
Fix capitalization in NTP task name
StarrryNight Feb 21, 2026
dcec07c
fix bugs, will test in mez later
StarrryNight Feb 24, 2026
dc61092
change tag
StarrryNight Feb 24, 2026
db6079f
temp commit
Thunderbots Feb 24, 2026
60c4f5a
add check time block
Thunderbots Feb 24, 2026
6de51be
Add changed when lines so we don't violate idempotency
StarrryNight Feb 24, 2026
c035f46
use become and ansible.builtin.sudo instead of sudo in the command
Thunderbots Feb 26, 2026
bcf9d26
Merge branch 'sync_time#3615' of https://github.com/StarrryNight/Thun…
Thunderbots Feb 26, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions src/software/embedded/ansible/playbooks/setup_pi.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,11 @@
ansible.builtin.import_tasks:
file: ../tasks/check_internet.yml

- name: Time sync by enabling ntp
Comment thread
annieisawesome2 marked this conversation as resolved.
Outdated
tags: configure_pi
Comment thread
suchirss marked this conversation as resolved.
Outdated
ansible.builtin.import_tasks:
file: ../tasks/enable_ntp.yml

- name: Enable passwordless sudo for rsync
tags:
- dependencies
Expand Down
12 changes: 12 additions & 0 deletions src/software/embedded/ansible/tasks/enable_ntp.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,12 @@
---
- name: Make sure NTP is enabled to sync time for set_up.yml
Comment thread
suchirss marked this conversation as resolved.
Outdated
block:
- name: Enable automatic time synchronization
ansible.builtin.command: sudo timedatectl set-ntp on
changed_when: true
Comment thread
suchirss marked this conversation as resolved.

- name: Ensure the systemd-timesyncd service is running and enabled
Comment thread
suchirss marked this conversation as resolved.
ansible.builtin.systemd_service:
name: systemd-timesyncd
state: started
enabled: true
Loading