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

sd-image-aarch64: use default (LTS) kernel #52886

Merged
merged 2 commits into from Dec 26, 2018

Conversation

samueldr
Copy link
Member

Motivation for this change

A follow-up to #52863 and #52721 (comment).

With this merged, please only paired with #52863 (so no sd-image with a regressed kernel gets built), this will fix the current issue holding back channel updates.

This uses the same kernel as it does for iso-image, the default kernel, usually the most recent LTS. This also adds a new attribute on release.nix sd_image_new_kernel which is like iso_minimal_new_kernel.

Things done
  • ✔️ Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
  • Built on platform(s)
    • ✔️ NixOS
    • ⬜ macOS
    • ⬜ other Linux distributions
  • ✔️ Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • ✔️ Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
  • ⬜ Tested execution of all binary files (usually in ./result/bin/)
  • ⬜ Determined the impact on package closure size (by running nix path-info -S before and after)
  • ✔️ Assured whether relevant documentation is up to date
  • ✔️ Fits CONTRIBUTING.md.

Do note that sd_image_new_kernel currently doesn't build, for the same reason sd_image currently doesn't. When the zfs issues for Linux 4.20 will be fixed, it should be fixed as well.

This, paired with the previous commit, ensures the channel won't be held
back from a kernel upgrade and a non-building sd image, while still
having a new-kernel variant available.
@samueldr samueldr force-pushed the aarch64/sd-image-kernel-version branch from 2368ef7 to 387098c Compare December 26, 2018 03:32
@domenkozar domenkozar merged commit 302d53d into NixOS:master Dec 26, 2018
@samueldr samueldr deleted the aarch64/sd-image-kernel-version branch December 26, 2018 15:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants