Move job image and services nodes to new CI config

2 files changed, 10 insertions, 9 deletions

diff --git a/lib/ci/gitlab_ci_yaml_processor.rb b/lib/ci/gitlab_ci_yaml_processor.rb
index 0704e8f1683..b8d84de2dbe 100644
--- a/lib/ci/gitlab_ci_yaml_processor.rb
+++ b/lib/ci/gitlab_ci_yaml_processor.rb
@@ -120,14 +120,6 @@ module Ci
     end
 
     def validate_job_types!(name, job)
-      if job[:image] && !validate_string(job[:image])
-        raise ValidationError, "#{name} job: image should be a string"
-      end
-
-      if job[:services] && !validate_array_of_strings(job[:services])
-        raise ValidationError, "#{name} job: services should be an array of strings"
-      end
-
       if job[:tags] && !validate_array_of_strings(job[:tags])
         raise ValidationError, "#{name} job: tags parameter should be an array of strings"
       end
diff --git a/lib/gitlab/ci/config/node/job.rb b/lib/gitlab/ci/config/node/job.rb
index 9280412a638..1c28969be14 100644
--- a/lib/gitlab/ci/config/node/job.rb
+++ b/lib/gitlab/ci/config/node/job.rb
@@ -32,7 +32,14 @@ module Gitlab
           node :cache, Cache,
             description: 'Cache definition for this job.'
 
-          helpers :before_script, :script, :stage, :type, :after_script, :cache
+          node :image, Image,
+            description: 'Image that will be used to execute this job.'
+
+          node :services, Services,
+            description: 'Services that will be used to execute this job.'
+
+          helpers :before_script, :script, :stage, :type, :after_script,
+                  :cache, :image, :services
 
           def name
             @metadata[:name]
@@ -48,6 +55,8 @@ module Gitlab
             { name: name,
               before_script: before_script,
               script: script,
+              image: image,
+              services: services,
               stage: stage,
               cache: cache,
               after_script: after_script }