Skip to content

Conversation

@artemist
Copy link
Member

@artemist artemist commented Jan 5, 2026

Motivation

A commit in #14800 broke tests around creating cgroups due to incorrect path handling logic.

Fix that logic and represent cgroups as CanonPath.

Context

See failure in https://hydra.nixos.org/build/318367985/nixlog/11 and comments on #14800


Add 👍 to pull requests you find important.

The Nix maintainer team uses a GitHub project board to schedule and track reviews.

A commit in NixOS#14800 broke tests around creating cgroups due to incorrect
path handling logic.
(See https://hydra.nixos.org/build/318367985/nixlog/11)

Fix that logic and represent cgroups as CanonPath.

Co-authored-by: John Ericson <[email protected]>
Co-authored-by: Sergei Zimmerman <[email protected]>
@github-actions github-actions bot added the new-cli Relating to the "nix" command label Jan 5, 2026
Copy link
Member

@Ericson2314 Ericson2314 left a comment

Choose a reason for hiding this comment

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

Waiting for @xokdvium since @artemist works with me now so I should recuse myself.

(Sergei is credited as coauthor, but only for 1 line which we ended up changing, so I don't think it is bad for him to review it. :))

@xokdvium xokdvium added this pull request to the merge queue Jan 5, 2026
Merged via the queue into NixOS:master with commit cc5a403 Jan 5, 2026
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

new-cli Relating to the "nix" command

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants