patch(2.0): Nil check DPU extension timestamp error logging (#3744)#4119
patch(2.0): Nil check DPU extension timestamp error logging (#3744)#4119hwadekar-nv wants to merge 2 commits into
Conversation
…A#3744) Description: - The timestamp parse branch was inverted, causing successful parses to log errors while failures were silent. - The fix logs non-empty parse failures and preserves the existing fallback timestamp. - Added coverage for valid and invalid timestamps, then ran the full dpuextensionservice package tests successfully. Type of Change - [x] **Fix** - Bug fixes Testing - [x] Unit tests added/updated
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Enterprise Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
🔐 TruffleHog Secret Scan✅ No secrets or credentials found! Your code has been scanned for 700+ types of secrets and credentials. All clear! 🎉 🕐 Last updated: 2026-07-24 17:46:23 UTC | Commit: dfedd6e |
🔍 Container Scan Summary
Per-CVE detail lives in the per-service |
Backports #3744 to release/v2.0.
Description:
Type of Change
Testing