diff options
author | Grzegorz Bizon <grzegorz@gitlab.com> | 2019-09-04 17:07:44 +0300 |
---|---|---|
committer | Grzegorz Bizon <grzegorz@gitlab.com> | 2019-09-04 17:07:44 +0300 |
commit | 0da3257c35b6c8c527c3a66b26548c134765829f (patch) | |
tree | 08c14142f231b2e8dcb3b97cdf18d10799b8b75f /lib/gitlab | |
parent | d5ca884bdcf9d90ea88ec767cf6cdcedc5204b63 (diff) | |
parent | 0a39a3d4cdaf6ad4a5a031c1ca287cdc555c60ab (diff) |
Merge branch 'persist-needs-error' into 'master'
Persist `needs:` validation as config error
See merge request gitlab-org/gitlab-ce!32648
Diffstat (limited to 'lib/gitlab')
-rw-r--r-- | lib/gitlab/ci/pipeline/chain/helpers.rb | 7 | ||||
-rw-r--r-- | lib/gitlab/ci/pipeline/chain/populate.rb | 2 |
2 files changed, 7 insertions, 2 deletions
diff --git a/lib/gitlab/ci/pipeline/chain/helpers.rb b/lib/gitlab/ci/pipeline/chain/helpers.rb index 6bb3a75291b..8ccb1066575 100644 --- a/lib/gitlab/ci/pipeline/chain/helpers.rb +++ b/lib/gitlab/ci/pipeline/chain/helpers.rb @@ -5,7 +5,12 @@ module Gitlab module Pipeline module Chain module Helpers - def error(message) + def error(message, config_error: false) + if config_error && command.save_incompleted + pipeline.yaml_errors = message + pipeline.drop!(:config_error) + end + pipeline.errors.add(:base, message) end end diff --git a/lib/gitlab/ci/pipeline/chain/populate.rb b/lib/gitlab/ci/pipeline/chain/populate.rb index 65029f5ce7f..13eca5a9d28 100644 --- a/lib/gitlab/ci/pipeline/chain/populate.rb +++ b/lib/gitlab/ci/pipeline/chain/populate.rb @@ -26,7 +26,7 @@ module Gitlab # Gather all runtime build/stage errors # if seeds_errors = pipeline.stage_seeds.flat_map(&:errors).compact.presence - return error(seeds_errors.join("\n")) + return error(seeds_errors.join("\n"), config_error: true) end ## |