openstack
/
openstack
Code Issues Proposed changes

Update git submodules 

* Update placement from branch 'master'
  - Merge "Cast Usage.usage to int"
  - Cast Usage.usage to int
    
    When using mysql (and perhaps postgresql, but not sqlite), the
    func.sum functionality provided in sqlalchemy will return a result
    that is a decimal.Decimal. The json module is unable to cope
    with this. It needs "native" types of int or float.
    
    In the case of Usage we want an int. Since there are two places
    where the data is loaded, but both then directly call to create
    a Usage class, do the casting in the constructor of Usage. A
    unit test of that cast is added.
    
    Since the problem does not show up in sqlite, no functional test
    is added, but patches which follow this one which add integration
    tests using gabbi will fail without this change in place, so if
    they are passing that services as additional verification that
    this is a good change.
    
    Change-Id: I6de32d0d06b6e2427ae0cf52ed24fd0173b74baf
    Closes-Bug: #1817633
This commit is contained in:
Zuul 2019-02-26 18:55:43 +00:00 committed by Gerrit Code Review
parent 60013908c6
commit ec18b8bf9f
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
placement

@ -1 +1 @@
Subproject commit eae9e302e8b516cb8ddd803e8eb4da5ab8104453
Subproject commit 6825b0f140dbedea9a4fb315dd7e943ab292c82c