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.

Sign up for GitHub

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

Jump to bottom

use own arm64 gha runners #7373

Merged
merged 19 commits into from
May 10, 2024
Merged

use own arm64 gha runners #7373

merged 19 commits into from
May 10, 2024

Conversation

cicdteam
Copy link
Member

@cicdteam cicdteam commented Apr 13, 2024 •
edited
Loading

Problem

Move from aws based arm64 runners to bare-metal based

Summary of changes

Changes in GitHub action workflows where runs-on: arm64 used. More parallelism added, build time for neon with extra platform builds workflow reduced from 45m to 25m

@github-actions GitHub Actions
Copy link

github-actions bot commented Apr 13, 2024 •
edited
Loading

3024 tests run: 2891 passed, 0 failed, 133 skipped (full report)

Code coverage* (full report)

  • functions: 31.4% (6323 of 20136 functions)
  • lines: 47.3% (47655 of 100777 lines)

* collected from Rust tests only

The comment gets automatically updated with the latest test results
f64e7e7 at 2024-05-10T11:12:33.699Z :recycle:

@cicdteam cicdteam added run-extra-build-stats When placed on a PR, tells the CI to run extra job gathering the build timing stats run-extra-build-macos When placed on a PR, tells the CI to run a build on macOS. No unit tests are run, though. run-extra-build-arm When placed on a PR, tells the CI to run a build on Linux arm run-extra-build-* When placed on a PR, tells the CI to run all extra builds and removed run-extra-build-stats When placed on a PR, tells the CI to run extra job gathering the build timing stats run-extra-build-macos When placed on a PR, tells the CI to run a build on macOS. No unit tests are run, though. run-extra-build-arm When placed on a PR, tells the CI to run a build on Linux arm labels Apr 17, 2024
@cicdteam cicdteam added the a/ci Area: related to continuous integration label Apr 18, 2024
@cicdteam cicdteam requested a review from bayandin April 18, 2024 12:57
bayandin
bayandin reviewed Apr 18, 2024
View reviewed changes
.github/actionlint.yml Outdated Show resolved Hide resolved
.github/workflows/build-build-tools-image.yml Outdated Show resolved Hide resolved
.github/workflows/neon_extra_builds.yml Show resolved Hide resolved
.github/workflows/neon_extra_builds.yml Outdated Show resolved Hide resolved
.github/workflows/neon_extra_builds.yml Show resolved Hide resolved
@cicdteam cicdteam requested a review from bayandin May 9, 2024 19:19
@cicdteam cicdteam enabled auto-merge (squash) May 9, 2024 20:59
bayandin
bayandin reviewed May 10, 2024
View reviewed changes
.github/workflows/build-build-tools-image.yml Outdated Show resolved Hide resolved
Dockerfile.build-tools Outdated Show resolved Hide resolved
bayandin
bayandin approved these changes May 10, 2024
View reviewed changes
Copy link
Member

@bayandin bayandin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great work, thanks Andrey!

@cicdteam cicdteam merged commit 873b222 into main May 10, 2024
52 of 53 checks passed
@cicdteam cicdteam deleted the use_own_arm64_runners branch May 10, 2024 11:04
a-masterov pushed a commit that referenced this pull request May 20, 2024
@cicdteam @a-masterov
use own arm64 gha runners (#7373)
b893a51 
## Problem

Move from aws based arm64 runners to bare-metal based

## Summary of changes
Changes in GitHub action workflows where `runs-on: arm64` used. More
parallelism added, build time for `neon with extra platform builds`
workflow reduced from 45m to 25m
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bayandin bayandin bayandin approved these changes

Assignees
No one assigned
Labels
a/ci Area: related to continuous integration run-extra-build-* When placed on a PR, tells the CI to run all extra builds
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@cicdteam @bayandin