-
Notifications
You must be signed in to change notification settings - Fork 807
[nix] Update to 25.05 #4050
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
[nix] Update to 25.05 #4050
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR updates the Nixpkgs input in flake.nix
from version 0.2411 to 0.2505 to align with the NixOS 25.05 release.
- Bump Nixpkgs flake input version to 0.2505
Taskfile.yml
Outdated
E2E_SERIAL: 1 | ||
cmds: | ||
- task: test-e2e-kube | ||
- cmd: bash -x ./scripts/tests.e2e.kube.sh {{.CLI_ARGS}} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I noticed this needed to be changed when the job flaked in CI on this PR. Our e2e jobs can't be reliably run in parallel on github action runners, and E2E_SERIAL is how we ensure serial execution. But since tasks don't inherit the environment of a calling task, this wasn't previously configured correctly. Calling the script directly ensures that E2E_SERIAL will be respected.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
A given nix release only receives security updates for 7 months, and though security updates aren't critical for our tooling, there's no reason not to upgrade regularly. https://nixos.org/blog/announcements/2025/nixos-2505/
Updated to limit the scope of the e2e-kube job to just xsvm (to ensure subnet coverage) to avoid the flakiness caused by the resource-constrained github runners. In any case, the kube job is now disabled. |
Why this should be merged
A given nix release only receives security updates for 7 months, and though security updates aren't critical for our tooling, there's no reason not to upgrade regularly.
https://nixos.org/blog/announcements/2025/nixos-2505/
How this was tested
CI
Need to be documented in RELEASES.md?
N/A