Skip to content

lib: include extraction time in Crash Log METADATA - #105

Merged
psurply merged 1 commit into
intel:mainfrom
jlrivasp:dev/filename_time
Sep 10, 2026
Merged

lib: include extraction time in Crash Log METADATA#105
psurply merged 1 commit into
intel:mainfrom
jlrivasp:dev/filename_time

Conversation

@jlrivasp

@jlrivasp jlrivasp commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

This patch includes the extraction time in the Crash Log METADATA when the Crash Log source executes its extraction flow.

Some Crash Log sources (e.g. Event Log) already assign a METADATA time during extraction, hence this flow is a fall back for the sources that didn't include that.

The time is expressed in local time of the system; a new API returns the current instant:

use intel_crashlog::metadata::Time;

if let Some(time) = Time::now() {
    println!("{}", time)
}

@jlrivasp
jlrivasp requested a review from psurply September 4, 2026 14:54
@jlrivasp
jlrivasp force-pushed the dev/filename_time branch 4 times, most recently from 98b4373 to 1b16eb2 Compare September 8, 2026 13:17
This patch includes the extraction time in the Crash Log `METADATA` when
the Crash Log source executes its extraction flow.

Each Crash Log extraction source should add explicit code to read and
assign the extraction time in their own extraction path.

The time is expressed in local time of the system; a new API returns the
current instant:

```
use intel_crashlog::metadata::Time;

if let Some(time) = Time::now() {
    println!("{}", time)
}
```

Some Crash Log sources, call directly the `Time::now()` API, for
getting the current instance

Signed-off-by: Rivas Paz, Jose L <jose.l.rivas.paz@intel.com>
@psurply
psurply merged commit 7065365 into intel:main Sep 10, 2026
5 checks passed
@psurply

psurply commented Sep 10, 2026

Copy link
Copy Markdown
Member

Thanks!

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants