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

Move all CI to GitHub Actions #923

Merged
merged 11 commits into from
Apr 15, 2024
Merged

Move all CI to GitHub Actions #923

merged 11 commits into from
Apr 15, 2024

Conversation

cole-h
Copy link
Member

@cole-h cole-h commented Apr 5, 2024

Description

Buildkite is awesome, but makes it difficult to accept PRs from external contributors (the action we were using to wait for Buildkite to complete requires a token secret, and we don't allow external contributors to have access to it, causing CI to not do anything useful for them). By moving CI to run entirely in GitHub Actions, it becomes much easier to accept external contributions.

Checklist
  • Formatted with cargo fmt
  • Built with nix build
  • Ran flake checks with nix flake check
  • Added or updated relevant tests (leave unchecked if not applicable)
  • Added or updated relevant documentation (leave unchecked if not applicable)
  • Linked to related issues (leave unchecked if not applicable)
Validating with install.determinate.systems

If a maintainer has added the upload to s3 label to this PR, it will become available for installation via install.determinate.systems:

curl --proto '=https' --tlsv1.2 -sSf -L https://install.determinate.systems/nix/pr/$PR_NUMBER | sh -s -- install

@cole-h cole-h added upload to s3 The labeled PR is allowed to upload its artifacts to S3 for easy testing and removed upload to s3 The labeled PR is allowed to upload its artifacts to S3 for easy testing labels Apr 8, 2024
@cole-h cole-h added upload to s3 The labeled PR is allowed to upload its artifacts to S3 for easy testing and removed upload to s3 The labeled PR is allowed to upload its artifacts to S3 for easy testing labels Apr 8, 2024
@cole-h cole-h changed the title wip: attempt to get rid of buildkite so that external contributors ca… Move all CI to GitHub Actions Apr 8, 2024
@cole-h cole-h marked this pull request as ready for review April 8, 2024 19:39
@cole-h cole-h added upload to s3 The labeled PR is allowed to upload its artifacts to S3 for easy testing and removed upload to s3 The labeled PR is allowed to upload its artifacts to S3 for easy testing labels Apr 8, 2024
@cole-h cole-h marked this pull request as draft April 10, 2024 14:10
@cole-h

This comment was marked as resolved.

@cole-h cole-h marked this pull request as ready for review April 12, 2024 19:14
As awesome as Buildkite is, it is unfortunately not useful for this
setup:

* We use Buildkite to build installer binaries
* We then use GitHub Actions to run the installer binaries on a host
without Nix already installed

We don't use Buildkite for both steps because setting up agents to
install Nix on would be a pain, as erasing everything takes a while.

This is preparation for using GitHub Actions for everything in our CI
pipeline, as it is much faster and we don't need to worry about keeping
the runners running ourselves (there have been times where our Buildkite
agents were unreachable for some reason or another).
So that when I go to fix the releases action, it will be easier to
reuse.
There are no GitHub Actions that make it easy to wait for another
workflow to complete :/
@cole-h cole-h merged commit 60458ab into main Apr 15, 2024
6 checks passed
@cole-h cole-h deleted the use-gha branch April 15, 2024 16:40
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