Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Added recursive fetch of child domains for listUsageRecords API call #4717

Merged
merged 10 commits into from Apr 10, 2021
Merged

Added recursive fetch of child domains for listUsageRecords API call #4717

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
0da721a
Added recursive fetch of domains for listUsageRecords API call
Spaceman1984 Feb 22, 2021
e66d92b
Added security check for domain admin
Spaceman1984 Feb 22, 2021
7a70230
Review changes
Spaceman1984 Feb 26, 2021
9270df4
Changed account check
Spaceman1984 Mar 1, 2021
5d43c1c
Removed unused var
Spaceman1984 Mar 1, 2021
71c7c71
Adding recursive check back
Spaceman1984 Mar 1, 2021
19fd612
Changed logic around setting the account id
Spaceman1984 Mar 1, 2021
a45fcee
Fixed null account issue
Spaceman1984 Mar 16, 2021
83c9b0a
Refactored code
Spaceman1984 Mar 17, 2021
80860ee
Added message regarding account id
Spaceman1984 Mar 17, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
8 changes: 8 additions & 0 deletions api/src/main/java/org/apache/cloudstack/api/command/admin/usage/ListUsageRecordsCmd.java
View file
Open in desktop
Expand Up @@ -85,6 +85,10 @@ public class ListUsageRecordsCmd extends BaseListCmd {
@Parameter(name = ApiConstants.OLD_FORMAT, type = CommandType.BOOLEAN, description = "Flag to enable description rendered in old format which uses internal database IDs instead of UUIDs. False by default.")
private Boolean oldFormat;

@Parameter(name = ApiConstants.IS_RECURSIVE, type = CommandType.BOOLEAN,
description = "Specify if usage records should be fetched recursively per domain.")
Spaceman1984 marked this conversation as resolved.
Show resolved Hide resolved
private Boolean recursive = false;

/////////////////////////////////////////////////////
/////////////////// Accessors ///////////////////////
/////////////////////////////////////////////////////
Expand Down Expand Up @@ -153,6 +157,10 @@ public boolean getOldFormat() {
return oldFormat != null && oldFormat;
}

public Boolean isRecursive() {
return recursive;
}

/////////////////////////////////////////////////////
/////////////// API Implementation///////////////////
/////////////////////////////////////////////////////
Expand Down
12 changes: 11 additions & 1 deletion server/src/main/java/com/cloud/usage/UsageServiceImpl.java
View file
Open in desktop
Expand Up @@ -248,7 +248,17 @@ public Pair<List<? extends Usage>, Integer> getUsageRecords(ListUsageRecordsCmd
sc.addAnd("domainId", SearchCriteria.Op.IN, domainIds.toArray());
}

if (domainId != null) {
if (cmd.isRecursive() && domainId != null){
Spaceman1984 marked this conversation as resolved.
Show resolved Hide resolved
SearchCriteria<DomainVO> sdc = _domainDao.createSearchCriteria();
sdc.addOr("path", SearchCriteria.Op.LIKE, _domainDao.findById(domainId).getPath() + "%");
List<DomainVO> domains = _domainDao.search(sdc, null);
List<Long> domainIds = new ArrayList<Long>();
for (DomainVO domain : domains)
domainIds.add(domain.getId());
sc.addAnd("domainId", SearchCriteria.Op.IN, domainIds.toArray());
}

if (!cmd.isRecursive() && domainId != null) {
Spaceman1984 marked this conversation as resolved.
Show resolved Hide resolved
sc.addAnd("domainId", SearchCriteria.Op.EQ, domainId);
}

Expand Down