Move the 'danger-review' job to the test stage and allow it to fail

Signed-off-by: Rémy Coutable <remy@rymai.me>

1 files changed, 21 insertions, 19 deletions

diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
index e7b28e540a5..afe9da08495 100644
--- a/.gitlab-ci.yml
+++ b/.gitlab-ci.yml
@@ -350,25 +350,6 @@ retrieve-tests-metadata:
     - wget -O $FLAKY_RSPEC_SUITE_REPORT_PATH http://${TESTS_METADATA_S3_BUCKET}.s3.amazonaws.com/$FLAKY_RSPEC_SUITE_REPORT_PATH || rm $FLAKY_RSPEC_SUITE_REPORT_PATH
     - '[[ -f $FLAKY_RSPEC_SUITE_REPORT_PATH ]] || echo "{}" > ${FLAKY_RSPEC_SUITE_REPORT_PATH}'
 
-danger-review:
-  image: registry.gitlab.com/gitlab-org/gitaly/dangercontainer:latest
-  stage: prepare
-  allow_failure: true
-  before_script:
-    - source scripts/utils.sh
-    - retry gem install danger --no-ri --no-rdoc
-  cache: {}
-  only:
-    refs:
-      - branches@gitlab-org/gitlab-ce
-      - branches@gitlab-org/gitlab-ee
-  except:
-    variables:
-      - $CI_COMMIT_REF_NAME =~ /^ce-to-ee-.*/
-  script:
-    - git version
-    - danger --fail-on-errors=true
-
 update-tests-metadata:
   <<: *tests-metadata-state
   <<: *only-canonical-masters
@@ -457,6 +438,27 @@ setup-test-env:
       - config/secrets.yml
       - vendor/gitaly-ruby
 
+danger-review:
+  image: registry.gitlab.com/gitlab-org/gitaly/dangercontainer:latest
+  stage: test
+  allow_failure: true
+  before_script:
+    - source scripts/utils.sh
+    - retry gem install danger --no-ri --no-rdoc
+  cache: {}
+  only:
+    refs:
+      - branches@gitlab-org/gitlab-ce
+      - branches@gitlab-org/gitlab-ee
+  except:
+    refs:
+      - master
+    variables:
+      - $CI_COMMIT_REF_NAME =~ /^ce-to-ee-.*/
+  script:
+    - git version
+    - danger --fail-on-errors=true
+
 rspec-pg 0 30: *rspec-metadata-pg
 rspec-pg 1 30: *rspec-metadata-pg
 rspec-pg 2 30: *rspec-metadata-pg