Skip to content

New integration for Chrony#2943

Open
naoyukiseki wants to merge 11 commits into
DataDog:masterfrom
naoyukiseki:naoyukiseki/chrony
Open

New integration for Chrony#2943
naoyukiseki wants to merge 11 commits into
DataDog:masterfrom
naoyukiseki:naoyukiseki/chrony

Conversation

@naoyukiseki
Copy link
Copy Markdown

What does this PR do?

This PR adds a new integration for chrony.

Motivation

Chrony is a widely used time‑synchronization tool. Having a Datadog integration for it would be very useful.

Review checklist

  • PR has a meaningful title or PR has the no-changelog label attached
  • Feature or bugfix has tests
  • Git history is clean
  • If PR impacts documentation, docs team has been notified or an issue has been opened on the documentation repo
  • If this PR includes a log pipeline, please add a description describing the remappers and processors.

Additional Notes

Anything else we should know when reviewing?

@naoyukiseki naoyukiseki requested review from a team as code owners March 26, 2026 07:35
@naoyukiseki naoyukiseki marked this pull request as draft March 26, 2026 10:27
@naoyukiseki
Copy link
Copy Markdown
Author

The validate-owner-validator check was failed. But I am a Datadog employee, I believe it can be ignored.
https://github.com/DataDog/integrations-extras/pull/2943/checks?check_run_id=68693094502
Owner is a required internal-only field. Use a valid internal team handle. External contributors should leave this unset.

If I unset the CODEOWNERS, the gh-worker-mergegate-4580a3 / devflow/mergegate was failed.
https://github.com/DataDog/integrations-extras/pull/2943/checks?check_run_id=68691500644

@naoyukiseki naoyukiseki marked this pull request as ready for review March 26, 2026 11:01
@maycmlee
Copy link
Copy Markdown
Contributor

Created DOCS-13854 for review.

@maycmlee maycmlee added the editorial review Waiting on a more in-depth review from a docs team editor label Mar 26, 2026
@github-actions
Copy link
Copy Markdown

This pull request has not been updated for more than 21 days. If there are no updates to this PR within 7 days, it will be closed. If you'd like to re-open this PR after it's been closed, you can start from the latest master branch or pull the latest changes into your branch and create a new pull request.

@github-actions github-actions Bot added the stale label Apr 16, 2026
@github-actions
Copy link
Copy Markdown

This pull request was not updated after an additional 7 days of no activity. If you would like to continue work on this PR, please re-open this PR or create a fresh branch off of the latest master branch.

@github-actions github-actions Bot closed this Apr 24, 2026
@naoyukiseki naoyukiseki reopened this Apr 25, 2026
@naoyukiseki
Copy link
Copy Markdown
Author

Reopened and update the base branch.
Thanks!

@github-actions github-actions Bot removed the stale label Apr 25, 2026
Comment thread chrony/README.md

To install the chrony check for development testing:

1. Install the [developer toolkit]
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Can you make this into a numbered link like the other links in this doc? Numbered links are required for the translation process to work.

Comment thread chrony/README.md
- **`chrony.rootdelay`**: Root delay (seconds)
- **`chrony.rootdispersion`**: Root dispersion (seconds)

## Service Checks
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
## Service Checks
### Service Checks

Comment thread chrony/README.md
Comment on lines +52 to +59
- **`chrony.stratum`**: NTP stratum level (unitless)
- **`chrony.systime`**: System time offset from NTP time (seconds)
- **`chrony.frequency`**: Frequency offset (unitless, ppm)
- **`chrony.residualfreq`**: Residual frequency (unitless, ppm)
- **`chrony.skew`**: Skew (unitless, ppm)
- **`chrony.rootdelay`**: Root delay (seconds)
- **`chrony.rootdispersion`**: Root dispersion (seconds)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
- **`chrony.stratum`**: NTP stratum level (unitless)
- **`chrony.systime`**: System time offset from NTP time (seconds)
- **`chrony.frequency`**: Frequency offset (unitless, ppm)
- **`chrony.residualfreq`**: Residual frequency (unitless, ppm)
- **`chrony.skew`**: Skew (unitless, ppm)
- **`chrony.rootdelay`**: Root delay (seconds)
- **`chrony.rootdispersion`**: Root dispersion (seconds)

The integration build process automatically inserts all of the metrics from your metadata.csv file into the docs under this heading. You don't need to include them.

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

Labels

editorial review Waiting on a more in-depth review from a docs team editor

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants