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

/cc @ayufan @stanhu

mentioned in merge request !2572 (merged)

marked this issue as related to #3068 (closed)

@dosuken123 Thanks for creating this issue.