summaryrefslogtreecommitdiff
path: root/lib/api
diff options
context:
space:
mode:
authorKamil Trzciński <ayufan@ayufan.eu>2017-09-05 09:46:03 +0000
committerKamil Trzciński <ayufan@ayufan.eu>2017-09-05 09:46:03 +0000
commit89efaf2aa2d65ae41ce5502059d40fa068be945f (patch)
tree4d36f16d0b211e69f0b6b73fad31b429288ea2d8 /lib/api
parent1aa8b38ce6fef5a6548421238ec3ecc9cf940ba0 (diff)
parent38d9b4d77d85e26f827ff9640243494adc8597ed (diff)
downloadgitlab-ce-89efaf2aa2d65ae41ce5502059d40fa068be945f.tar.gz
Merge branch 'feature/sm/37239-implement-failure_reason-on-ci_builds' into 'master'
Implement `failure_reason` on `ci_builds` Closes #37239 See merge request !13937
Diffstat (limited to 'lib/api')
-rw-r--r--lib/api/commit_statuses.rb2
-rw-r--r--lib/api/runner.rb4
2 files changed, 4 insertions, 2 deletions
diff --git a/lib/api/commit_statuses.rb b/lib/api/commit_statuses.rb
index 6314ea63197..829eef18795 100644
--- a/lib/api/commit_statuses.rb
+++ b/lib/api/commit_statuses.rb
@@ -103,7 +103,7 @@ module API
when 'success'
status.success!
when 'failed'
- status.drop!
+ status.drop!(:api_failure)
when 'canceled'
status.cancel!
else
diff --git a/lib/api/runner.rb b/lib/api/runner.rb
index 11999354594..a3987c560dd 100644
--- a/lib/api/runner.rb
+++ b/lib/api/runner.rb
@@ -114,6 +114,8 @@ module API
requires :id, type: Integer, desc: %q(Job's ID)
optional :trace, type: String, desc: %q(Job's full trace)
optional :state, type: String, desc: %q(Job's status: success, failed)
+ optional :failure_reason, type: String, values: CommitStatus.failure_reasons.keys,
+ desc: %q(Job's failure_reason)
end
put '/:id' do
job = authenticate_job!
@@ -127,7 +129,7 @@ module API
when 'success'
job.success
when 'failed'
- job.drop
+ job.drop(params[:failure_reason] || :unknown_failure)
end
end