Geo: Investigate unhandled uploaders

For each of these uploaders, determine if we need to open an issue to replicate it with Geo:

      DeletedObjectUploader,
      DependencyProxy::FileUploader,
      Packages::Composer::CacheUploader,
      Packages::Debian::ComponentFileUploader,
      Packages::Debian::DistributionReleaseFileUploader,
      Pages::DeploymentUploader

These were surfaced in !59798 (diffs, comment 556327961).

Results

  • DeletedObjectUploader: Used by Ci::DeletedObject. I believe we don't want to replicate this since I believe the files are physically the same files referenced by Ci::JobArtifact.
  • DependencyProxy::FileUploader: We do want to replicate this, and here is the issue #259694 (closed)
  • Pages::DeploymentUploader: We do want to replicate this, and here is the epic &589 (closed)
  • Packages::Debian::ComponentFileUploader: We do want to replicated this, and here is the issue #333611. Note that this feature is not yet released.
  • Packages::Debian::DistributionReleaseFileUploader: We do want to replicated this, and here is the issue #333615 (closed). Note that this feature is not yet released.
  • Packages::Composer::CacheUploader: Might not be needed #328491 (comment 600822092)

References

This spec fails on new Uploader classes, to ensure that Geo support is not forgotten, or is documented as not needed: https://gitlab.com/gitlab-org/gitlab/-/blob/master/ee/spec/uploaders/every_gitlab_uploader_spec.rb

Edited Jul 19, 2025 by Michael Kozono
Assignee Loading
Time tracking Loading