From 28d83e766a64b52b3e5a1ed6e64aa6fe6797c667 Mon Sep 17 00:00:00 2001
From: Yinling <heyinling@espressif.com>
Date: Mon, 17 Oct 2016 17:03:54 +0800
Subject: [PATCH] fix bug that deploy when test failed:

test report will be a single stage. The result of test report will be calculated from the result of all test jobs in test stage. So it will only deploy when all test job passed.
---
 .gitlab-ci.yml | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
index dd4049358a..5718cc542d 100644
--- a/.gitlab-ci.yml
+++ b/.gitlab-ci.yml
@@ -1,6 +1,7 @@
 stages:
   - build
   - test
+  - test_report
   - deploy
 
 before_script:
@@ -106,13 +107,12 @@ test_build_system:
     - ./make/test_build_system.sh
 
 test_report:
-  stage: deploy
+  stage: test_report
   only:
     - master
     - triggers
   tags:
     - test_report
-  allow_failure: true
   variables:
     LOG_PATH: "$CI_PROJECT_DIR/$CI_BUILD_REF"
     TEST_CASE_FILE_PATH: "$CI_PROJECT_DIR/components/idf_test"
@@ -121,7 +121,7 @@ test_report:
     when: always
     paths:
       - $REPORT_PATH
-    expire_in: 6 mos
+    expire_in: 12 mos
   script:
     # clone test bench
     - git clone $GITLAB_SSH_SERVER/yinling/auto_test_script.git