From 6331ddbe56a30b230935995d1daf644761f154eb Mon Sep 17 00:00:00 2001 From: Robert Speicher Date: Mon, 22 Jul 2019 11:42:16 -0500 Subject: [PATCH] CE pipelines don't respond to sourced_pipelines We also scope the `rescue`s to only those raised by ActiveRecord, which would have allowed us to spot this logic error sooner. --- db/fixtures/development/14_pipelines.rb | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/db/fixtures/development/14_pipelines.rb b/db/fixtures/development/14_pipelines.rb index ffba934021e..05bda7d3672 100644 --- a/db/fixtures/development/14_pipelines.rb +++ b/db/fixtures/development/14_pipelines.rb @@ -81,7 +81,7 @@ class Gitlab::Seeder::Pipelines @project.repository.commits('master', limit: 4).map do |commit| create_pipeline!(@project, 'master', commit).tap do |pipeline| random_pipeline.tap do |triggered_by_pipeline| - triggered_by_pipeline.sourced_pipelines.create( + triggered_by_pipeline.try(:sourced_pipelines)&.create( source_job: triggered_by_pipeline.builds.all.sample, source_project: triggered_by_pipeline.project, project: pipeline.project, @@ -89,7 +89,7 @@ class Gitlab::Seeder::Pipelines end end end - rescue + rescue ActiveRecord::ActiveRecordError [] end @@ -106,7 +106,7 @@ class Gitlab::Seeder::Pipelines end pipelines.flatten - rescue + rescue ActiveRecord::ActiveRecordError [] end