Skip to content

Fix fractional seconds in RFC3339 timestamps - #25497

Open
fallintoplace wants to merge 2 commits into
DataDog:masterfrom
fallintoplace:fix/rfc3339-fractional-seconds
Open

fallintoplace wants to merge 2 commits into
DataDog:masterfrom
fallintoplace:fix/rfc3339-fractional-seconds

Conversation

@fallintoplace

@fallintoplace fallintoplace commented Oct 4, 2026 •

Copy link
Copy Markdown

What does this PR do?

  • What: Fix fractional seconds in parse_rfc3339.
  • Implementation: Pad short fractions to six digits. Truncate longer fractions.

Motivation

  • Why: Valid timestamps like 2019-02-18T16:00:06.123Z raise ValueError. This can stop kubelet pod expiry filtering.

Review checklist (to be filled by reviewers)

  • QA decision: qa/required.

@dd-octo-sts

dd-octo-sts Bot commented Oct 4, 2026 •

Copy link
Copy Markdown
Contributor

❌ Dispatcher tests: failed

Dispatcher beta: informational only
Existing CI remains the merge signal.

Caution

2 integrations failed. 4 of 861 jobs failed. Dispatcher could not collect test results for 2 targets.

  861/861 jobs

✅ 857 passed · ❌ 4 failed

Batches · ✅ batch-01 240/240 · ✅ batch-02 219/219 · ❌ batch-03 224/224 · ❌ batch-04 178/178

❌ sap_hana: 2 failed targets
❌ weblogic: 2 failed targets

Dispatcher finished on 710a7f8 — GitHub Run · Dispatcher Logs.

@datadog-datadog-prod-us1

Copy link
Copy Markdown
Contributor

Code Coverage

🎯 Code Coverage (details)
• Patch Coverage: 100.00%
• Overall Coverage: 89.35% (-0.05%)

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 1b02171 | Docs | View more details | Give us feedback!

This branch has not been deployed

No deployments
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.

1 participant