Skip to content

actions/checkout: use single tmpfs with subfolders#435806

Merged
wolfgangwalther merged 1 commit intoNixOS:masterfrom
wolfgangwalther:ci-tmpfs-darwin
Aug 22, 2025
Merged

actions/checkout: use single tmpfs with subfolders#435806
wolfgangwalther merged 1 commit intoNixOS:masterfrom
wolfgangwalther:ci-tmpfs-darwin

Conversation

@wolfgangwalther
Copy link
Contributor

@wolfgangwalther wolfgangwalther commented Aug 22, 2025

This fixes problems on darwin, where the new tmpfs folder would not be fully empty, but git only creates worktrees on fully empty directories.

Fixes https://github.com/NixOS/nixpkgs/pull/435526/files#r2292959252

Things done

Still needs testing.


Add a 👍 reaction to pull requests you find important.

This fixes problems on darwin, where the new tmpfs folder would not be
fully empty, but git only creates worktrees on fully empty directories.
@nixpkgs-ci nixpkgs-ci bot added 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux. 6.topic: continuous integration Affects continuous integration (CI) in Nixpkgs, including Ofborg and GitHub Actions 6.topic: policy discussion Discuss policies to work in and around Nixpkgs backport release-25.05 labels Aug 22, 2025
@wolfgangwalther
Copy link
Contributor Author

I double checked that all checkout times are still ~10s, i.e. they happen on tmpfs. No instances missed, no failures in my fork, nor in the pull_request trigger here. Whether it fixes the problem on darwin or not, can't be told from the results of this PR, because the failure appears only randomly. But conceptually this should 100% work, the problem is well understood and fixed that way.

@wolfgangwalther wolfgangwalther merged commit 7c349fc into NixOS:master Aug 22, 2025
52 of 55 checks passed
@wolfgangwalther wolfgangwalther deleted the ci-tmpfs-darwin branch August 22, 2025 10:12
@nixpkgs-ci
Copy link
Contributor

nixpkgs-ci bot commented Aug 22, 2025

Successfully created backport PR for release-25.05:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

6.topic: continuous integration Affects continuous integration (CI) in Nixpkgs, including Ofborg and GitHub Actions 6.topic: policy discussion Discuss policies to work in and around Nixpkgs 8.has: port to stable This PR already has a backport to the stable release. 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant