Merge ~ack/maas:1947087-vm-host-api-fix-used-counts into maas:master
Proposed by
Alberto Donato
Status: | Merged |
---|---|
Approved by: | Alberto Donato |
Approved revision: | aaed0938fbdbc635d3c1ff4791cb57b672ff7fd4 |
Merge reported by: | MAAS Lander |
Merged at revision: | not available |
Proposed branch: | ~ack/maas:1947087-vm-host-api-fix-used-counts |
Merge into: | maas:master |
Diff against target: |
63 lines (+41/-2) 2 files modified
src/maasserver/models/tests/test_virtualmachine.py (+33/-0) src/maasserver/models/virtualmachine.py (+8/-2) |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Christian Grabowski | Approve | ||
MAAS Lander | Approve | ||
Review via email: mp+410668@code.launchpad.net |
Commit message
LP:1947087 fix query to get used vmhost resources for the API
The previous aggregation was counting memory and cores multiple times if VMs
had more than one disk because of the join with the VirtualMachineDisk table.
To post a comment you must log in.
UNIT TESTS vm-host- api-fix- used-counts lp:~ack/maas/+git/maas into -b master lp:~maas-committers/maas
-b 1947087-
STATUS: SUCCESS 5d3c1ff4791cb57 b672ff7fd4
COMMIT: aaed0938fbdbc63