From ad308386d82a2aa31b431da264a38e8ffa4b99c3 Mon Sep 17 00:00:00 2001
From: Vladimir Istyufeev <me@rcn.me>
Date: Sat, 23 Oct 2021 00:54:12 +0300
Subject: [PATCH] Introduce `linux-docker-benches` (#10085)

* Introduce `linux-docker-benches`

* Add additional info to `node-bench-regression-guard` job's run
---
 substrate/.gitlab-ci.yml | 50 ++++++++++++++++++++++------------------
 1 file changed, 27 insertions(+), 23 deletions(-)

diff --git a/substrate/.gitlab-ci.yml b/substrate/.gitlab-ci.yml
index 6ad2ff1a46..87efc43af2 100644
--- a/substrate/.gitlab-ci.yml
+++ b/substrate/.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