Skip to content

kernel-modules: update the docs to use sysext #432

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

danzatt
Copy link
Contributor

@danzatt danzatt commented Jun 6, 2025

Update the docs for building custom kernel modules to use sysexts instead of manually mounting overlayfs using a systemd service.

[Describe the testing you have done before submitting this PR. Please include both the commands you issued as well as the output you got.]

  • Changelog entries added in the respective changelog/ directory (user-facing change, bug fix, security fix, update)
  • Inspected CI output for image differences: /boot and /usr size, packages, list files for any missing binaries, kernel modules, config files, kernel modules, etc.

Copy link

github-actions bot commented Jun 6, 2025

Azure Static Web Apps: Your stage site is ready! Visit it here: https://lemon-wave-085522403-432.westeurope.1.azurestaticapps.net

Comment on lines +44 to +48
source /etc/os-release && \
printf "ID=flatcar\nVERSION_ID=%s\nARCHITECTURE=%s\n" \
"$VERSION_ID" \
"$(hostnamectl | grep 'Architecture:' | awk '{print $2}')" \
| sudo tee /var/lib/extensions/kernel-modules/usr/lib/extension-release.d/extension-release.kernel-modules
Copy link
Contributor

Choose a reason for hiding this comment

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

Nicer?

Suggested change
source /etc/os-release && \
printf "ID=flatcar\nVERSION_ID=%s\nARCHITECTURE=%s\n" \
"$VERSION_ID" \
"$(hostnamectl | grep 'Architecture:' | awk '{print $2}')" \
| sudo tee /var/lib/extensions/kernel-modules/usr/lib/extension-release.d/extension-release.kernel-modules
sudo tee /var/lib/extensions/kernel-modules/usr/lib/extension-release.d/extension-release.kernel-modules <<EOF
ID=flatcar
VERSION_ID=$(. /etc/os-release && echo \$VERSION_ID)
ARCHITECTURE=$(hostnamectl | grep 'Architecture:' | awk '{print $2}')
EOF

I had hoped to find a nicer way to get the architecture but no luck.

chewi
chewi previously approved these changes Jun 9, 2025
Copy link
Contributor

@chewi chewi left a comment

Choose a reason for hiding this comment

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

Other than that, I think this is okay, although I'll admit I'm not that familiar with this process.

Update the docs for building custom kernel modules to use sysexts
instead of manually mounting overlayfs using a systemd service.

Signed-off-by: Daniel Zatovic <daniel.zatovic@gmail.com>
Copy link

Azure Static Web Apps: Your stage site is ready! Visit it here: https://lemon-wave-085522403-432.westeurope.1.azurestaticapps.net

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