Skip to content

Commit

Permalink
logging improved
Browse files Browse the repository at this point in the history
  • Loading branch information
benzekrimaha committed May 13, 2024
1 parent fc2e9c0 commit 6e9efd3
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions CountItems/CountManager.js
Original file line number Diff line number Diff line change
Expand Up @@ -99,15 +99,19 @@ class CountManager {
if (metricLevel === 'account' && !this.uniqueAccounts.has(resourceName)) {
this.uniqueAccounts.add(resourceName);
monitoring.metricsCount.inc({ metricLevel : metricLevel});

Check failure on line 101 in CountItems/CountManager.js

View workflow job for this annotation

GitHub Actions / tests

Expected property shorthand

Check failure on line 101 in CountItems/CountManager.js

View workflow job for this annotation

GitHub Actions / tests

Extra space after key 'metricLevel'

Check failure on line 101 in CountItems/CountManager.js

View workflow job for this annotation

GitHub Actions / tests

A space is required before '}'
console.log(`uniqueAccount : ${this.uniqueAccounts}`);

Check failure on line 102 in CountItems/CountManager.js

View workflow job for this annotation

GitHub Actions / tests

Unexpected console statement
}
if (metricLevel === 'location' && !this.uniqueLocations.has(resourceName)) {
this.uniqueLocations.add(resourceName);
monitoring.metricsCount.inc({ metricLevel : metricLevel});

Check failure on line 106 in CountItems/CountManager.js

View workflow job for this annotation

GitHub Actions / tests

Expected property shorthand

Check failure on line 106 in CountItems/CountManager.js

View workflow job for this annotation

GitHub Actions / tests

Extra space after key 'metricLevel'

Check failure on line 106 in CountItems/CountManager.js

View workflow job for this annotation

GitHub Actions / tests

A space is required before '}'
console.log(`uniqueLocation : ${this.uniqueLocations}`);

Check failure on line 107 in CountItems/CountManager.js

View workflow job for this annotation

GitHub Actions / tests

Unexpected console statement
}
if (metricLevel === 'bucket' && !this.uniqueBuckets.has(resourceName)) {
this.uniqueBuckets.add(resourceName);
monitoring.metricsCount.inc({ metricLevel : metricLevel});

Check failure on line 111 in CountItems/CountManager.js

View workflow job for this annotation

GitHub Actions / tests

Expected property shorthand

Check failure on line 111 in CountItems/CountManager.js

View workflow job for this annotation

GitHub Actions / tests

Extra space after key 'metricLevel'
console.log(`uniqueBucket : ${this.uniqueBuckets}`);
}
console.log(`resourceName : ${resourceName}`);
console.log(monitoring.metricsCount);
// resourceName can be the name of bucket, location or account
this.dataMetrics[metricLevel][resourceName] = consolidateDataMetrics(
Expand Down

0 comments on commit 6e9efd3

Please sign in to comment.