Skip to content

Comments

results/hardware: pass all filter options to cmd_tests and cmd_builds#264

Merged
aliceinwire merged 1 commit intokernelci:mainfrom
bhcopeland:fix/hardware-missing-filter-params
Feb 20, 2026
Merged

results/hardware: pass all filter options to cmd_tests and cmd_builds#264
aliceinwire merged 1 commit intokernelci:mainfrom
bhcopeland:fix/hardware-missing-filter-params

Conversation

@bhcopeland
Copy link
Member

The builds_and_tests_options decorator adds filter options (start_date, end_date, compiler, config, hardware, test_path, git_branch, compatible, min_duration, max_duration) but the boots, builds, and tests commands did not accept or forward them, causing a TypeError on invocation.

Add all missing parameters to the function signatures and pass them through to cmd_tests and cmd_builds.

@bhcopeland
Copy link
Member Author

Currently SSL is down on dashboard.kernelci.org so I cannot verify. We could add a --no-verify as a option down the road

The builds_and_tests_options decorator adds filter options (start_date,
end_date, compiler, config, hardware, test_path, git_branch, compatible,
min_duration, max_duration) but the boots, builds, and tests commands
did not accept or forward them, causing a TypeError on invocation.

Add all missing parameters to the function signatures and pass them
through to cmd_tests and cmd_builds.

Signed-off-by: Ben Copeland <ben.copeland@linaro.org>
@bhcopeland bhcopeland force-pushed the fix/hardware-missing-filter-params branch from 5752cb3 to 00c3b7f Compare February 18, 2026 12:01
@aliceinwire aliceinwire merged commit 184905e into kernelci:main Feb 20, 2026
6 of 8 checks passed
@aliceinwire
Copy link
Member

aliceinwire commented Feb 20, 2026

thanks
--no-verify could be useful also for local instances of kernelci maestro

@bhcopeland
Copy link
Member Author

thanks --no-verify could be useful also for local instances of kernelci maestro

No worries I will spin another patch with this in

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