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

calamares: 3.3.8, calamares-nixos-extensions: 0.3.18 #331607

Merged
merged 2 commits into from
Aug 12, 2024

Conversation

ElvishJerricco
Copy link
Contributor

@ElvishJerricco ElvishJerricco commented Aug 1, 2024

Description of changes

Waiting on NixOS/calamares-nixos-extensions#43
Fixes #322342

Things done

  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)
    • sandbox = relaxed
    • sandbox = true
  • Tested, as applicable:
  • Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
  • Tested basic functionality of all binary files (usually in ./result/bin/)
  • 24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)
    • (Package updates) Added a release notes entry if the change is major or breaking
    • (Module updates) Added a release notes entry if the change is significant
    • (Module addition) Added a release notes entry if adding a new NixOS module
  • Fits CONTRIBUTING.md.

Add a 👍 reaction to pull requests you find important.

@emilazy emilazy added 1.severity: security Issues which raise a security issue, or PRs that fix one backport release-24.05 Backport PR automatically labels Aug 1, 2024
@ElvishJerricco ElvishJerricco changed the title calamares: 3.3.8, calamares-nixos-extensions: 7f0d1803b8cf calamares: 3.3.8, calamares-nixos-extensions: 0.3.17 Aug 1, 2024
@ofborg ofborg bot requested review from vlinkz and manveru August 1, 2024 23:45
@ofborg ofborg bot added 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild 10.rebuild-linux: 1-10 labels Aug 1, 2024
@ElvishJerricco ElvishJerricco marked this pull request as ready for review August 3, 2024 17:39
@wegank wegank added the 12.approvals: 1 This PR was reviewed and approved by one reputable person label Aug 3, 2024
@ElvishJerricco ElvishJerricco changed the title calamares: 3.3.8, calamares-nixos-extensions: 0.3.17 calamares: 3.3.8, calamares-nixos-extensions: 0.3.18 Aug 12, 2024
@ElvishJerricco
Copy link
Contributor Author

@DerRockWolf Can I get your review on the calamares bump? I made it into proper format-patch style patches, but I didn't remove the ActiveDirectory code like yours does in #333877 because it seems inconsequential. I'm pretty sure that's the only difference.

@vlinkz
Copy link
Member

vlinkz commented Aug 12, 2024

@DerRockWolf Can I get your review on the calamares bump? I made it into proper format-patch style patches, but I didn't remove the ActiveDirectory code like yours does in #333877 because it seems inconsequential. I'm pretty sure that's the only difference.

ActiveDirectory is disabled by default, so it shouldn't matter regardless: https://github.com/calamares/calamares/blob/v3.3.8/src/modules/users/users.conf#L272

Copy link
Contributor

@DerRockWolf DerRockWolf left a comment

Choose a reason for hiding this comment

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

Looks great, thanks.

I totally missed that you've already had a PR open ^^

@DerRockWolf DerRockWolf mentioned this pull request Aug 12, 2024
13 tasks
@ofborg ofborg bot requested a review from vlinkz August 12, 2024 22:36
@emilazy emilazy merged commit 588eb7a into NixOS:master Aug 12, 2024
22 of 24 checks passed
Copy link
Contributor

Successfully created backport PR for release-24.05:

@ArtemChandragupta
Copy link

Dear contributers, can you please clarify, if this pr fixes the bug about swap partitioning during graphical installation and if it is, then it will be included in nixos iso on official website?

@DerRockWolf
Copy link
Contributor

Dear contributers, can you please clarify, if this pr fixes the bug about swap partitioning during graphical installation and if it is, then it will be included in nixos iso on official website?

What "bug"? Is there any issue you can link?

Because the PR alread got back ported it will, afaik also land in the ISOs you can download on nixos.org

@ArtemChandragupta
Copy link

#322342

https://www.reddit.com/r/NixOS/s/7dPUFMZgVv

The upper issue is closed with your PR.

I have three completely different hosts (amd laptop, intel+nvidia desktop, intel laptop), all three machines can't install nixos with swap from graphical installer. It's with 24.05. When - 23.11 was released swap could been created at least on two of my machines.

Interesting, after encountering the issue I tried to install 23.11 iso that I found here, and swapon from graphical installer also crushed.

Installation from terminal works as intended.

@DerRockWolf
Copy link
Contributor

#322342 is closed because this PR contains the upstream fix (NixOS/calamares-nixos-extensions#36).

So, yes creating a swap partition via the graphical installer should work again.

@nixos-discourse
Copy link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1.severity: security Issues which raise a security issue, or PRs that fix one 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild 10.rebuild-linux: 1-10 12.approvals: 1 This PR was reviewed and approved by one reputable person backport release-24.05 Backport PR automatically
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bug: NixOS 24.05 graphical installer fails with UEFI firmware and virtio-blk device (/dev/vda) on swapon
8 participants