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

lxd: 5.20 -> 5.21.0 #296439

Merged
merged 1 commit into from
Mar 21, 2024
Merged

lxd: 5.20 -> 5.21.0 #296439

merged 1 commit into from
Mar 21, 2024

Conversation

marsam
Copy link
Contributor

@marsam marsam commented Mar 16, 2024

Changelog: https://github.com/canonical/lxd/releases/tag/lxd-5.21.0

Description of changes

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.05 Release Notes (or backporting 23.05 and 23.11 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.

@adamcstephens
Copy link
Contributor

@marsam how extensively do you use lxd? What would you think of deprecating lxd for 24.05 in favor of incus?

@jnsgruk
Copy link
Member

jnsgruk commented Mar 16, 2024

I mean selfishly I'd rather not because i use the crap out of it for work 😅

@marsam
Copy link
Contributor Author

marsam commented Mar 17, 2024

tbh I'm not using lxd nowadays, for different reasons.
I just saw the new release in my feed, and I thought that I may well update it 😅

imho lxd is working rn and there may be people depending on it, so I'm a bit reluctant to deprecate it without a strong reason. However, I won't object if you prefer to depreciate it.

@adamcstephens
Copy link
Contributor

Yeah I realize it could be disruptive but it also feels duplicative to have both lxd and incus. I’ve also continued (and will continue) to progress the support for incus but am not updating lxd. If enough people are fine with lxd as is and don’t care about the improvements maybe we can just leave it. I do find that incus fits nixos a bit better given their focus on being distro-agnostic, while lxd is really only focused on supporting snap environments. That said lxd has worked well enough for other distributions.

@adamcstephens
Copy link
Contributor

@ofborg build lxd lxd.passthru.tests

@adamcstephens
Copy link
Contributor

This version is an LTS. If we're planning to keep lxd in tree, I'd suggest we stick with LTS moving forward.

Note that this is currently failing the lxd-to-incus test, since 5.21 is not currently supported for migrations. Users who make this upgrade will currently not be able to migrate to incus if they so choose. Incus may add this as a supported migration, at the cost of an auth table.

@adamcstephens
Copy link
Contributor

lxc/incus#639 lxd-to-incus PR for updating to support migrations.

@adamcstephens adamcstephens merged commit c962450 into NixOS:master Mar 21, 2024
29 checks passed
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.

5 participants