openstack
/
glance_store
Code Issues Proposed changes
You cannot select more than 25 topics Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.
master
stable/yoga
stable/zed
stable/2023.1
stable/xena
stable/wallaby
stable/victoria
stable/ussuri
stable/train
stable/rocky
stable/stein
4.5.0
4.1.1
3.0.1
4.3.1
4.4.0
xena-em
4.3.0
4.2.0
queens-eol
wallaby-em
4.1.0
pike-eol
4.0.1
2.5.1
2.7.1
4.0.0
victoria-em
3.0.0
2.3.1
ussuri-em
2.7.0
2.0.1
2.6.0
ocata-eol
train-em
2.5.0
2.4.0
stein-em
2.3.0
2.2.0
2.1.0
0.28.1
2.0.0
rocky-em
0.26.2
1.1.0
queens-em
1.0.1
1.0.0
0.29.1
0.29.0
pike-em
ocata-em
0.28.0
0.27.0
0.26.1
0.26.0
0.25.0
0.24.0
0.23.0
newton-eol
0.22.0
0.21.0
mitaka-eol
liberty-eol
0.20.0
0.19.0
0.9.3
0.18.0
0.17.0
0.16.0
0.15.0
0.14.0
kilo-eol
0.13.1
0.9.2
0.13.0
0.12.0
0.11.0
0.10.0
0.9.1
0.9.0
0.8.0
0.7.1
0.7.0
0.6.0
0.5.0
0.4.0
0.3.0
0.2.0
0.1.12
0.1.11
0.1.10
0.1.9
0.1.8
0.1.7
0.1.6
0.1.5
0.1.4
0.1.3
0.1.2
0.1.1
0.1.0
0.0.1a2
0.0.1a1
Branches Tags
${ item.name }
Create tag ${ searchTerm }
Create branch ${ searchTerm }
from 'stable/stein'
${ noResults }
glance_store/glance_store
History
Alfredo Moralejo 053614be14 Do not include ETag when puting manifest in chunked uploads 
While testing glance with Ceph Rados Gateway using latest Ceph release
(Nautilus), i've found that glance fails to upload the manifest using
dynamic large objects mode because of the value used in ETag request.

This issue has been reported to Ceph as it seems related to some recent
change in radosgw code [1].

However, checking at the upload workflow used by glance and comparing
to Swift documentation [2], I wonder if adding the etag is actually
providing any value. In the Swift the ETag header is used to validate
integrity when uploading chunks, not the manifest while glance is doing
exactly the oposite, not sending the etag in the chunks (I guess to
avoid checksuming big images, which makes sense to me) and sending it
when puting the manifest.

This patch is removing the etag header when sending the PUT request for
the manifest in chunked uploads.

[1] https://tracker.ceph.com/issues/39160
[2] https://docs.openstack.org/swift/latest/api/large_objects.html#dynamic-large-objects

Closes-bug: #1824533
Change-Id: I0b563dfcdc30026669fb089c82db8c3df7edc808
(cherry picked from commit ed356fc6b4)
 		4 years ago
..
_drivers Do not include ETag when puting manifest in chunked uploads 4 years ago
common Use oslo_utils.encodeutils.exception_to_unicode() 7 years ago
locale Imported Translations from Zanata 5 years ago
tests Do not include ETag when puting manifest in chunked uploads 4 years ago
__init__.py Enable multi store support for glance 5 years ago
backend.py Update deprecation notices 5 years ago
capabilities.py Cleanup i18n marker functions to match Oslo usage 7 years ago
driver.py Multihash Implementation for Glance 5 years ago
exceptions.py Multihash Implementation for Glance 5 years ago
i18n.py Update oslo libraries 8 years ago
location.py Merge "Address multi store nits" 5 years ago
multi_backend.py Provision to add new config options in sample config file 5 years ago