Follow-up from "Ensure artifacts are moved locally within the filesystem to prevent timeouts"
The following discussion from !2572 (merged) should be addressed:
-
@stanhu started a discussion: (+1 comment) This code is copied from
GitlabUploader. It probably should be split out somehow.
ArtifactUploader had been inherited from GitlabUploader, although, we made it to be inherited from ObjectStoreUploader.
However, We should align this to be inherited from GitlabUploader for the consistency. We had a bug as this consequence.
ArtifactUploader < ObjectStoreUploader < CarrierWave::Uploader::Base
PersonalFileUploader < FileUploader < GitlabUploader < CarrierWave::Uploader::Base
AttachmentUploader < GitlabUploader < CarrierWave::Uploader::Base
AvatarUploader < GitlabUploader < CarrierWave::Uploader::Base
LfsObjectUploader < GitlabUploader < CarrierWave::Uploader::Base