Commit c428aaac authored by Shinya Maeda's avatar Shinya Maeda

Revert unnecessary code running_or_pending_build_count removal

parent 60664373
...@@ -359,6 +359,7 @@ module Ci ...@@ -359,6 +359,7 @@ module Ci
project.execute_hooks(build_data.dup, :job_hooks) project.execute_hooks(build_data.dup, :job_hooks)
project.execute_services(build_data.dup, :job_hooks) project.execute_services(build_data.dup, :job_hooks)
PagesService.new(build_data).execute PagesService.new(build_data).execute
project.running_or_pending_build_count(force: true)
end end
def artifacts_metadata_entry(path, **options) def artifacts_metadata_entry(path, **options)
......
...@@ -1494,6 +1494,12 @@ class Project < ActiveRecord::Base ...@@ -1494,6 +1494,12 @@ class Project < ActiveRecord::Base
update_column(:import_jid, nil) update_column(:import_jid, nil)
end end
def running_or_pending_build_count(force: false)
Rails.cache.fetch(['projects', id, 'running_or_pending_build_count'], force: force) do
builds.running_or_pending.count(:all)
end
end
# Lazy loading of the `pipeline_status` attribute # Lazy loading of the `pipeline_status` attribute
def pipeline_status def pipeline_status
@pipeline_status ||= Gitlab::Cache::Ci::ProjectPipelineStatus.load_for_project(self) @pipeline_status ||= Gitlab::Cache::Ci::ProjectPipelineStatus.load_for_project(self)
......
...@@ -31,5 +31,7 @@ describe 'User browses a job', :js do ...@@ -31,5 +31,7 @@ describe 'User browses a job', :js do
page.within('.erased') do page.within('.erased') do
expect(page).to have_content('Job has been erased') expect(page).to have_content('Job has been erased')
end end
expect(build.project.running_or_pending_build_count).to eq(build.project.builds.running_or_pending.count(:all))
end end
end end
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment