diff options
-rw-r--r-- | lib/ci/gitlab_ci_yaml_processor.rb | 30 | ||||
-rw-r--r-- | lib/gitlab/ci/config.rb | 2 |
2 files changed, 3 insertions, 29 deletions
diff --git a/lib/ci/gitlab_ci_yaml_processor.rb b/lib/ci/gitlab_ci_yaml_processor.rb index 33492775fe1..01ef13df57a 100644 --- a/lib/ci/gitlab_ci_yaml_processor.rb +++ b/lib/ci/gitlab_ci_yaml_processor.rb @@ -13,7 +13,7 @@ module Ci ALLOWED_CACHE_KEYS = [:key, :untracked, :paths] ALLOWED_ARTIFACTS_KEYS = [:name, :untracked, :paths, :when, :expire_in] - attr_reader :path, :cache + attr_reader :path, :cache, :stages def initialize(config, path = nil) @ci_config = Gitlab::Ci::Config.new(config) @@ -44,10 +44,6 @@ module Ci end end - def stages - @stages - end - def global_variables @variables end @@ -68,8 +64,8 @@ module Ci @services = @ci_config.services @variables = @ci_config.variables @stages = @ci_config.stages + @cache = @ci_config.cache - @cache = @config[:cache] @jobs = {} @config.except!(*ALLOWED_YAML_KEYS) @@ -116,8 +112,6 @@ module Ci end def validate! - validate_global_cache! if @cache - @jobs.each do |name, job| validate_job!(name, job) end @@ -125,26 +119,6 @@ module Ci true end - def validate_global_cache! - @cache.keys.each do |key| - unless ALLOWED_CACHE_KEYS.include?(key) - raise ValidationError, "Cache config has unknown parameter: #{key}" - end - end - - if @cache[:key] && !validate_string(@cache[:key]) - raise ValidationError, "cache:key parameter should be a string" - end - - if @cache[:untracked] && !validate_boolean(@cache[:untracked]) - raise ValidationError, "cache:untracked parameter should be an boolean" - end - - if @cache[:paths] && !validate_array_of_strings(@cache[:paths]) - raise ValidationError, "cache:paths parameter should be an array of strings" - end - end - def validate_job!(name, job) validate_job_name!(name) validate_job_keys!(name, job) diff --git a/lib/gitlab/ci/config.rb b/lib/gitlab/ci/config.rb index 61a2d2069a3..e6cc1529760 100644 --- a/lib/gitlab/ci/config.rb +++ b/lib/gitlab/ci/config.rb @@ -8,7 +8,7 @@ module Gitlab # Temporary delegations that should be removed after refactoring # delegate :before_script, :image, :services, :after_script, :variables, - :stages, to: :@global + :stages, :cache, to: :@global def initialize(config) @config = Loader.new(config).load! |