Read about our upcoming Code of Conduct on this issue

Commit f9eebd26 authored by Shinya Maeda's avatar Shinya Maeda
Browse files

Revert unnecessary code running_or_pending_build_count removal

parent 5988987b2c42
......@@ -359,6 +359,7 @@ def execute_hooks
project.execute_hooks(build_data.dup, :job_hooks)
project.execute_services(build_data.dup, :job_hooks)
PagesService.new(build_data).execute
project.running_or_pending_build_count(force: true)
end
def artifacts_metadata_entry(path, **options)
......
......@@ -1494,6 +1494,12 @@ def remove_import_jid
update_column(:import_jid, nil)
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
def pipeline_status
@pipeline_status ||= Gitlab::Cache::Ci::ProjectPipelineStatus.load_for_project(self)
......
......@@ -31,5 +31,7 @@
page.within('.erased') do
expect(page).to have_content('Job has been erased')
end
expect(build.project.running_or_pending_build_count).to eq(build.project.builds.running_or_pending.count(:all))
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