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: create multiarch vm images #11017

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

mikhail-sakhnov
Copy link
Contributor

@mikhail-sakhnov mikhail-sakhnov commented Feb 27, 2025

Problem

We build compute-nodes as multi-arch images, but not the vm-compute-nodes. The PR adds multiarch vm images the same way as in autoscaling repo.

Summary of changes

Add architecture to the matrix for vm compute build steps
Add merge job

Copy link

github-actions bot commented Feb 27, 2025

7744 tests run: 7365 passed, 0 failed, 379 skipped (full report)


Flaky tests (1)

Postgres 17

Code coverage* (full report)

  • functions: 32.8% (8644 of 26359 functions)
  • lines: 48.7% (73208 of 150476 lines)

* collected from Rust tests only


The comment gets automatically updated with the latest test results
d69f1ff at 2025-02-27T14:55:05.757Z :recycle:

@mikhail-sakhnov mikhail-sakhnov force-pushed the misha/build-vm-compute-images-for-arm branch from 43fe99d to d69f1ff Compare February 27, 2025 13:41
@mikhail-sakhnov mikhail-sakhnov marked this pull request as ready for review February 27, 2025 15:26
@mikhail-sakhnov mikhail-sakhnov requested a review from a team as a code owner February 27, 2025 15:26
Comment on lines +758 to +765
- pg: v14
debian: bullseye
- pg: v15
debian: bullseye
- pg: v16
debian: bullseye
- pg: v17
debian: bookworm
Copy link
Member

Choose a reason for hiding this comment

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

It seems we don't use ${{ matrix.version.debian }} in this job, so we can delete it from parameters:

Suggested change
- pg: v14
debian: bullseye
- pg: v15
debian: bullseye
- pg: v16
debian: bullseye
- pg: v17
debian: bookworm
- pg: v14
- pg: v15
- pg: v16
- pg: v17

vm-compute-node-image:
needs: [ vm-compute-node-image-arch, meta ]
if: ${{ contains(fromJSON('["push-main", "pr", "compute-release", "compute-rc-pr"]'), needs.meta.outputs.run-kind) }}
runs-on: [ self-hosted, large ]
Copy link
Member

Choose a reason for hiding this comment

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

In other jobs we just use GitHub-hosted runner for that

Suggested change
runs-on: [ self-hosted, large ]
runs-on: ubuntu-22.04

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.

3 participants