Skip to content
This repository has been archived by the owner on May 6, 2020. It is now read-only.

Fix nanosecond logging test #1017

Closed
jodh-intel opened this issue Feb 21, 2018 · 0 comments
Closed

Fix nanosecond logging test #1017

jodh-intel opened this issue Feb 21, 2018 · 0 comments
Assignees
@jodh-intel

Comments

@jodh-intel
Copy link
Contributor

TestNewSystemLogHook() checks to ensure a nanonsecond timestamp is generated. However, time.RFC3339Nano truncates trailing zeros so the current test is incorrect.
@jodh-intel jodh-intel self-assigned this Feb 21, 2018
jodh-intel added a commit to jodh-intel/runtime that referenced this issue Feb 21, 2018
@jodh-intel
logger: Fix nanosecond timestamp test
702ee56 
`TestNewSystemLogHook()` checks to ensure a nanonsecond timestamp is
generated. However, the `time.RFC3339Nano` time format truncates
trailing zeros meaning a nanosecond timestamp won't always have nine
digits.

Fixes clearcontainers#1017.

Signed-off-by: James O. D. Hunt <[email protected]>
@jodh-intel jodh-intel mentioned this issue Feb 21, 2018
Merged
@erick0z erick0z mentioned this issue Feb 21, 2018
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@jodh-intel jodh-intel

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@jodh-intel