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

Windows NTP client time sync failed #6480

Closed
DavidePrincipi opened this issue Apr 12, 2021 · 5 comments
Closed

Windows NTP client time sync failed #6480

DavidePrincipi opened this issue Apr 12, 2021 · 5 comments
Labels
bug A defect of the software verified All test cases were verified successfully

Comments

@DavidePrincipi
Copy link
Member

DavidePrincipi commented Apr 12, 2021

Windows clients cannot synchronize their clock with a new installation of nethserver-dc (the local AD accounts provider).

Steps to reproduce

  • Install a local AD account provider
  • Join a Windows machine and launch the w32tm /resync command

Expected behavior

The client clock gets synchronized.

Actual behavior

The sync command fails.

Components

nethserver-dc-1.8.1-1.ns7.x86_64


Thanks to @syntaxerrormmm @lucagasparini @nrauso

@DavidePrincipi DavidePrincipi added the bug A defect of the software label Apr 12, 2021
@DavidePrincipi DavidePrincipi self-assigned this Apr 12, 2021
@DavidePrincipi
Copy link
Member Author

Workaround

The issue origin seems to be in the DC installation procedure. The install procedure does not fix the filesystem group owner of ${nsroot}/var/lib/samba/ntp_signd/ to the ntp group.

[root@server ~]# nsdc-run -e -- ls -ld /var/lib/samba/ntp_signd
drwxr-x--- 2 root ntp 20 Apr  6 20:45 /var/lib/samba/ntp_signd

Instead the upgrade procedure fixes it with the following action:

/etc/e-smith/events/nethserver-dc-update/S40nethserver-dc-fixchroot

Run it manually to fix the bug.

@nethbot
Copy link
Member

nethbot commented Apr 12, 2021

in 7.9.2009/testing:

  • nethserver-dc-1.8.1-1.1.g4cf652f.ns7.x86_64.rpm x86_64

@DavidePrincipi
Copy link
Member Author

Test case 1

Check the new package correctly fixes an existing installation:

nsdc-run -e -- ls -ld /var/lib/samba/ntp_signd

Ownership must change from root:root to root:ntp

Test case 2

Check the bug is not reproducible in a new installation.

@DavidePrincipi DavidePrincipi removed their assignment Apr 12, 2021
@DavidePrincipi DavidePrincipi added the testing Packages are available from testing repositories label Apr 12, 2021
@nrauso nrauso self-assigned this Apr 12, 2021
@nrauso
Copy link

nrauso commented Apr 12, 2021

Tested: VERIFIED

The package in testing fixes ownerships on both updated and new DC installations.

@nrauso nrauso removed their assignment Apr 12, 2021
@nrauso nrauso added verified All test cases were verified successfully and removed testing Packages are available from testing repositories labels Apr 12, 2021
@nethbot
Copy link
Member

nethbot commented Apr 13, 2021

in 7.9.2009/updates:

  • nethserver-dc-1.8.2-1.ns7.x86_64.rpm x86_64

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug A defect of the software verified All test cases were verified successfully
Projects
None yet
Development

No branches or pull requests

3 participants