diff options
author | GitLab Bot <gitlab-bot@gitlab.com> | 2020-11-19 11:27:35 +0300 |
---|---|---|
committer | GitLab Bot <gitlab-bot@gitlab.com> | 2020-11-19 11:27:35 +0300 |
commit | 7e9c479f7de77702622631cff2628a9c8dcbc627 (patch) | |
tree | c8f718a08e110ad7e1894510980d2155a6549197 /app/workers/container_expiration_policies | |
parent | e852b0ae16db4052c1c567d9efa4facc81146e88 (diff) |
Add latest changes from gitlab-org/gitlab@13-6-stable-eev13.6.0-rc42
Diffstat (limited to 'app/workers/container_expiration_policies')
-rw-r--r-- | app/workers/container_expiration_policies/cleanup_container_repository_worker.rb | 96 |
1 files changed, 96 insertions, 0 deletions
diff --git a/app/workers/container_expiration_policies/cleanup_container_repository_worker.rb b/app/workers/container_expiration_policies/cleanup_container_repository_worker.rb new file mode 100644 index 00000000000..8c3c2e9e103 --- /dev/null +++ b/app/workers/container_expiration_policies/cleanup_container_repository_worker.rb @@ -0,0 +1,96 @@ +# frozen_string_literal: true + +module ContainerExpirationPolicies + class CleanupContainerRepositoryWorker + include ApplicationWorker + include LimitedCapacity::Worker + include Gitlab::Utils::StrongMemoize + + queue_namespace :container_repository + feature_category :container_registry + urgency :low + worker_resource_boundary :unknown + idempotent! + + def perform_work + return unless throttling_enabled? + return unless container_repository + + log_extra_metadata_on_done(:container_repository_id, container_repository.id) + + unless allowed_to_run?(container_repository) + container_repository.cleanup_unscheduled! + log_extra_metadata_on_done(:cleanup_status, :skipped) + return + end + + result = ContainerExpirationPolicies::CleanupService.new(container_repository) + .execute + log_extra_metadata_on_done(:cleanup_status, result.payload[:cleanup_status]) + end + + def remaining_work_count + cleanup_scheduled_count = ContainerRepository.cleanup_scheduled.count + cleanup_unfinished_count = ContainerRepository.cleanup_unfinished.count + total_count = cleanup_scheduled_count + cleanup_unfinished_count + + log_info( + cleanup_scheduled_count: cleanup_scheduled_count, + cleanup_unfinished_count: cleanup_unfinished_count, + cleanup_total_count: total_count + ) + + total_count + end + + def max_running_jobs + return 0 unless throttling_enabled? + + ::Gitlab::CurrentSettings.current_application_settings.container_registry_expiration_policies_worker_capacity + end + + private + + def allowed_to_run?(container_repository) + return false unless policy&.enabled && policy&.next_run_at + + Time.zone.now + max_cleanup_execution_time.seconds < policy.next_run_at + end + + def throttling_enabled? + Feature.enabled?(:container_registry_expiration_policies_throttling) + end + + def max_cleanup_execution_time + ::Gitlab::CurrentSettings.current_application_settings.container_registry_delete_tags_service_timeout + end + + def policy + project.container_expiration_policy + end + + def project + container_repository&.project + end + + def container_repository + strong_memoize(:container_repository) do + ContainerRepository.transaction do + # rubocop: disable CodeReuse/ActiveRecord + # We need a lock to prevent two workers from picking up the same row + container_repository = ContainerRepository.waiting_for_cleanup + .order(:expiration_policy_cleanup_status, :expiration_policy_started_at) + .limit(1) + .lock('FOR UPDATE SKIP LOCKED') + .first + # rubocop: enable CodeReuse/ActiveRecord + container_repository&.tap(&:cleanup_ongoing!) + end + end + end + + def log_info(extra_structure) + logger.info(structured_payload(extra_structure)) + end + end +end |