Introduce linux-docker-benches (#10085)

* Introduce `linux-docker-benches` * Add additional info to `node-bench-regression-guard` job's run
paritytech · Oct 22, 2021 · 969a70d · 969a70d
1 parent 6725823
commit 969a70d
Showing 1 changed file with 27 additions and 23 deletions.
diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
@@ -363,30 +363,34 @@ cargo-check-benches:
     - *rust-info-script
   script:
     - *cargo-check-benches-script
+  tags:
+    - linux-docker-benches
 
-# TODO: re-enable after dedicated bench hosts provisioning 
-# node-bench-regression-guard:
-#   # it's not belong to `build` semantically, but dag jobs can't depend on each other
-#   # within the single stage - https://gitlab.com/gitlab-org/gitlab/-/issues/30632
-#   # more: https://github.com/paritytech/substrate/pull/8519#discussion_r608012402
-#   stage:                           build
-#   <<:                              *docker-env
-#   <<:                              *test-refs-no-trigger-prs-only
-#   needs:
-#     # this is a DAG
-#     - job:                         cargo-check-benches
-#       artifacts:                   true
-#     # this does not like a DAG, just polls the artifact
-#     - project:                     $CI_PROJECT_PATH
-#       job:                         cargo-check-benches
-#       ref:                         master
-#       artifacts:                   true
-#   variables:
-#     CI_IMAGE:                      "paritytech/node-bench-regression-guard:latest"
-#   before_script: [""]
-#   script:
-#     - 'node-bench-regression-guard --reference artifacts/benches/master-*
-#        --compare-with artifacts/benches/$CI_COMMIT_REF_NAME-$CI_COMMIT_SHORT_SHA'
+node-bench-regression-guard:
+  # it's not belong to `build` semantically, but dag jobs can't depend on each other
+  # within the single stage - https://gitlab.com/gitlab-org/gitlab/-/issues/30632
+  # more: https://github.com/paritytech/substrate/pull/8519#discussion_r608012402
+  stage:                           build
+  <<:                              *docker-env
+  <<:                              *test-refs-no-trigger-prs-only
+  needs:
+    # this is a DAG
+    - job:                         cargo-check-benches
+      artifacts:                   true
+    # this does not like a DAG, just polls the artifact
+    - project:                     $CI_PROJECT_PATH
+      job:                         cargo-check-benches
+      ref:                         master
+      artifacts:                   true
+  variables:
+    CI_IMAGE:                      "paritytech/node-bench-regression-guard:latest"
+  before_script: [""]
+  script:
+    - echo "------- IMPORTANT -------"
+    - echo "node-bench-regression-guard depends on the results of a cargo-check-benches job"
+    - echo "In case of this job failure, check your pipeline's cargo-check-benches"
+    - 'node-bench-regression-guard --reference artifacts/benches/master-*
+       --compare-with artifacts/benches/$CI_COMMIT_REF_NAME-$CI_COMMIT_SHORT_SHA'
 
 cargo-check-subkey:
   stage:                           test