Skip to content

Commit 2fa3652

Browse files
mkruskal-googlecopybara-github
authored andcommitted
Mark aarch64 tests continous-only
These are by far our slowest tests due to their QEMU use. They rarely catch unique issues, so we can handle this in post-submits instead of paying ~4 minutes on every presubmit PiperOrigin-RevId: 818096859
1 parent 47d3012 commit 2fa3652

File tree

2 files changed

+12
-2
lines changed

2 files changed

+12
-2
lines changed

.github/workflows/test_cpp.yml

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -110,26 +110,33 @@ jobs:
110110
fail-fast: false # Don't cancel all jobs if one fails.
111111
matrix:
112112
arch: [x86_64, aarch64]
113-
name: Linux Release ${{ matrix.arch }}
113+
include:
114+
- arch: aarch64
115+
continuous-only: true
116+
name: ${{ matrix.config.continuous-only && inputs.continuous-prefix || '' }} Linux Release ${{ matrix.arch }}
114117
runs-on: ubuntu-22-4core
115118
steps:
116119
- name: Checkout pending changes
120+
if: ${{ !matrix.config.continuous-only || inputs.continuous-run }}
117121
uses: protocolbuffers/protobuf-ci/checkout@v4
118122
with:
119123
ref: ${{ inputs.safe-checkout }}
120124
- name: Cross compile protoc for ${{ matrix.arch }}
125+
if: ${{ !matrix.config.continuous-only || inputs.continuous-run }}
121126
id: cross-compile
122127
uses: protocolbuffers/protobuf-ci/cross-compile-protoc@v4
123128
with:
124129
image: us-docker.pkg.dev/protobuf-build/containers/common/linux/bazel:7.6.1-e0df73e51131ccaf53451355d22577f377357604 #TODO: b/409557730 - upgrade to bazel 8
125130
credentials: ${{ secrets.GAR_SERVICE_ACCOUNT }}
126131
architecture: linux-${{ matrix.arch }}
127132
- name: Setup sccache
133+
if: ${{ !matrix.config.continuous-only || inputs.continuous-run }}
128134
uses: protocolbuffers/protobuf-ci/sccache@v4
129135
with:
130136
cache-prefix: linux-release-${{ matrix.arch }}
131137
credentials: ${{ secrets.GAR_SERVICE_ACCOUNT }}
132138
- name: Run tests
139+
if: ${{ !matrix.config.continuous-only || inputs.continuous-run }}
133140
uses: protocolbuffers/protobuf-ci/docker@v4
134141
with:
135142
image: us-docker.pkg.dev/protobuf-build/containers/test/linux/emulation:8.0.1-${{ matrix.arch }}-168f9c9d015a0fa16611e1e9eede796fe9bfbb69

.github/workflows/test_ruby.yml

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -96,15 +96,17 @@ jobs:
9696
PROTOC=/workspace/${{ steps.cross-compile.outputs.protoc }} rake test'
9797
9898
linux-aarch64:
99-
name: Linux aarch64
99+
name: ${{ inputs.continuous-prefix || '' }} Linux aarch64
100100
runs-on: ubuntu-22-4core
101101
steps:
102102
- name: Checkout pending changes
103+
if: ${{ inputs.continuous-run }}
103104
uses: protocolbuffers/protobuf-ci/checkout@v4
104105
with:
105106
ref: ${{ inputs.safe-checkout }}
106107

107108
- name: Cross compile protoc for aarch64
109+
if: ${{ inputs.continuous-run }}
108110
id: cross-compile
109111
uses: protocolbuffers/protobuf-ci/cross-compile-protoc@v4
110112
with:
@@ -113,6 +115,7 @@ jobs:
113115
architecture: linux-aarch64
114116

115117
- name: Run tests
118+
if: ${{ inputs.continuous-run }}
116119
uses: protocolbuffers/protobuf-ci/docker@v4
117120
with:
118121
image: arm64v8/ruby:3.1.4-buster

0 commit comments

Comments
 (0)