api: Set network name as part of the network usage response#3234
Conversation
Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>
|
@blueorangutan package |
|
@borisstoyanov a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. |
|
Packaging result: ✔centos6 ✔centos7 ✔debian. JID-2652 |
|
@blueorangutan test |
|
@borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests |
|
Trillian test result (tid-3451)
|
|
@blueorangutan package |
|
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. |
|
Packaging result: ✔centos6 ✔centos7 ✔debian. JID-2771 |
borisstoyanov
left a comment
There was a problem hiding this comment.
LGTM, network name is part of the response as well as the project and any tags if available:
list usagerecords type=5 startdate=2019-05-28 enddate=2019-05-29 includetags=true
{
"description": "network bytes received for Host: 6",
"domain": "ROOT",
"domainid": "aedcba99-8059-11e9-b304-1e00c301071f",
"enddate": "2019-05-29'T'09:59:59+00:00",
"name": "test-isolated",
"networkid": "8c890707-2a84-452f-818c-c15c39c0bb94",
"project": "test-project-fr6",
"projectid": "7fe6e967-e045-48d0-80b3-ed6ed29751c2",
"rawusage": "0",
"startdate": "2019-05-29'T'09:00:00+00:00",
"tags": [
{
"key": "isolated",
"value": "with-project"
}
],
"type": "DomainRouter",
"usage": "0 bytes received",
"usageid": "8a4510fb-b2e8-487f-9b3b-348eab4ddc96",
"usagetype": 5,
"zoneid": "11aabe95-f621-4c04-977c-cbbf033c1797"
}
|
@rhtyd I think this is ready to merge (still tagged [WIP]) |
|
@DaanHoogland we're waiting for @PaulAngus or @andrijapanic 's test confirmation. |
|
LGTM |
Problem: Network name is not part of the network usage response
Root Cause: Code does not set the network name
Solution: Set the network name for network usage type usage records in the API response
Types of changes