← Back to issue list

kernel plugin: check_config() and check_initrd() support

View original Github issue

Metadata

Project
snapcraft
Number
#1223
Type
pull request
State
merged
Author
piso77
Labels
—
Created
Updated
Closed

Current evaluation

Merged kernel-plugin checks that parse .config and initrd module lists, reporting missing mandatory options or boot-required code; landed in commit fda091f50.

Resolution: closed resolved

Reason: PR #1223 was merged into Snapcraft, adding check_config() and check_initrd() support in commit fda091f50.

Evaluated by: mr-cal-bot

No scores available.

Issue body

Make snapcraft smarter by parsing .config and initrd modules list and printing out if there are mandatory options missing, or if a boot required piece of code is not builtin or part packaged into initrd Signed-off-by: Paolo Pisati <paolo.pisati@canonical.com> LP: #1676806

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged kernel-plugin checks that parse .config and initrd module lists, reporting missing mandatory options or boot-required code; landed in commit fda091f50.
stealth/space-bunny-alpha — closed resolved Merged PR #1223, adding kernel-plugin checks of .config and initrd contents for missing mandatory options or boot code; recorded in commit fda091f50.
qwen/qwen3.6-35b-a3b — — Merged implementation of check_config() and check_initrd() for the kernel plugin. The update parses .config and initrd files to verify mandatory kernel options and boot code are present. Minor test coverage decrease was accepted.
qwen3.6-35b-a3b-mtp-q6 — — Merged into master. Introduces check_config() and check_initrd() to the kernel plugin for validating mandatory kernel options and initrd modules. Accepted following minor file placement discussion and code review, with a negligible coverage decrease.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#1303 kernel plugin: slightly improve the messaging of check_config() snapcraft merged Merged as 05c941787: PR #1303 by piso77 reworded kernel plugin check_config() error messages to state which kernel versions each CONFIG option applies to (LP: #1689564). Message-only, no functional change.
76%
#1689564 kernel plugin: check_config(): improve the messaging for some CONFIG_* option snapcraft (launchpad) closed Messaging-only change to kernel plugin check_config() landed via GitHub PR snapcore/snapcraft#1303, merged as commit 05c94178; maintainer sergiusens linked the PR and closed LP #1689564.
74%
#476 kernel plugin: iterate over modules to add to initrd snapcraft merged Kernel plugin now iterates over required modules when assembling the initrd; PR #476 was merged as commit 53941d2b0, resolving LP #1572118.
72%
#1319 kernel plugin: config check: remove DMIID option snapcraft merged Merged #1319 as 1dc35f17e, removing the kernel plugin’s DMIID warning so non-EFI targets such as armhf no longer receive a false warning.
71%