9af46f4688
The value fetched by Quotasv2_detail.get_resources() is ultimately
returned by Reservation.get_total_reservations_map. The total_reserved
value is returned by a sqlalchemy sum(), and is of a type defined by
the database driver in use. PyMySQL returns a Decimal here, which
the api layer serialises to JSON as a string.
Note that if a resource has no reservations then its value will be
defaulted in DbQuotaDriver.get_detailed_tenant_quotas to the integer
value 0. This means that the value is an integer if zero, or a string
otherwise. This causes parsing difficulties for client libraries which
do strict type checking.
Testing note:
This method is already covered by the unit test
TestQuotaDbApi.test_get_reservations_for_resource, which also already
tests the return value and, implicitly, its type. However, it does not
currently fail because the unit tests run with SQLite rather than MySQL.
SQLite returns an int where MySQL returns a Decimal, which does not
trigger the bug.
TestQuotaDbApi.test_get_reservations_for_resource is sufficient to
demonstrate that the patch doesn't regress SQLite. We were not able to
think of a deterministic way to write a Tempest test for this, which
would cover MySQL. Ideally we would have the ability to run the DB test
suites under MySQL and PostreSQL in addition to SQLite using oslo.db's
OpportunisticDbFixture, but the scope of that is larger than this
change.
Closes-Bug: #1918565
Change-Id: Icda0d63f2f5ea72bde423296c76aa46646fa98a8
(cherry picked from commit
|
||
|---|---|---|
| .. | ||
| db | ||
| extensions | ||
| logapi | ||
| plugins | ||
| port | ||
| qos | ||
| README.rst | ||
| __init__.py | ||
| address_scope.py | ||
| agent.py | ||
| auto_allocate.py | ||
| base.py | ||
| common_types.py | ||
| flavor.py | ||
| floatingip.py | ||
| ipam.py | ||
| l3_hamode.py | ||
| l3agent.py | ||
| metering.py | ||
| network.py | ||
| port_forwarding.py | ||
| ports.py | ||
| provisioning_blocks.py | ||
| quota.py | ||
| rbac_db.py | ||
| router.py | ||
| securitygroup.py | ||
| servicetype.py | ||
| stdattrs.py | ||
| subnet.py | ||
| subnetpool.py | ||
| tag.py | ||
| trunk.py | ||
README.rst
Neutron Objects
Directory
This directory is designed to contain all modules which have objects definitions shipped with core Neutron. The files and directories located inside of this directory should follow the guidelines below.
Structure
The Neutron objects tree should have the following structure:
- The expected directory structure is flat, except for the ML2 plugins. All ML2 plugin objects should fall under the plugins subdirectory (i.e. plugins/ml2/gre_allocation).
- Module names should use singular forms for nouns (network.py, not networks.py).