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

nixos/release: add proxmox LXC and VMA #172076

Merged
merged 2 commits into from
Jul 13, 2022
Merged

Conversation

illustris
Copy link
Contributor

@illustris illustris commented May 8, 2022

Description of changes
  • Fix failing build of proxmox VMA
  • Add hydra builds for proxmox VMA and LXC tarball

Closes: #171989

Things done
  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
  • 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/)
  • 22.05 Release Notes (or backporting 21.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
    • (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes
  • Fits CONTRIBUTING.md.

@illustris illustris changed the title nixos/proxmox-image: use qemu 6.2 for building VMA nixos/release: add proxmox LXC and VMA May 8, 2022
@nixos-discourse
Copy link

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:

https://discourse.nixos.org/t/prs-ready-for-review/3032/857

@nixos-discourse
Copy link

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:

https://discourse.nixos.org/t/prs-ready-for-review/3032/871

@illustris
Copy link
Contributor Author

[illustris@illustris-thinkpad:~/src/nixpkgs/.worktree/proxmox-kvm]$ nix-build ./nixos/release.nix -A proxmoxImage
trace: warning: system.stateVersion is not set, defaulting to 22.11. Read why this matters on https://nixos.org/manual/nixos/stable/options.html#opt-system.stateVersion.
/nix/store/q30abjnd0h8fvb0cvvvgqsg9y5kl2y2i-proxmox-nixos-22.11pre-git
[illustris@illustris-thinkpad:~/src/nixpkgs/.worktree/proxmox-kvm]$ nix-build ./nixos/release.nix -A proxmoxLXC
trace: warning: system.stateVersion is not set, defaulting to 22.11. Read why this matters on https://nixos.org/manual/nixos/stable/options.html#opt-system.stateVersion.
/nix/store/fwvyckvy5mgzp8mwh1z7xr1h9a6vcds2-tarball

@nixos-discourse
Copy link

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:

https://discourse.nixos.org/t/prs-ready-for-review/3032/950

@Profpatsch
Copy link
Member

I don’t know about these parts of the ecosystem, but the changes look good to me.

@Profpatsch Profpatsch merged commit 26c66bc into NixOS:master Jul 13, 2022
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.

proxmox-image fails to build with qemu 7.0
3 participants