Merge branch 'fix-pipeline-schedule-owner-is-nil' into 'master'

Fix pipeline schedule when owner is nil

Closes gitlab-com/gl-infra/production#805 and #63086

See merge request gitlab-org/gitlab-ce!29477
This commit is contained in:
Stan Hu 2019-06-11 16:06:07 +00:00
commit 74850f1f8e
3 changed files with 14 additions and 1 deletions

View File

@ -7,7 +7,7 @@ module Ci
# Otherwise, multiple pipelines could be created in a short interval.
schedule.schedule_next_run!
RunPipelineScheduleWorker.perform_async(schedule.id, schedule.owner.id)
RunPipelineScheduleWorker.perform_async(schedule.id, schedule.owner&.id)
end
end
end

View File

@ -0,0 +1,5 @@
---
title: Fix pipeline schedules when owner is nil
merge_request:
author:
type: fixed

View File

@ -24,5 +24,13 @@ describe Ci::PipelineScheduleService do
subject
end
context 'when owner is nil' do
let(:schedule) { create(:ci_pipeline_schedule, project: project, owner: nil) }
it 'does not raise an error' do
expect { subject }.not_to raise_error
end
end
end
end