| Commit message (Collapse) | Author | Age | Files | Lines |
... | |
| |/ / / /
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
There is a difference in how Swift and radosgw behave when a zero
length write is aborted. Swift returns 499 and doesn't create an object,
but radosgw appears to creates a zero size object. Because of this, new
behaviour which was introduced to prevent zero-size chunks being written
worked for Swift, but appears not to work for the radosgw case.
This patch changes how we detect a zero size chunk so that no request
is made to the backend. We no longer rely on the Swift specific behaviour.
Change-Id: I1a48e2eceaa6b07b8e5bbfd42fef6fd3d91f5834
Closes-bug: 1537721
|
|\ \ \ \ \ |
|
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
read_tenants, write_tenants parameters in set_acls method of
glance_store/driver.Store have empty lists as default values.
It is not recommended approach cause it can lead to some neat
issues.
Moreover, Swift store class rewrites this method and uses None as
default values (which is right). So we need to follow this
approach in base Store class.
Change-Id: Ifa0f0c286dbb822b2d34d03978655d720bb109d4
|
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Change-Id: Ie828a31821491c90e459a0b115aa38e6028b6d47
|
| |/ / / /
|/| | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Removing the unnecessary handling of NotFound exception in
get_size. Earlier we used to ignore more generic exceptions
and needed this, but that logic was changed in commit
a0620e195db0604aaa7c9fd12930c10dbe2e7b0d.
Change-Id: I527ed2748a594adf0a463690ab8df3140f8f8beb
|
| |/ / /
|/| | |
| | | |
| | | | |
Change-Id: Ia4add8d3fd6268da9aafd6617d73d956e4a31bda
|
|\ \ \ \
| |_|/ /
|/| | | |
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
This exception should be handled rather than ignored. Also, we should not
return 0 when it is raised.
Change-Id: Iad017574f034d6e36a02e769225c1adb03adaf4d
|
|\ \ \ \
| | |_|/
| |/| | |
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
In order to use the 'sign-the-data' approach instead of the 'sign-the-
hash' approach for signature verification, the verifier must be
updated with the image data at the same time that the image data is
being read in order to update the checksum.
This patch adds the ability for glance to pass in a verifier object to
the backend drivers when an image is being uploaded, which is given
the image data by the backend drivers, and used to verify the
signature in glance.
The following drivers are supported:
* file
* rbd
* sheepdog
* s3
* vsphere
The swift driver is supported for images that are greater than the
'large_object_size' since glance_store does not read the data for
images that are less than 'large_object_size' (swiftclient does the
reading), and as a result the verifier cannot be updated with the
data. A child patch addresses this issue.
The cinder and http drivers are not supported because they do not have
the 'write' capability.
Partial-Bug: #1516031
Change-Id: I43799e6a4a6643a23769af8d839a2beb4e0ff9bf
|
|\ \ \ \ |
|
| |/ / /
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Replace glance_store.common.utils.exception_to_str() with
encodeutils.exception_to_unicode(). exception_to_unicode() tries more
options to get a nice error message as Unicode from an exception
object. As exception_to_str(), it catches and handles exceptions.
Remove glance_store.common.utils.exception_to_str() and related unit
tests. The function is not part of the store API and Glance does not
use it.
glance already uses exception_to_unicode(): see change
I86008c8adc0c5664f96573c1015cc15e2d06e3e2. By the way, glance did not
use exception_to_str() from glance_store but had its own copy of the
function (which was removed by the mentioned change).
Change-Id: I52cab2e773c257e36d36290f6060811f83f18bb0
|
|\ \ \ \
| |/ / /
|/| | | |
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
We document that the default value of filesystem_store_datadir is
/var/lib/glance/images without ever actually setting that default.
Co-Authored-By: kairat_kushaev <kkushaev@mirantis.com>
Closes-bug: #1501662
Change-Id: If6ab4f727f205c7828f247c7588b1f9bad78d1e8
|
|\ \ \ \ |
|
| | |/ /
| |/| |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
os.popen() is deprecated since python 2.6. Resolved with use of
subprocess module.
Closes-Bug: #1529836
Change-Id: Ie98335b65c41906d66abe07989ab014cf079476e
|
|\ \ \ \ |
|
| | |_|/
| |/| |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
This commit removes common/auth.py file and changes
the way how endpoint for MultiTenant store is found.
Without having custom implementation it's better to
use an unified method 'url_for' that service_catalog
provides.
Co-Authored-By: kairat_kushaev <kkushaev@mirantis.com>
Change-Id: If8d07e378140a940317db1259a58477d7809948e
|
|\ \ \ \ |
|
| |/ / /
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
There are a lot of exceptions that seem to be unused in the
glance_store. This change deprecates all these unused exceptions.
Closes-Bug: 1516020
Change-Id: I58e69ddced3ca7fad0a9df8c7f52f5f5c077a844
|
|\ \ \ \
| |/ / /
|/| | | |
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
This patch changes storelocation format from 'sheepdog://image' to
'sheepdog://addr:port:image'. It add ip and port to the location.
This offers the advantages of:
1.The ip and port represents a sheepdog cluster which contains the
image. The format like 'sheepdog://addr:port:image' can describe image
location accurately.
2.When clone an image in cinder, it need ip and port to decide whether
the image is in the same sheepdog cluster with cinder-backend, this is
very important for us to determine if the image is cloneable.
This patch also change a long data type to int to compatible with
python version 3.x.
Change-Id: Id79487e05e3af2f7b1287476ddbb174ad9de78c8
|
|\ \ \ \ |
|
| | | | |
| | | | |
| | | | |
| | | | | |
Change-Id: I3d5c09908aec664b9f35cca9a49c1b05028027f4
|
|\ \ \ \ \ |
|
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
As mentioned in this mailing list thread[0], we've started a cleanup
process for the drivers in glance_store. In order to guarantee some
level of stability, we need each driver to have an assigned maintainer
that would be the main contact point for things related to such driver.
The S3 driver doesn't have one. Therefore, as advised in the ML thread,
we're marking it as deprecated and prepare it for removal during the N
development cycle.
[0] http://lists.openstack.org/pipermail/openstack-dev/2015-December/081966.html
Change-Id: I139c36c2d591a55f0f91ea32efa963f80c28e444
|
|\ \ \ \ \ \ |
|
| | |_|_|_|/
| |/| | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Removed unreachable code (pylint warning code W0101).
TrivialFix
Change-Id: I0bf22e997c412feabbec3b5bab900a28770cc456
|
|\ \ \ \ \ \ |
|
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Caching is enabled by default from pip version 6.0
More info:
https://testrun.org/tox/latest/config.html#confval-downloadcache=path
https://pip.pypa.io/en/stable/reference/pip_install/#caching
Change-Id: I5a8aecadc2dedca3f34f0e240fc5ba977b2cebb3
|
|\ \ \ \ \ \ \
| | |_|/ / / /
| |/| | | | | |
|
| | |_|/ / /
| |/| | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
As per the latest communication sent to openstack-dev[0], this commit
adds the list of drivers and maintainers to the glance_store
documentation. The drivers listed here are considered supported by the
overall community but mainly by the listed maintainers.
Drivers that currently exist in the source tree that are not listed here
will be marked as deprecated following what was communicated in the
aforementioned thread[0].
[0] http://lists.openstack.org/pipermail/openstack-dev/2015-December/081966.html
Change-Id: I0ae750a8bbd936e421a5221b7a97fef1f0d02750
|
|/ / / / /
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
The gridfs driver was marked as deprecated some releases ago. This patch
removes the deprecated driver from the tree.
Change-Id: I72785eb77447909fc67d42e17d805eaa77cf45a2
|
|\ \ \ \ \ |
|
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Change-Id: Ie59dd20ed5a369b86c9ae7ad160f2ef8ef04b6c6
|
|/ / / / /
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
For more information about this automatic import see:
https://wiki.openstack.org/wiki/Translations/Infrastructure
Change-Id: I4514273730d4569edbc3065675f98279dc69eba5
|
| |_|_|/
|/| | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
When a chunk of size 0 is added, it is immediately deleted. Swift might try to
delete it before its creation is complete, though, leading to a race condition.
Not sending this 0 byte chunk soves this issue.
Co-Authored-By: Stuart McLaren <stuart.mclaren@hp.com>
Change-Id: I2179a1e6e27a636bfca448fa18481eb11e64e6da
Closes-Bug: #1518431
|
|/ / /
| | |
| | |
| | | |
Change-Id: Ideafaa1bbdd5c8c63d29bfaa64c678a24387d7cd
|
|\ \ \ |
|
| |/ /
| | |
| | |
| | | |
Change-Id: I694de56e3927968e179e7e101dfb740c627699a8
|
|\ \ \ |
|
| |/ /
| | |
| | |
| | | |
Change-Id: Icefe1353131f638d10213b47629e79f5ca2f289f
|
|\ \ \ |
|
| |/ /
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
There are few places where duplicate keys are defined in the
dictionary. Removed all such occurrences throughout the code.
TrivialFix
Change-Id: I693b7006330f3f2918bca9fc2cdcb32fdf202179
|
|/ /
| |
| |
| |
| |
| |
| |
| | |
Replace assertEqual(None, *) with assertIsNone in tests to have
more clear messages in case of failure.
Change-Id: If651a433db3e573c3b0b86b2a9f87ab84e65b96b
Closes-bug: #1280522
|
| |
| |
| |
| | |
Change-Id: Iffb215f27d0c7fe71ecbf7b5080682e8155b3231
|
|\ \ |
|
| |/
| |
| |
| | |
Change-Id: I16017e665bbe0f2ac690d01fd3503a0369d432f7
|
|\ \ |
|
| |/
| |
| |
| | |
Change-Id: Idf4fd68514beef1afc90aaa6dd7625c30febfb93
|