Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rework kci config with new framework #2138

Merged
merged 2 commits into from
Nov 8, 2023

Conversation

gctucker
Copy link
Contributor

@gctucker gctucker commented Oct 19, 2023

Rework kci config with the new command-line framework based on Click.

Note: This doesn't implement kci config set yet, it can be added as a follow-up.

See the discussion about this command: kernelci/kernelci-project#263 (comment)

@gctucker gctucker mentioned this pull request Oct 19, 2023
40 tasks
@gctucker gctucker force-pushed the kci-config-rework branch 3 times, most recently from dbebc77 to c93dd85 Compare October 23, 2023 19:19
@gctucker gctucker force-pushed the kci-config-rework branch 2 times, most recently from 124e3e6 to 4a74841 Compare October 24, 2023 10:47
@gctucker gctucker marked this pull request as draft October 24, 2023 10:47
@gctucker gctucker force-pushed the kci-config-rework branch 4 times, most recently from e14aad0 to 7fd5e0d Compare October 31, 2023 12:45
@gctucker gctucker marked this pull request as ready for review October 31, 2023 12:46
@gctucker gctucker added the staging-skip Don't test automatically on staging.kernelci.org label Nov 3, 2023
@gctucker
Copy link
Contributor Author

gctucker commented Nov 3, 2023

Adding staging-skip so that #2160 can be tested on staging and merged first.

@gctucker gctucker force-pushed the kci-config-rework branch 2 times, most recently from fd78d81 to 511b5bf Compare November 7, 2023 08:30
@gctucker gctucker removed the staging-skip Don't test automatically on staging.kernelci.org label Nov 7, 2023
Implement `kci config` again with the new framework using Click and
include a better way of dumping a particular section of the config
with `kci config dump`.

Enable `kci config validate` again in the Makefile test for YAML
validation.

Signed-off-by: Guillaume Tucker <guillaume.tucker@collabora.com>
Remove the old `kci config` command implementation now that there's a
replacement based on Click.

Signed-off-by: Guillaume Tucker <guillaume.tucker@collabora.com>
Copy link
Contributor Author

@gctucker gctucker left a comment

Choose a reason for hiding this comment

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

Alright, we're back to "merge and fixup" mode due to lack of available reviewers.

@gctucker gctucker added this pull request to the merge queue Nov 8, 2023
Merged via the queue into kernelci:main with commit a6406e3 Nov 8, 2023
4 checks passed
@gctucker gctucker deleted the kci-config-rework branch November 8, 2023 13:13
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.

1 participant