Skip to content

feat: enable extract and control commands to access non-repeated Crash Log sources - #104

Merged
psurply merged 2 commits into
intel:mainfrom
jlrivasp:dev/default_control
Sep 8, 2026
Merged

feat: enable extract and control commands to access non-repeated Crash Log sources#104
psurply merged 2 commits into
intel:mainfrom
jlrivasp:dev/default_control

Conversation

@jlrivasp

@jlrivasp jlrivasp commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

feat: enable extract and control commands to access non-repeated Crash Log sources

This patch enables extract command to collect all discovered Crash Log
sources when no -s parameter is provided and avoids to collect sources
that are mapped several times (e.g. BDF and sys class PMT sources):

$ iclg extract

This change also impact the control commands; the CLI will execute the
control command in all discovered Crash Log sources avoiding to execute
the command twice in the repeated sources:

$ iclg trigger

This patch also includes discover_distinct method in the CrashLogSource
API, which is used to handle the logic to avoid returning these sources
that are mapped to 2 different endpoints

Signed-off-by: Rivas Paz, Jose L jose.l.rivas.paz@intel.com

@jlrivasp
jlrivasp requested a review from psurply September 2, 2026 14:09
@psurply psurply changed the title Dev/default control feat: enable extract and control commands to access non-repeated Crash Log sources Sep 3, 2026
This patch includes `discover_distinct` method in the CrashLogSource
API.
This is a discovery method but as a difference with `discover` this one
is avoiding the return of repeated sources, which is the case of the
PMT sys class and PMT BDF sources.

The usage of this method is for the automatic discovery when executing a
control command or extracting and avoiding extracting data twice or
sending the control command twice

Signed-off-by: Rivas Paz, Jose L <jose.l.rivas.paz@intel.com>
This patch enables the CLI to execute a control command on all the
discoverable endpoints when no parameter was indicated with "-s"

There is another change in the extraction flow as well, where the
extracted data from PMT devices will be done only for the sys class PMT
devices and not from the BDF devices.

Signed-off-by: Rivas Paz, Jose L <jose.l.rivas.paz@intel.com>
@jlrivasp
jlrivasp force-pushed the dev/default_control branch from e291a7f to c918e28 Compare September 4, 2026 16:48
@psurply
psurply merged commit eae992f into intel:main Sep 8, 2026
5 checks passed
@psurply

psurply commented Sep 8, 2026

Copy link
Copy Markdown
Member

Thank you!

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