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: use buildjet instead of self-hosted runner #212

Merged
merged 31 commits into from
Aug 30, 2024
Merged
Changes from 1 commit
Commits
Show all changes
31 commits
Select commit Hold shift + click to select a range
00755a4
CI: use buildjet instead of self-hosted runner
sveitser Aug 28, 2024
27406d7
CI: use public runner / buildest for docker build
sveitser Aug 28, 2024
ff98738
CI: more disk space on public runner
sveitser Aug 28, 2024
2a86757
Fix yaml: add quotes
sveitser Aug 28, 2024
1395991
Maybe single quotes?
sveitser Aug 28, 2024
ee77ce9
CI: reserve more space on root for docker build
sveitser Aug 28, 2024
c9c70b0
Manually remove unused software
sveitser Aug 28, 2024
bece2ea
More quickly remove unused software
sveitser Aug 28, 2024
118f29f
Merge branch 'integration' into ma/ci-buildjet
sveitser Aug 28, 2024
64f414f
Don't install rmz if not needed
sveitser Aug 28, 2024
9f8f607
Try using public runners or CI test job
sveitser Aug 28, 2024
b46ad72
Always free disk space
sveitser Aug 28, 2024
f8ae648
Keep hostedtoolcache
sveitser Aug 28, 2024
05c1b99
Comment the madness
sveitser Aug 28, 2024
80ee641
Check disk space at end of runs
sveitser Aug 29, 2024
c6222cd
Try running CI tests from different directory
sveitser Aug 29, 2024
452c0f9
Set working directory for external actions
sveitser Aug 29, 2024
5911cb3
Make workspace dir a symlink to /tmp/gha-work
sveitser Aug 29, 2024
2dee25f
Fix yarn cache path: does not work with symlinks
sveitser Aug 29, 2024
c195440
Comment out setup nodejs version
sveitser Aug 29, 2024
42cea12
Use new caches
sveitser Aug 29, 2024
bc72ef0
Fix upload log action
sveitser Aug 29, 2024
b533188
Maybe fix cache ref
sveitser Aug 29, 2024
ad2311f
Fix typo
sveitser Aug 29, 2024
496ecf5
Remove non-working docker cache
sveitser Aug 29, 2024
3690a9c
Revert "Remove non-working docker cache"
sveitser Aug 29, 2024
9a9dde1
Remove comments
sveitser Aug 29, 2024
795cc0a
Keep azure tools (maybe fix redis tests)
sveitser Aug 29, 2024
c34946a
Revert "Keep azure tools (maybe fix redis tests)"
sveitser Aug 30, 2024
19f1edb
Fix redis URL
sveitser Aug 30, 2024
8c5f5a9
Move redis tests to own workflow
sveitser Aug 30, 2024
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
CI: use public runner / buildest for docker build
This removes usage of our self-hosted github runners.
sveitser committed Aug 28, 2024

Verified

This commit was signed with the committer’s verified signature.
sveitser Mathis
commit 27406d7ff045e83152e0c002463e91ca64297a65
4 changes: 2 additions & 2 deletions .github/actions/docker-image/action.yml
Original file line number Diff line number Diff line change
@@ -40,8 +40,8 @@ runs:
target: ${{ inputs.target }}
labels: ${{ steps.metadata.outputs.labels }}
platforms: ${{ inputs.platforms }}
cache-from: type=gha
cache-to: type=gha,mode=max
cache-from: type=registry,ref=user/app:latest
sveitser marked this conversation as resolved.
Show resolved Hide resolved
cache-to: type=inline
outputs: type=image,name=${{ inputs.images }},push-by-digest=true,name-canonical=true,push=true

# This will upload the digest for each architecture to the same artifact,
6 changes: 3 additions & 3 deletions .github/workflows/espresso-docker.yml
Original file line number Diff line number Diff line change
@@ -36,11 +36,11 @@ jobs:
platform: [linux/amd64, linux/arm64]
include:
- platform: linux/amd64
runs-on: X64
runs-on: ubuntu-latest
- platform: linux/arm64
runs-on: ARM64
runs-on: buildjet-4vcpu-ubuntu-2204-arm

runs-on: [self-hosted, "${{ matrix.runs-on }}"]
runs-on: ${{ matrix.runs-on }}

steps:
# TODO We should be able to remove this but currently it's needed to avoid