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

Optimize ARM runners in AKS #27836

Merged
merged 2 commits into from
Dec 2, 2024
Merged

Optimize ARM runners in AKS #27836

merged 2 commits into from
Dec 2, 2024

Conversation

ababushk
Copy link
Contributor

Details:

  • Switch Docker images builds to smaller runners
  • Switch compilation to newer generation of ARM VMs
  • Switch tests to newer generation of ARM VMs and/or smaller runners, where applicable

@ababushk ababushk requested a review from a team as a code owner November 29, 2024 16:36
@github-actions github-actions bot added category: CI OpenVINO public CI github_actions Pull requests that update GitHub Actions code labels Nov 29, 2024
@ababushk ababushk added this pull request to the merge queue Dec 2, 2024
Merged via the queue into openvinotoolkit:master with commit a0222b8 Dec 2, 2024
141 checks passed
@ababushk ababushk deleted the optimize_arm_runners branch December 2, 2024 19:41
ilya-lavrenov
ilya-lavrenov reviewed Dec 3, 2024
View reviewed changes
.github/workflows/debian_10_arm.yml
@@ -49,7 +49,7 @@ jobs:
Docker:
needs: Smart_CI
if: "!needs.smart_ci.outputs.skip_workflow"
runs-on: aks-linux-16-cores-arm-docker-build
runs-on: aks-linux-4-cores-8gb-arm-docker-build
Copy link
Contributor

Choose a reason for hiding this comment

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

for Smart CI it's not required to use ARM machine at all

Copy link
Contributor

Choose a reason for hiding this comment

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

BTW, if ARM machines are more expensive than x86, we can perform cross-compilation on x86 for arm64 to reduce costs.

Copy link
Collaborator

Choose a reason for hiding this comment

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

for Smart CI it's not required to use ARM machine at all

It's Docker job, not Smart CI, so that's expected

Copy link
Contributor Author

Choose a reason for hiding this comment

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

BTW, if ARM machines are more expensive than x86, we can perform cross-compilation on x86 for arm64 to reduce costs.

ARM machines are less expensive than x86 ones

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@akladiev akladiev akladiev approved these changes

@ilya-lavrenov ilya-lavrenov ilya-lavrenov left review comments

Assignees
No one assigned
Labels
category: CI OpenVINO public CI github_actions Pull requests that update GitHub Actions code
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ababushk @ilya-lavrenov @akladiev