-
-
Notifications
You must be signed in to change notification settings - Fork 59
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
Add leapsectz
option config option
#65
Conversation
@adrienthebo nice work, thanks for this, I don't mind future PR's from you!!! |
Thank you! |
@adrienthebo is this module something vox pupuli would want to take over? |
Looks like I've just described #64 |
Woah! Completely missed this comment. Yes, if you're open to that, I'm sure people would be interested. I can personally help with the migration too. |
https://voxpupuli.org/docs/migrate_module I'll preemptively send you a github invite, but also no worries if you've changed your mind and would prefer not to migrate your module. |
@alexjfisher no problem, I have joined by your invite. |
@aboe76 Have you had any more thoughts on this and are you ready to perform the migration? |
@alexjfisher yes I have, I would like to migrate the chrony module. I would like to ask you to assist with this. The reason for the migration is to improve the support and maintenance. |
Great! I've got some time free to do most of the migration process. All you really need to do is transfer the repository to the voxpupuli github organisation. Once we've made the first puppet/chrony release, you can use the puppet forge api to deprecate aboe76/chrony and suggest puppet/chrony as its replacement. |
I've created an issue with a checklist of stuff to do. Vox Pupuli can handle everything apart from the first and last steps. |
This pull request exposes the
leapsectz
option, which when set uses a specified offset from UTC to setCLOCK_TAI
.This current revision has tests but could probably benefit from additional work. In addition it looks like the
leapsecmode
option isn't plumbed through on archlinux and I haven't pushedleapsectz
through either. I'm happy to fix that in another PR or incorporate those fixes here.