diff options
author | Kamil Trzciński <ayufan@ayufan.eu> | 2018-07-17 13:58:57 +0300 |
---|---|---|
committer | Kamil Trzciński <ayufan@ayufan.eu> | 2018-08-01 13:25:19 +0300 |
commit | 6b0bfda8ac0c2eebfa0e89dd0d37c5fc58160c54 (patch) | |
tree | b209d556ad48786fcd05820f249b113341d7d34d /app/models/ci | |
parent | 6cccf59cb447fed490c5975e8e7fec6d28b2446b (diff) |
Add `runner_unsupported` CI failure
Diffstat (limited to 'app/models/ci')
-rw-r--r-- | app/models/ci/build.rb | 18 |
1 files changed, 13 insertions, 5 deletions
diff --git a/app/models/ci/build.rb b/app/models/ci/build.rb index 35b20bc1e0b..5a6af97af62 100644 --- a/app/models/ci/build.rb +++ b/app/models/ci/build.rb @@ -174,10 +174,6 @@ module Ci end end - before_transition any => [:running] do |build| - build.validates_dependencies! unless Feature.enabled?('ci_disable_validates_dependencies') - end - after_transition pending: :running do |build| build.ensure_metadata.update_timeout_state end @@ -343,6 +339,10 @@ module Ci { trace_sections: true } end + def runner_required_features + %w(variables) + end + def merge_request return @merge_request if defined?(@merge_request) @@ -581,7 +581,9 @@ module Ci options[:dependencies]&.empty? end - def validates_dependencies! + def valid_build_dependencies? + return unless Feature.enabled?('ci_disable_validates_dependencies') + dependencies.each do |dependency| raise MissingDependenciesError unless dependency.valid_dependency? end @@ -594,6 +596,12 @@ module Ci true end + def supported_runner?(features) + runner_required_features.all? do |feature_name| + features[feature_name] + end + end + def hide_secrets(trace) return unless trace |