feature: linux kernel telemetry#627
Merged
harp-intel merged 8 commits intointel:mainfrom Feb 10, 2026
Merged
Conversation
Move the duration exit check to after output but before sleep, so the final sample is emitted before exiting. This ensures the sample count matches other telemetry scripts (e.g., duration=10 with interval=2 now produces 5 samples instead of 4). Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Change kernel telemetry output to use human-readable timestamp format (hh:mm:ss) consistent with other telemetry tables, while keeping Unix timestamp for duration calculations. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Add minor page faults/s, major page faults/s, and context switches/s to the telemetry summary table. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Contributor
There was a problem hiding this comment.
Pull request overview
Adds Linux kernel-derived telemetry metrics (from /proc/stat and /proc/vmstat) and exposes them in new Telemetry report tables and summary fields.
Changes:
- Introduces a new
kernel telemetryscript that outputs kernel/process/vm counters as CSV. - Adds new Telemetry tables (“Virtual Memory” and “Process”) wired to the new script and HTML chart renderers.
- Extends the telemetry summary to include page fault and context switch averages.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 5 comments.
| File | Description |
|---|---|
| internal/script/scripts.go | Adds a new embedded script that samples kernel stats (/proc/stat, /proc/vmstat) and emits CSV. |
| cmd/telemetry/telemetry_tables.go | Registers new table definitions and parses the kernel telemetry CSV into table fields. |
| cmd/telemetry/telemetry_renderers.go | Adds new HTML renderers to chart the new tables. |
| cmd/telemetry/telemetry.go | Adds CLI flags, registers renderers, and includes new metrics in the summary. |
…eader Signed-off-by: Harper, Jason M <jason.m.harper@intel.com>
The 10 second timeout was insufficient for graceful shutdown when kernel telemetry scripts need time to complete their cleanup handlers. This caused intermittent exit code 255 errors with empty output. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
harp-intel
approved these changes
Feb 9, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Adds a few kernel-related metrics to the telemetry report: