Skip to content

Commit e25675c

Browse files
Fix aarch64 acceptance tests (#18135) (#18138)
This commit fixes the aarch64 acceptance tests. With #18079 a new env var was added to aarch64 docker acceptance tests. At that time the fact that the `command` was missing a block scalar `|` was missed. Thus the newlines were not respected and the script for preparing a vm agent was not executed correctly. This resulted in `bundle` command not being found on the test runner. This commit fixes the `command` section to include a block scalar. Additionally we stop waiting on unit tests before starting docker tests as this is unncessary. (cherry picked from commit 1e127fd) Co-authored-by: Cas Donoghue <[email protected]>
1 parent 19d9e13 commit e25675c

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

.buildkite/aarch64_pipeline.yml

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -127,14 +127,15 @@ steps:
127127
- limit: 3
128128

129129
- group: "Acceptance Phase"
130-
depends_on: "testing-phase"
131130
key: "acceptance-phase"
132131
steps:
133132
- label: "Docker [{{matrix}}] flavor acceptance"
134-
command:
133+
command: |
135134
set -euo pipefail
135+
136+
source .buildkite/scripts/common/vm-agent.sh
136137
export ARCH="aarch64"
137-
source .buildkite/scripts/common/vm-agent.sh && ci/docker_acceptance_tests.sh {{matrix}}
138+
ci/docker_acceptance_tests.sh {{matrix}}
138139
retry:
139140
automatic:
140141
- limit: 3

0 commit comments

Comments
 (0)