Skip to content

Use arm runner#75

Merged
dc-mak merged 3 commits intorems-project:mainfrom
dc-mak:use-arm-runner
Apr 25, 2025
Merged

Use arm runner#75
dc-mak merged 3 commits intorems-project:mainfrom
dc-mak:use-arm-runner

Conversation

@dc-mak
Copy link
Contributor

@dc-mak dc-mak commented Apr 15, 2025

Cuts down CI time from 6 hours to like 20ish minutes.

@dc-mak
Copy link
Contributor Author

dc-mak commented Apr 18, 2025

@podhrmic

@podhrmic
Copy link

@dc-mak attests is related to building an SBOM. We don't need it quite yet, so it is OK to remove (worth speeding up the build). The changes look good, I am in favor of merging. There is a condition that if the build-and-release job passes, but combine-multiplatform fails, then the existing image manifest will be corrupted, as the hashes will be incorrect. I recommend merging as is (well, once the failed tests are fixed), and if we see these failures happening we can address it.

@dc-mak dc-mak marked this pull request as ready for review April 23, 2025 13:02
@dc-mak dc-mak force-pushed the use-arm-runner branch 2 times, most recently from d34cb32 to fce5616 Compare April 23, 2025 13:09
@dc-mak dc-mak merged commit 1d99b68 into rems-project:main Apr 25, 2025
9 checks passed
@dc-mak dc-mak deleted the use-arm-runner branch April 25, 2025 16:21
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