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

Adds support for configuring chrony #575

Merged
merged 7 commits into from
Feb 20, 2025
Merged

Adds support for configuring chrony #575

merged 7 commits into from
Feb 20, 2025

Conversation

jovial
Copy link
Collaborator

@jovial jovial commented Feb 13, 2025

Chrony is already bundled in the generic cloud image so we just have to configure it. For this I am using an off the shelf role. I've chosen mrlesmithjr.chrony as it is also used in kayobe (so we can share development effort)

@jovial jovial requested a review from a team as a code owner February 13, 2025 10:09
@jovial jovial marked this pull request as draft February 13, 2025 10:12
@jovial
Copy link
Collaborator Author

jovial commented Feb 13, 2025

Need to add group to layouts and reference docs

@jovial jovial marked this pull request as ready for review February 14, 2025 09:56
Chrony is already bundled in the generic cloud image so we just have to
configure it. For this I am using an off the shelf role. I've chosen
mrlesmithjr.chrony as it is also used in kayobe (so we can share
development effort)
Copy link
Collaborator

@sjpb sjpb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Minor docs location tweak

@jovial jovial requested a review from sjpb February 20, 2025 10:48
Copy link
Collaborator

@sjpb sjpb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@sjpb sjpb merged commit 673ef13 into main Feb 20, 2025
1 of 2 checks passed
@sjpb sjpb deleted the feature/chrony branch February 20, 2025 11:35
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.

2 participants