Skip to content

[16.10] Move SyncServiceTokenWorker to CC namespace

Aleksei Lipniagov requested to merge 439738-step-2 into master

What does this MR do and why?

We want to move SyncServiceTokenWorker from Ai:: to CloudConnector:: namespace.

This worker is only used for Self-Managed. To keep GitLab fully backward-compatible and forward-compatible, as described in https://docs.gitlab.com/ee/development/sidekiq/compatibility_across_updates.html, we need to do it in three steps (3 milestones):

  • 16.9: Introduce an identical worker with a new name. Keep the old named worker in 16.9 in case something is already scheduled. It is done in !143203 (merged).
  • 16.10: Replace the references of the Ai::SyncServiceTokenWorker with CloudConnector::SyncServiceTokenWorker. This can't be done in the same milestone as previous step (16.9) to avoid the situation when the new application code schedules a job that will be executed on the Sidekiq with the old application code (will result in unknown const). This MR.
  • 16.11: Drop the old name. Future MR.

MR acceptance checklist

Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

How to set up and validate locally

This is a no-op change. We rely on CI, reviews, and can refer to !143203 (merged) to test individual classes.

Related to #439738 (closed)

Merge request reports