Skip to content

Collect system info in consecutive tests mode with per-test label association - #23

Draft
christiam with Copilot wants to merge 2 commits into
masterfrom
copilot/collect-system-info-in-tests
Draft

Collect system info in consecutive tests mode with per-test label association#23
christiam with Copilot wants to merge 2 commits into
masterfrom
copilot/collect-system-info-in-tests

Conversation

Copilot AI commented Aug 6, 2026

Copy link
Copy Markdown

System resource snapshots (system_info) were only collected during parallel execution via run_on_wait. In consecutive mode, no system info was recorded. Additionally, there was no way to associate a system snapshot with the specific test that was running at the time.

Schema (ddl/create.sql)

  • Added test_label VARCHAR(255) DEFAULT '' to system_info table — links each snapshot to the executing test
  • Updated system_info_view to expose test_label

Driver (bin/driver.pl)

  • Updated SQL_SYS_INFO to include test_label
  • Added memory/CPU collection before each command in consecutive mode, tagging each snapshot with label4run:
    my @sysdata = ($mem->{USED}, ..., $mem->{PMEM_USED}, $pcpu, $label4run);
    &save2db($sth_sysinfo, $host_id, @sysdata);
  • Parallel mode passes '' as test_label (schema compatibility; individual label attribution isn't meaningful there)

Test data (ddl/insert.sql)

  • Updated system_info fixture inserts to include test_label values

…tion

- Add test_label column to system_info table and system_info_view
- Update SQL_SYS_INFO constant to include test_label
- Collect system info before each command in consecutive (non-parallel) mode
- Pass test label to associate system snapshots with individual commands
- Update parallel mode to pass empty test_label for compatibility
- Update insert.sql test data with test_label values

Co-authored-by: christiam <736979+christiam@users.noreply.github.com>
Copilot AI changed the title [WIP] Add collection of system information in consecutive tests mode Collect system info in consecutive tests mode with per-test label association Aug 6, 2026
Copilot AI requested a review from christiam August 6, 2026 00:24
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.

Collect system information in consecutive tests mode

2 participants