Automatic merge of gitlab-org/gitlab-ce master

.gitlab-ci.yml

@@ -83,13 +83,15 @@ stages:
 
 .use-pg: &use-pg
   services:
-    - postgres:9.6
-    - redis:alpine
+    - name: postgres:9.6
+      command: ["postgres", "-c", "fsync=off", "-c", "synchronous_commit=off", "-c", "full_page_writes=off"]
+    - name: redis:alpine
 
 .use-pg-10: &use-pg-10
   services:
-    - postgres:10.0
-    - redis:alpine
+    - name: postgres:10.0
+      command: ["postgres", "-c", "fsync=off", "-c", "synchronous_commit=off", "-c", "full_page_writes=off"]
+    - name: redis:alpine
 
 .use-mysql: &use-mysql
   services:

app/services/ci/prepare_build_service.rb

@@ -11,9 +11,11 @@ module Ci
     def execute
       prerequisites.each(&:complete!)
 
-      unless build.enqueue
-        build.drop!(:unmet_prerequisites)
-      end
+      build.enqueue!
+    rescue => e
+      Gitlab::Sentry.track_acceptable_exception(e, extra: { build_id: build.id })
+
+      build.drop(:unmet_prerequisites)
     end
 
     private

spec/services/ci/prepare_build_service_spec.rb

@@ -38,7 +38,21 @@ describe Ci::PrepareBuildService do
         end
 
         it 'drops the build' do
-          expect(build).to receive(:drop!).with(:unmet_prerequisites).once
+          expect(build).to receive(:drop).with(:unmet_prerequisites).once
+
+          subject
+        end
+      end
+
+      context 'prerequisites raise an error' do
+        before do
+          allow(prerequisite).to receive(:complete!).and_raise Kubeclient::HttpError.new(401, 'unauthorized', nil)
+        end
+
+        it 'drops the build and notifies Sentry' do
+          expect(build).to receive(:drop).with(:unmet_prerequisites).once
+          expect(Gitlab::Sentry).to receive(:track_acceptable_exception)
+            .with(instance_of(Kubeclient::HttpError), hash_including(extra: { build_id: build.id }))
 
           subject
         end