Skip to content

Conversation

@padovan
Copy link
Contributor

@padovan padovan commented Jun 25, 2025

Implements commit history view for the summary command using the dashboard API's /commits endpoint. The new --history flag displays historical test results in a colored table format with pass/fail/inconclusive counts for easy visualization of test trends across commits.

  • Add dashboard_fetch_commits_history() API function
  • Add --history flag to summary command options
  • Implement cmd_commits_history() parser with colored table output
  • Auto-fetch latest commit when using history mode
  • Handle both JSON and table output formats
  • Use kci_ color helpers for consistent formatting

@sashalevin
Copy link

The --history flag was added to the common_options decorator, which is used by multiple commands (summary, builds, boots, tests). However, only the summary command was updated to handle the history parameter. The other commands (builds, boots, tests) were missing this parameter in their function signatures.

You need something like the following to unbreak other commands:

diff --git a/kcidev/subcommands/results/init.py b/kcidev/subcommands/results/init.py
index 373eeca..55dc2d7 100644
--- a/kcidev/subcommands/results/init.py
+++ b/kcidev/subcommands/results/init.py
@@ -140,6 +140,7 @@ def builds(
latest,
arch,
tree,

  • history,
    download_logs,
    status,
    filter,
    @@ -196,6 +197,7 @@ def boots(
    latest,
    arch,
    tree,
  • history,
    download_logs,
    status,
    filter,
    @@ -254,6 +256,7 @@ def tests(
    latest,
    arch,
    tree,
  • history,
    download_logs,
    status,
    filter,

@padovan padovan force-pushed the history branch 2 times, most recently from ed02477 to 0cd11f1 Compare June 27, 2025 18:19
Implements commit history view for the summary command using the dashboard
API's /commits endpoint. The new --history flag displays historical test
results in a colored table format with pass/fail/inconclusive counts for
easy visualization of test trends across commits.

Signed-off-by: Gustavo Padovan <gustavo.padovan@collabora.com>
@padovan padovan marked this pull request as ready for review June 27, 2025 19:32
@aliceinwire
Copy link
Member

thanks for also adding the documentation

@aliceinwire aliceinwire merged commit 80ce03f into kernelci:main Jul 2, 2025
5 checks passed
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.

3 participants