summaryrefslogtreecommitdiff
path: root/lib/gitlab/ci/pipeline/seed/build.rb
diff options
context:
space:
mode:
authorKamil Trzciński <ayufan@ayufan.eu>2019-08-14 11:09:11 +0200
committerKamil Trzciński <ayufan@ayufan.eu>2019-08-14 15:48:16 +0200
commitfcc20fde8346a9c2dd9e79ded2557c8058ac1e22 (patch)
treeb5a61f9ac0a81abf7958cc89819453d4b80bcaad /lib/gitlab/ci/pipeline/seed/build.rb
parentd1e80af6035d6f726cb75dde00b6a6bde256b5a3 (diff)
downloadgitlab-ce-fcc20fde8346a9c2dd9e79ded2557c8058ac1e22.tar.gz
Add `ci_dag_limit_needs` feature flaglimit-amount-of-needs
This makes to limit `needs:` to 5 by default. Allow to increase the limit to 50 with disable of FF.
Diffstat (limited to 'lib/gitlab/ci/pipeline/seed/build.rb')
-rw-r--r--lib/gitlab/ci/pipeline/seed/build.rb20
1 files changed, 19 insertions, 1 deletions
diff --git a/lib/gitlab/ci/pipeline/seed/build.rb b/lib/gitlab/ci/pipeline/seed/build.rb
index 32086735556..b0ce7457926 100644
--- a/lib/gitlab/ci/pipeline/seed/build.rb
+++ b/lib/gitlab/ci/pipeline/seed/build.rb
@@ -9,6 +9,10 @@ module Gitlab
delegate :dig, to: :@attributes
+ # When the `ci_dag_limit_needs` is enabled it uses the lower limit
+ LOW_NEEDS_LIMIT = 5
+ HARD_NEEDS_LIMIT = 50
+
def initialize(pipeline, attributes, previous_stages)
@pipeline = pipeline
@attributes = attributes
@@ -77,9 +81,15 @@ module Gitlab
end
def needs_errors
- return unless Feature.enabled?(:ci_dag_support, @pipeline.project)
return if @needs_attributes.nil?
+ if @needs_attributes.size > max_needs_allowed
+ return [
+ "#{name}: one job can only need #{max_needs_allowed} others, but you have listed #{@needs_attributes.size}. " \
+ "See needs keyword documentation for more details"
+ ]
+ end
+
@needs_attributes.flat_map do |need|
result = @previous_stages.any? do |stage|
stage.seeds_names.include?(need[:name])
@@ -88,6 +98,14 @@ module Gitlab
"#{name}: needs '#{need[:name]}'" unless result
end.compact
end
+
+ def max_needs_allowed
+ if Feature.enabled?(:ci_dag_limit_needs, @project, default_enabled: true)
+ LOW_NEEDS_LIMIT
+ else
+ HARD_NEEDS_LIMIT
+ end
+ end
end
end
end