You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@cloudstack.apache.org by ra...@apache.org on 2017/02/15 05:14:21 UTC

[2/2] git commit: updated refs/heads/master to 9d8eebf

Merge pull request #1939 from Accelerite/CLOUDSTACK-8886

CLOUDSTACK-8886: Limitations is listUsageRecords output, listUsageRecAs @kansal  is inactive created new branch and raised the PR. This is continuation of PR #858
This closes #858

Problem: Only domainid is returned by usageReports API call. In cloudstack documention it mentions "domain" as being in the usage response. The API should really be returning the domain as account information has both account and accountid.

Fix: Missing setDomainName at the time of creating response.

* pr/1939:
  CLOUDSTACK-8886: Limitations is listUsageRecords output, listUsageRecords does not return domain - Fixed and tests added

Signed-off-by: Rajani Karuturi <ra...@accelerite.com>


Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo
Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/9d8eebf6
Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/9d8eebf6
Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/9d8eebf6

Branch: refs/heads/master
Commit: 9d8eebf68d0750cef7836bfdd5413af9863993e3
Parents: 13bfdd7 f17d27d
Author: Rajani Karuturi <ra...@accelerite.com>
Authored: Wed Feb 15 10:37:50 2017 +0530
Committer: Rajani Karuturi <ra...@accelerite.com>
Committed: Wed Feb 15 10:37:50 2017 +0530

----------------------------------------------------------------------
 .../api/response/UsageRecordResponse.java       |  4 ++
 server/src/com/cloud/api/ApiResponseHelper.java |  1 +
 .../com/cloud/api/ApiResponseHelperTest.java    | 70 ++++++++++++++++----
 3 files changed, 62 insertions(+), 13 deletions(-)
----------------------------------------------------------------------