Skip to content

Commit 0392748

Browse files
committed
Write PR comment in different step
1 parent 2b17c40 commit 0392748

File tree

1 file changed

+25
-3
lines changed

1 file changed

+25
-3
lines changed

.gitlab/benchmarks.yml

Lines changed: 25 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
variables:
22
BASE_CI_IMAGE: 486234852809.dkr.ecr.us-east-1.amazonaws.com/ci/benchmarking-platform:dd-wasm-js-rewriter
33

4-
benchmarks:
4+
.benchmarks:
55
stage: benchmarks
66
when: on_success
77
tags: ["runner:apm-k8s-tweaked-metal"]
@@ -16,7 +16,6 @@ benchmarks:
1616
- ./steps/run-benchmarks.sh
1717
- ./steps/analyze-results.sh
1818
- "./steps/upload-results-to-s3.sh || :"
19-
- "./steps/post-pr-comment.sh || :"
2019
artifacts:
2120
name: "artifacts"
2221
when: always
@@ -30,5 +29,28 @@ benchmarks:
3029
UPSTREAM_COMMIT_SHA: $CI_COMMIT_SHA # The commit revision the project is built for.
3130
UPSTREAM_PROJECT_ID: $CI_PROJECT_ID # The ID of the current project. This ID is unique across all projects on the GitLab instance.
3231
UPSTREAM_PROJECT_NAME: $CI_PROJECT_NAME # "dd-wasm-js-rewriter"
33-
MAJOR_VERSION: 22
3432

33+
benchmark:
34+
extends: .benchmarks
35+
parallel:
36+
matrix:
37+
- MAJOR_VERSION: 18
38+
- MAJOR_VERSION: 20
39+
- MAJOR_VERSION: 22
40+
41+
benchmarks-pr-comment:
42+
stage: benchmarks-pr-comment
43+
needs: [ benchmark ]
44+
when: on_success
45+
tags: ["arch:amd64"]
46+
image: $BASE_CI_IMAGE
47+
script:
48+
- export ARTIFACTS_DIR="$(pwd)/artifacts" && (mkdir "${ARTIFACTS_DIR}" || :)
49+
- git config --global url."https://gitlab-ci-token:${CI_JOB_TOKEN}@gitlab.ddbuild.io/DataDog/".insteadOf "https://github.com/DataDog/"
50+
- git clone --branch dd-wasm-js-rewriter https://github.com/DataDog/benchmarking-platform platform && cd platform
51+
- ./steps/download-results-from-s3.sh
52+
- ./steps/post-pr-comment.sh
53+
allow_failure: true
54+
variables:
55+
# Gitlab and BP specific env vars. Do not modify.
56+
KUBERNETES_SERVICE_ACCOUNT_OVERWRITE: dd-wasm-js-rewriter

0 commit comments

Comments
 (0)