diff options
author | Grzegorz Bizon <grzesiek.bizon@gmail.com> | 2018-04-10 15:27:34 +0300 |
---|---|---|
committer | Grzegorz Bizon <grzesiek.bizon@gmail.com> | 2018-04-11 11:52:09 +0300 |
commit | 2ea25cbcaff0ca62b0570a129771f08bca532820 (patch) | |
tree | b3f27851d6c81059ecb769a7ae73b8bd6c7e4767 /lib/gitlab/ci/status | |
parent | 199e31eb24ae314428c2801454f9de676a78496c (diff) |
Revert adding `Empty` extended status to avoid N+1 queries
Diffstat (limited to 'lib/gitlab/ci/status')
-rw-r--r-- | lib/gitlab/ci/status/build/common.rb | 3 | ||||
-rw-r--r-- | lib/gitlab/ci/status/build/empty.rb | 21 | ||||
-rw-r--r-- | lib/gitlab/ci/status/build/factory.rb | 3 |
3 files changed, 3 insertions, 24 deletions
diff --git a/lib/gitlab/ci/status/build/common.rb b/lib/gitlab/ci/status/build/common.rb index 639e88ebf1f..52d2df0a40a 100644 --- a/lib/gitlab/ci/status/build/common.rb +++ b/lib/gitlab/ci/status/build/common.rb @@ -6,7 +6,8 @@ module Gitlab def illustration { image: 'illustrations/skipped-job_empty.svg', - size: 'svg-430' + size: 'svg-430', + title: _('This job does not have a trace.'), } end diff --git a/lib/gitlab/ci/status/build/empty.rb b/lib/gitlab/ci/status/build/empty.rb deleted file mode 100644 index 3e598877602..00000000000 --- a/lib/gitlab/ci/status/build/empty.rb +++ /dev/null @@ -1,21 +0,0 @@ -module Gitlab - module Ci - module Status - module Build - class Empty < Status::Extended - def illustration - { - image: 'illustrations/skipped-job_empty.svg', - size: 'svg-430', - title: _('This job does not have a trace.') - } - end - - def self.matches?(build, user) - !build.has_trace? - end - end - end - end - end -end diff --git a/lib/gitlab/ci/status/build/factory.rb b/lib/gitlab/ci/status/build/factory.rb index 9b3a97a0f0f..2b26ebb45a1 100644 --- a/lib/gitlab/ci/status/build/factory.rb +++ b/lib/gitlab/ci/status/build/factory.rb @@ -9,8 +9,7 @@ module Gitlab Status::Build::Canceled, Status::Build::Created, Status::Build::Pending, - Status::Build::Skipped, - Status::Build::Empty], + Status::Build::Skipped], [Status::Build::Cancelable, Status::Build::Retryable], [Status::Build::Failed], |