Skip to content

Conversation

maru-ava
Copy link
Contributor

@maru-ava maru-ava commented Jul 2, 2025

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

@maru-ava maru-ava self-assigned this Jul 2, 2025
@Copilot Copilot AI review requested due to automatic review settings July 2, 2025 22:50
@maru-ava maru-ava added the tooling Build, test and development tooling label Jul 2, 2025
Copy link
Contributor

@Copilot Copilot AI left a 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

@maru-ava maru-ava requested a review from StephenButtolph as a code owner July 2, 2025 22:54
@maru-ava maru-ava added the ci This focuses on changes to the CI process label Jul 2, 2025
@maru-ava maru-ava moved this to Ready 🚦 in avalanchego Jul 2, 2025
Taskfile.yml Outdated
E2E_SERIAL: 1
cmds:
- task: test-e2e-kube
- cmd: bash -x ./scripts/tests.e2e.kube.sh {{.CLI_ARGS}}
Copy link
Contributor Author

@maru-ava maru-ava Jul 2, 2025

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.

Copy link
Contributor

@RodrigoVillar RodrigoVillar left a comment

Choose a reason for hiding this comment

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

LGTM

@StephenButtolph StephenButtolph added this pull request to the merge queue Jul 3, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jul 3, 2025
@joshua-kim joshua-kim moved this from Ready 🚦 to In Progress 🏗️ in avalanchego Jul 15, 2025
@maru-ava maru-ava moved this from In Progress 🏗️ to Ready 🚦 in avalanchego Jul 17, 2025
@maru-ava
Copy link
Contributor Author

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.

@maru-ava maru-ava enabled auto-merge July 17, 2025 02:28
@maru-ava maru-ava added this pull request to the merge queue Jul 17, 2025
Merged via the queue into master with commit eb6f96d Jul 17, 2025
29 checks passed
@maru-ava maru-ava deleted the nix-2505 branch July 17, 2025 02:48
@github-project-automation github-project-automation bot moved this from Ready 🚦 to Done 🎉 in avalanchego Jul 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci This focuses on changes to the CI process tooling Build, test and development tooling
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

4 participants