Skip to content

installer: make sure we can always test the installer in ci and locally (backport #12059)#12074

Merged
Mic92 merged 6 commits into2.24-maintenancefrom
mergify/bp/2.24-maintenance/pr-12059
Jan 6, 2025
Merged

installer: make sure we can always test the installer in ci and locally (backport #12059)#12074
Mic92 merged 6 commits into2.24-maintenancefrom
mergify/bp/2.24-maintenance/pr-12059

Conversation

@mergify
Copy link
Copy Markdown
Contributor

@mergify mergify bot commented Dec 16, 2024

Just now there is a dependency on cachix, which means we cannot test the installer in CI if forks do not have the necessary secrets set up. We replace this with a simple http server that serves the installer and can be both used in CI and locally.

Another nice side effect is that now every pull request has a downloadable installer: https://github.com/NixOS/nix/actions/runs/12340760590

Motivation

Context


Add 👍 to pull requests you find important.

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


This is an automatic backport of pull request #12059 done by [Mergify](https://mergify.com).

@mergify
Copy link
Copy Markdown
Contributor Author

mergify bot commented Dec 16, 2024

Cherry-pick of 6162e12 has failed:

On branch mergify/bp/2.24-maintenance/pr-12059
Your branch is up to date with 'origin/2.24-maintenance'.

You are currently cherry-picking commit 6162e1220.
  (fix conflicts and run "git cherry-pick --continue")
  (use "git cherry-pick --skip" to skip this patch)
  (use "git cherry-pick --abort" to cancel the cherry-pick operation)

Changes to be committed:
	modified:   packaging/hydra.nix
	modified:   scripts/prepare-installer-for-github-actions
	new file:   scripts/serve-installer-for-github-actions

Unmerged paths:
  (use "git add <file>..." to mark resolution)
	both modified:   .github/workflows/ci.yml
	both modified:   flake.nix

To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally

@Mic92 Mic92 force-pushed the mergify/bp/2.24-maintenance/pr-12059 branch from 358fac6 to b114769 Compare January 6, 2025 22:36
@Mic92 Mic92 enabled auto-merge January 6, 2025 22:39
Mic92 and others added 6 commits January 7, 2025 00:04
Just now there is a dependency on cachix, which means we cannot test
the installer in CI if forks do not have the necessary secrets set up.
We replace this with a simple http server that serves the installer and
can be both used in CI and locally.

(cherry picked from commit 6162e12)
(cherry picked from commit df3eb4f)
Lock the Ubuntu runner to ubuntu-22.04 to avoid accidental updates [1]
and increase reproducibility.

[1]: actions/runner-images#10636
This lets us update "runs-on" without creating a mismatch with the
required checks that are configured for the repo in the github ui.
@Mic92 Mic92 force-pushed the mergify/bp/2.24-maintenance/pr-12059 branch from a120480 to 1a710e5 Compare January 6, 2025 23:04
@Mic92 Mic92 merged commit 2c89c38 into 2.24-maintenance Jan 6, 2025
@Mic92 Mic92 deleted the mergify/bp/2.24-maintenance/pr-12059 branch January 6, 2025 23:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants