Skip to content
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

CI improvements #11625

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from
Draft

Conversation

edolstra
Copy link
Member

@edolstra edolstra commented Oct 1, 2024

Motivation

Some improvements to simplify / speed up GHA CI:

  • Have separate actions for building Nix, running the regular tests and running the VM tests.

  • Use magic-nix-cache so users don't need to get Cachix credentials for their forks.

  • Drop the installer job (which takes ~50 minutes) since we're already building that as part of nix flake check.

Context

Priorities and Process

Add 👍 to pull requests you find important.

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

@edolstra edolstra marked this pull request as draft October 1, 2024 14:23
* Have separate actions for building Nix, running the regular tests
  and running the VM tests.

* Use magic-nix-cache so users don't need to get Cachix credentials
  for their forks.

* Drop the installer job since we're already building that as part of
  `nix flake check`.
@domenkozar
Copy link
Member

Would have appreciated a ping here, but luckily I saw it by preparing for my NixCon talk.

Two things:

  1. This is going to break install-nix tests
  2. I find the rationalization of replacing Cachix weird, does that mean if let's say we add OIDC and make Cachix faster, then we can replace it again?

I'm not sure I even want to debate things, but it looks to me the development of Nix is becoming unfriendly to other competitors beyond the blessed one, so I'm not going to try to argue back and forth.

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.

2 participants