Skip to content

subcommand: add new subsystem/subcommand - storage#274

Open
nuclearcat wants to merge 2 commits intokernelci:mainfrom
nuclearcat:storage
Open

subcommand: add new subsystem/subcommand - storage#274
nuclearcat wants to merge 2 commits intokernelci:mainfrom
nuclearcat:storage

Conversation

@nuclearcat
Copy link
Copy Markdown
Member

Add new subcommand to use KernelCI storage.
Useful to store logs, artifacts and etc.

Example: kci-dev --instance=staging storage checkauth

Add new subcommand to use KernelCI storage.
Useful to store logs, artifacts and etc.

Signed-off-by: Denys Fedoryshchenko <denys.f@collabora.com>
Copy link
Copy Markdown

@tales-aparecida tales-aparecida left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Could do with a bit of "assert_called_once_with" smoke testing, to ensuring the cli args are being forwarded as expected

Signed-off-by: Denys Fedoryshchenko <denys.f@collabora.com>
@aliceinwire
Copy link
Copy Markdown
Member

aliceinwire commented May 2, 2026

looks like black check is failing, can you fix the black formatting issues?

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