Skip to content

Support multipart upload when image_size == 0#7

Closed
priteau wants to merge 1 commit intostackhpc/2024.1from
s3-multipart-caracal
Closed

Support multipart upload when image_size == 0#7
priteau wants to merge 1 commit intostackhpc/2024.1from
s3-multipart-caracal

Conversation

@priteau
Copy link
Member

@priteau priteau commented Sep 3, 2025

When doing an image upload of a volume from cinder-volume to glance with an s3 backend, image_size in this context is zero. As a result the check that image_size is larger than s3_store_large_object_size is always negative and glance_store never uses multipart upload.

As a workaround, allow setting s3_store_large_object_size to 0, which forces glance_store to use multipart upload.

Related-Bug: #2043740
Change-Id: I436d56d0489b081822ed55108e10d43cc38512bb (cherry picked from commit e63e0e4)

When doing an image upload of a volume from cinder-volume to glance with
an s3 backend, image_size in this context is zero. As a result the check
that image_size is larger than s3_store_large_object_size is always
negative and glance_store never uses multipart upload.

As a workaround, allow setting s3_store_large_object_size to 0, which
forces glance_store to use multipart upload.

Related-Bug: #2043740
Change-Id: I436d56d0489b081822ed55108e10d43cc38512bb
(cherry picked from commit e63e0e4)
@priteau priteau self-assigned this Sep 3, 2025
@priteau priteau requested a review from a team as a code owner September 3, 2025 10:39
@priteau priteau requested a review from MoteHue September 3, 2025 10:41
@MoteHue
Copy link

MoteHue commented Sep 3, 2025

Duplicate of: #6

@priteau priteau closed this Sep 3, 2025
@priteau
Copy link
Member Author

priteau commented Sep 3, 2025

Duplicate of #6

@priteau priteau marked this as a duplicate of #6 Sep 3, 2025
@priteau priteau deleted the s3-multipart-caracal branch September 3, 2025 10:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants