CLDSRV-520: remove bucket quota count metrics update from api

scality · Apr 18, 2024 · 5c4d934 · 5c4d934
1 parent d5ea11a
commit 5c4d934
Show file tree

Hide file tree

Showing 3 changed files with 0 additions and 9 deletions.
diff --git a/lib/api/bucketDelete.js b/lib/api/bucketDelete.js
@@ -58,9 +58,6 @@ function bucketDelete(authInfo, request, log, cb) {
                         authInfo,
                         bucket: bucketName,
                     });
-                    if (bucketMD.getQuota()) {
-                        monitoring.numberOfBucketQuota.dec();
-                    }
                     monitoring.promMetrics(
                         'DELETE', bucketName, '204', 'deleteBucket');
                     return cb(null, corsHeaders);

diff --git a/lib/api/bucketDeleteQuota.js b/lib/api/bucketDeleteQuota.js
@@ -23,12 +23,10 @@ function bucketDeleteQuota(authInfo, request, log, callback) {
         requestType: request.apiMethods || 'bucketDeleteQuota',
         request,
     };
-    let originalQuota = 0;
     return waterfall([
         next => standardMetadataValidateBucket(metadataValParams, request.actionImplicitDenies, log,
             (err, bucket) => next(err, bucket)),
         (bucket, next) => {
-            originalQuota = bucket.getQuota();
             bucket.setQuota(0);
             metadata.updateBucket(bucket.getName(), bucket, log, err =>
                 next(err, bucket));
@@ -45,9 +43,6 @@ function bucketDeleteQuota(authInfo, request, log, callback) {
                 'bucketDeleteQuota');
             return callback(err, err.code, corsHeaders);
         } else {
-            if (originalQuota > 0) {
-                monitoring.numberOfBucketQuota.dec();
-            }
             monitoring.promMetrics(
                 'DELETE', bucketName, '204', 'bucketDeleteQuota');
             pushMetric('bucketDeleteQuota', log, {

diff --git a/lib/api/bucketUpdateQuota.js b/lib/api/bucketUpdateQuota.js
@@ -67,7 +67,6 @@ function bucketUpdateQuota(authInfo, request, log, callback) {
                 'updateBucketQuota');
             return callback(err, err.code, corsHeaders);
         } else {
-            monitoring.numberOfBucketQuota.inc();
             monitoring.promMetrics(
                 'PUT', bucketName, '200', 'updateBucketQuota');
             pushMetric('updateBucketQuota', log, {