Commit 215e48ac authored by John T Skarbek's avatar John T Skarbek

Removes the Disk IO tag from various queues

All queues where this tag has been removed are items which have been
migrated over into an environment where shared storage is not available
to the workers.  Due to this, it should be safe to remove this tag.

Refer to the following issues for some additional details:

* https://gitlab.com/gitlab-org/gitlab/-/issues/249825
* https://gitlab.com/gitlab-org/gitlab/-/issues/249829
* https://gitlab.com/gitlab-org/gitlab/-/issues/249820
* https://gitlab.com/gitlab-org/gitlab/-/issues/249824
* https://gitlab.com/gitlab-org/gitlab/-/issues/249818
* https://gitlab.com/gitlab-org/gitlab/-/issues/249835
* https://gitlab.com/gitlab-org/gitlab/-/issues/249837
parent a6f7fc24
...@@ -1106,8 +1106,7 @@ ...@@ -1106,8 +1106,7 @@
:resource_boundary: :unknown :resource_boundary: :unknown
:weight: 1 :weight: 1
:idempotent: :idempotent:
:tags: :tags: []
- :requires_disk_io
- :name: pipeline_background:ci_build_trace_chunk_flush - :name: pipeline_background:ci_build_trace_chunk_flush
:feature_category: :continuous_integration :feature_category: :continuous_integration
:has_external_dependencies: :has_external_dependencies:
...@@ -1251,8 +1250,7 @@ ...@@ -1251,8 +1250,7 @@
:resource_boundary: :cpu :resource_boundary: :cpu
:weight: 5 :weight: 5
:idempotent: :idempotent:
:tags: :tags: []
- :requires_disk_io
- :name: pipeline_processing:build_queue - :name: pipeline_processing:build_queue
:feature_category: :continuous_integration :feature_category: :continuous_integration
:has_external_dependencies: :has_external_dependencies:
...@@ -1484,8 +1482,7 @@ ...@@ -1484,8 +1482,7 @@
:resource_boundary: :unknown :resource_boundary: :unknown
:weight: 2 :weight: 2
:idempotent: :idempotent:
:tags: :tags: []
- :requires_disk_io
- :name: ci_delete_objects - :name: ci_delete_objects
:feature_category: :continuous_integration :feature_category: :continuous_integration
:has_external_dependencies: :has_external_dependencies:
...@@ -1645,8 +1642,7 @@ ...@@ -1645,8 +1642,7 @@
:resource_boundary: :unknown :resource_boundary: :unknown
:weight: 1 :weight: 1
:idempotent: :idempotent:
:tags: :tags: []
- :requires_disk_io
- :name: export_csv - :name: export_csv
:feature_category: :issue_tracking :feature_category: :issue_tracking
:has_external_dependencies: :has_external_dependencies:
......
...@@ -4,8 +4,6 @@ class ArchiveTraceWorker # rubocop:disable Scalability/IdempotentWorker ...@@ -4,8 +4,6 @@ class ArchiveTraceWorker # rubocop:disable Scalability/IdempotentWorker
include ApplicationWorker include ApplicationWorker
include PipelineBackgroundQueue include PipelineBackgroundQueue
tags :requires_disk_io
# rubocop: disable CodeReuse/ActiveRecord # rubocop: disable CodeReuse/ActiveRecord
def perform(job_id) def perform(job_id)
Ci::Build.without_archived_trace.find_by(id: job_id).try do |job| Ci::Build.without_archived_trace.find_by(id: job_id).try do |job|
......
...@@ -7,7 +7,6 @@ class BuildFinishedWorker # rubocop:disable Scalability/IdempotentWorker ...@@ -7,7 +7,6 @@ class BuildFinishedWorker # rubocop:disable Scalability/IdempotentWorker
queue_namespace :pipeline_processing queue_namespace :pipeline_processing
urgency :high urgency :high
worker_resource_boundary :cpu worker_resource_boundary :cpu
tags :requires_disk_io
ARCHIVE_TRACES_IN = 2.minutes.freeze ARCHIVE_TRACES_IN = 2.minutes.freeze
......
...@@ -7,7 +7,6 @@ class ChatNotificationWorker # rubocop:disable Scalability/IdempotentWorker ...@@ -7,7 +7,6 @@ class ChatNotificationWorker # rubocop:disable Scalability/IdempotentWorker
sidekiq_options retry: false sidekiq_options retry: false
feature_category :chatops feature_category :chatops
tags :requires_disk_io
urgency :low # Can't be high as it has external dependencies urgency :low # Can't be high as it has external dependencies
weight 2 weight 2
worker_has_external_dependencies! worker_has_external_dependencies!
......
...@@ -4,7 +4,6 @@ class ExpireBuildInstanceArtifactsWorker # rubocop:disable Scalability/Idempoten ...@@ -4,7 +4,6 @@ class ExpireBuildInstanceArtifactsWorker # rubocop:disable Scalability/Idempoten
include ApplicationWorker include ApplicationWorker
feature_category :continuous_integration feature_category :continuous_integration
tags :requires_disk_io
# rubocop: disable CodeReuse/ActiveRecord # rubocop: disable CodeReuse/ActiveRecord
def perform(build_id) def perform(build_id)
......
...@@ -626,8 +626,7 @@ ...@@ -626,8 +626,7 @@
:resource_boundary: :unknown :resource_boundary: :unknown
:weight: 2 :weight: 2
:idempotent: :idempotent:
:tags: :tags: []
- :requires_disk_io
- :name: security_scans:store_security_scans - :name: security_scans:store_security_scans
:feature_category: :static_application_security_testing :feature_category: :static_application_security_testing
:has_external_dependencies: :has_external_dependencies:
...@@ -635,8 +634,7 @@ ...@@ -635,8 +634,7 @@
:resource_boundary: :unknown :resource_boundary: :unknown
:weight: 2 :weight: 2
:idempotent: :idempotent:
:tags: :tags: []
- :requires_disk_io
- :name: security_scans:sync_security_reports_to_report_approval_rules - :name: security_scans:sync_security_reports_to_report_approval_rules
:feature_category: :static_application_security_testing :feature_category: :static_application_security_testing
:has_external_dependencies: :has_external_dependencies:
...@@ -908,8 +906,7 @@ ...@@ -908,8 +906,7 @@
:resource_boundary: :unknown :resource_boundary: :unknown
:weight: 1 :weight: 1
:idempotent: true :idempotent: true
:tags: :tags: []
- :requires_disk_io
- :name: security_auto_fix - :name: security_auto_fix
:feature_category: :vulnerability_management :feature_category: :vulnerability_management
:has_external_dependencies: :has_external_dependencies:
......
...@@ -6,7 +6,6 @@ module RequirementsManagement ...@@ -6,7 +6,6 @@ module RequirementsManagement
feature_category :requirements_management feature_category :requirements_management
idempotent! idempotent!
tags :requires_disk_io
def perform(build_id) def perform(build_id)
::Ci::Build.find_by_id(build_id).try do |build| ::Ci::Build.find_by_id(build_id).try do |build|
......
...@@ -6,8 +6,6 @@ class StoreSecurityReportsWorker # rubocop:disable Scalability/IdempotentWorker ...@@ -6,8 +6,6 @@ class StoreSecurityReportsWorker # rubocop:disable Scalability/IdempotentWorker
include ApplicationWorker include ApplicationWorker
include SecurityScansQueue include SecurityScansQueue
tags :requires_disk_io
def perform(pipeline_id) def perform(pipeline_id)
Ci::Pipeline.find(pipeline_id).try do |pipeline| Ci::Pipeline.find(pipeline_id).try do |pipeline|
break unless pipeline.project.can_store_security_reports? break unless pipeline.project.can_store_security_reports?
......
...@@ -4,8 +4,6 @@ class StoreSecurityScansWorker # rubocop:disable Scalability/IdempotentWorker ...@@ -4,8 +4,6 @@ class StoreSecurityScansWorker # rubocop:disable Scalability/IdempotentWorker
include ApplicationWorker include ApplicationWorker
include SecurityScansQueue include SecurityScansQueue
tags :requires_disk_io
# rubocop: disable CodeReuse/ActiveRecord # rubocop: disable CodeReuse/ActiveRecord
def perform(*) def perform(*)
# no-op # no-op
......
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