Skip to content

ci: install Determinate Nix in shared setup action - #4593

Merged
SDAChess merged 1 commit into
mainfrom
ci/4592-determinate-nix/SDAChess
Oct 15, 2026
Merged

SDAChess merged 1 commit into
mainfrom
ci/4592-determinate-nix/SDAChess

Conversation

@SDAChess

Copy link
Copy Markdown
Member

Summary

Use Determinate Nix for CI jobs through the shared setup-nix action, while retaining the existing OpenShell Cachix cache.

Related Issue

Closes #4592. Author has verified ADMIN access; the issue does not have state:accepted.

Changes

  • Replace cachix/install-nix-action with commit-pinned DeterminateSystems/determinate-nix-action v3.23.2 and use its github-token input.
  • Keep Cachix setup before builds, authenticated uploads, and read-only access without a token.

Testing

  • Actionlint passes for workflows calling the shared Nix and protobuf compatibility actions.
  • Zizmor reports no findings for the modified composite action.
  • git diff --check passes.
  • Full repository Actionlint reports existing concurrency.queue errors in unchanged publish-docs-website.yml and sync-docs.yml.
  • Installation and Cachix downloads/uploads require validation on GitHub-hosted CI.
  • The pre-commit hook could not run because mise failed to install Python/Node toolchains on this NixOS host; committed with that hook bypassed after the scoped checks above passed.
  • Unit and E2E tests are not applicable to this installer-only change.

Checklist

  • Follows Conventional Commits
  • Commits are signed off (DCO)
  • Architecture docs: not applicable.

Signed-off-by: Simon Scatton <sscatton@nvidia.com>
@SDAChess
SDAChess added this pull request to the merge queue Oct 15, 2026
Merged via the queue into main with commit 2c8f30f Oct 15, 2026
81 checks passed
@SDAChess
SDAChess deleted the ci/4592-determinate-nix/SDAChess branch October 15, 2026 13:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ci: use Determinate Nix while retaining Cachix

2 participants