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

replace chrony with timesyncd #369

Merged
merged 3 commits into from
Sep 16, 2024
Merged

Conversation

andiradulescu
Copy link
Collaborator

@andiradulescu andiradulescu commented Sep 14, 2024

Resolves #318

  • removes chrony and replaces it with systemd-timesyncd
  • makes time syncing faster than chrony and when network configuration or connectivity changes
  • installed build and tested both with internet and without internet on boot - time is synced almost instantly after network connectivity is up
$ journalctl -u systemd-timesyncd
Aug 08 14:51:13 comma systemd[1]: Starting systemd-timesyncd.service - Network Time Synchronization...
Aug 08 14:51:13 comma (imesyncd)[2819]: systemd-timesyncd.service: ProtectHostname=yes is configured, but the kernel does not support UTS na>
Aug 08 14:51:13 comma systemd[1]: Started systemd-timesyncd.service - Network Time Synchronization.
Aug 08 14:51:13 comma systemd-timesyncd[2819]: No network connectivity, watching for changes.
Aug 08 14:54:47 comma-cb80a5fa systemd-timesyncd[2819]: Network configuration changed, trying to establish connection.
Sep 15 06:40:29 comma-cb80a5fa systemd-timesyncd[2819]: Contacted time server 185.125.190.57:123 (ntp.ubuntu.com).
Sep 15 06:40:29 comma-cb80a5fa systemd-timesyncd[2819]: Initial clock synchronization to Sun 2024-09-15 06:40:29.697339 UTC.

Copy link

✅ AGNOS update 11-d378a36 ready

In an openpilot branch, download agnos.json in system/hardware/tici/ and update AGNOS_VERSION to "11-d378a36" in launch_env.sh.

For flashing locally, download and unarchive the images in agnos-builder/output and flash with ./flash_all.sh.

Images:

Stats:

3.7G	/
621M	/usr/local/venv
836M	/usr/lib/aarch64-linux-gnu
338M	/usr/lib/llvm-18
155M	/usr/lib/arm-linux-gnueabihf
321M	/usr/share
283M	/usr/bin
166M	/usr/comma

@andiradulescu andiradulescu marked this pull request as ready for review September 15, 2024 07:22
@adeebshihadeh adeebshihadeh merged commit a45d1a1 into commaai:master Sep 16, 2024
2 checks passed
@adeebshihadeh
Copy link
Contributor

Once we get to #315, I can see all sorts of checks being add if the script is written generically enough. Then, we can easily test the reliability of these things in a loop.

@andiradulescu andiradulescu deleted the ntp-sync branch September 16, 2024 17:53
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.

NTP sync sometimes not instant
2 participants