DaanHoogland commented on code in PR #8230:
URL: https://github.com/apache/cloudstack/pull/8230#discussion_r1474326835
##########
framework/quota/src/main/java/org/apache/cloudstack/quota/QuotaManagerImpl.java:
##########
@@ -308,15 +303,15 @@ protected List<UsageVO>
getPendingUsageRecordsForQuotaAggregation(AccountVO acco
protected List<QuotaUsageVO>
createQuotaUsagesAccordingToQuotaTariffs(AccountVO account, List<UsageVO>
usageRecords,
Map<Integer, Pair<List<QuotaTariffVO>, Boolean>>
mapQuotaTariffsPerUsageType) {
String accountToString = account.reflectionToString();
- s_logger.info(String.format("Calculating quota usage of [%s] usage
records for account [%s].", usageRecords.size(), accountToString));
+ s_logger.info(String.format("Calculating quota usage of [%s] usage
records for %s.", usageRecords.size(), accountToString));
Review Comment:
minor but this means that in a search in the log the '[]' can no longer be
included and false positive might pop up;
think account `dahn` and account `dahn-admin` or `dahn-south`
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
