Skip to content

fix: remove redundant chronyd - #957

Merged
K900 merged 2 commits into
nix-community:mainfrom
ljxfstorm:f/remove-chronyd
Jan 14, 2026
Merged

fix: remove redundant chronyd#957
K900 merged 2 commits into
nix-community:mainfrom
ljxfstorm:f/remove-chronyd

Conversation

@ljxfstorm

Copy link
Copy Markdown
Contributor

Resolve #956

Remove the services.chrony section as chronyd is already running in the WSL2 root namespace.

@K900

K900 commented Jan 14, 2026

Copy link
Copy Markdown
Contributor

Hm, for some reason I thought this was supposed to be running in the distro namespace. We do need to set services.timesyncd.enable = false; then, because otherwise that will end up fighting root chrony instead.

@ljxfstorm

Copy link
Copy Markdown
Contributor Author

Hm, for some reason I thought this was supposed to be running in the distro namespace. We do need to set services.timesyncd.enable = false; then, because otherwise that will end up fighting root chrony instead.

services.timesyncd.enable defaults to !config.boot.isContainer, which evaluates to false in WSL.

@K900

K900 commented Jan 14, 2026

Copy link
Copy Markdown
Contributor

No, we do not set boot.isContainer.

@ljxfstorm

Copy link
Copy Markdown
Contributor Author

No, we do not set boot.isContainer.

You're right. I had it set in my own config, so I missed that.
I've pushed a new commit to disable timesyncd. Let me know if I should squash the commits.

@K900
K900 enabled auto-merge January 14, 2026 15:14
auto-merge was automatically disabled January 14, 2026 15:21

Head branch was pushed to by a user without write access

@K900
K900 enabled auto-merge January 14, 2026 15:21
@ljxfstorm

Copy link
Copy Markdown
Contributor Author

Oops, I think I've inserted the disabling part in the wrong place.
Please review it again.

@K900
K900 merged commit d0d3636 into nix-community:main Jan 14, 2026
24 checks passed
@ljxfstorm
ljxfstorm deleted the f/remove-chronyd branch January 14, 2026 15:32
@nzbr nzbr changed the title Remove redundant chronyd fix: remove redundant chronyd Mar 1, 2026
@nzbr nzbr added bug Something isn't working backport release-25.11 and removed backport release-25.11 labels Mar 1, 2026
@github-actions

github-actions Bot commented Mar 1, 2026

Copy link
Copy Markdown
Contributor

Successfully created backport PR for release-25.11:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport release-25.11 bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

chronyd keeps warning about "System clock interference"

3 participants