Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

ipfix-elements: fix NTP_USEC_TO_FRAC macro #241

Merged
merged 1 commit into from
Dec 9, 2024

Conversation

TheSableCZ
Copy link
Contributor

Use standard conversion to ntp timestamps as in https://tickelton.gitlab.io/articles/ntp-timestamps/.

@TheSableCZ TheSableCZ requested a review from SiskaPavel December 9, 2024 09:15
@codecov-commenter
Copy link

codecov-commenter commented Dec 9, 2024

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 40.61%. Comparing base (6532d55) to head (f1877cb).
Report is 2 commits behind head on master.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #241   +/-   ##
=======================================
  Coverage   40.61%   40.61%           
=======================================
  Files         104      104           
  Lines       10120    10120           
  Branches     1492     1492           
=======================================
  Hits         4110     4110           
  Misses       5145     5145           
  Partials      865      865           
Flag Coverage Δ
tests 40.61% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@SiskaPavel SiskaPavel merged commit 7ee2138 into master Dec 9, 2024
5 checks passed
@SiskaPavel SiskaPavel deleted the fix-ipfix-ntp-conversion branch December 9, 2024 13:11
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants