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

Added CHRONY_EXTRA option for additional configuration #60

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

WattsUp
Copy link

@WattsUp WattsUp commented Feb 27, 2023

This feature allows special chrony configurations to be added.

Such as having a PTP reference clock and using docker-ntp to serve to ntp clients

@Mimikoo
Copy link

Mimikoo commented Jul 10, 2023

Voting for extra configurations.

@cturra
Copy link
Owner

cturra commented Nov 2, 2023

thanks for your patience with my review here. while i understand there may be other chronyd configuration options that can be passed to this container, i would like to keep those purposeful. it feels like a catch-all environment variable is a bit of a shortcut.

using your example, if you're looking to add the ability to pass a device to the container, let's get a specific option for that. this way we can also ensure that during startup, any relevant checks, like validating permissions on the device(s).

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

Successfully merging this pull request may close these issues.

3 participants