Merge remote-tracking branch 'origin/development/9.4' into w/9.4/feature/CLDSRV-860/monitor-async-await-migration

Author: DarkIsDude

.claude/skills/review-pr/SKILL.md

@@ -79,6 +79,8 @@ Parse `$ARGUMENTS` to extract the repo and PR number:
     - **Test quality** — no `.only()` tests (eslint enforces this),
       assertions match the behavior being tested, `require()`/`import`
       at top of file (never inside `describe` or functions).
+    - **Test prefix** — Name of tests using the `it()` Mocha test function
+      should start with `should`. Only raise this criteria once per file.
 
 4. **Deliver your review:**
 

.github/docker/docker-compose.yaml

@@ -129,7 +129,7 @@ services:
     depends_on:
       - redis
   metadata-standalone:
-    image: ghcr.io/scality/metadata:8.11.0-standalone
+    image: ghcr.io/scality/metadata:8.25.0-standalone
     profiles: ['metadata-standalone']
     network_mode: 'host'
     volumes:

.github/workflows/tests.yaml

@@ -261,6 +261,20 @@ jobs:
           cache-from: type=gha,scope=mongodb
           cache-to: type=gha,mode=max,scope=mongodb
 
+      - name: Install Oras
+        run: |
+          curl -L https://github.com/oras-project/oras/releases/download/v${ORAS_VERSION}/oras_${ORAS_VERSION}_linux_amd64.tar.gz | \
+            tar -xz -C /usr/local/bin oras
+        env:
+          ORAS_VERSION: 1.2.2
+
+      - name: Push dashboards
+        run: |
+          oras push ghcr.io/${{ github.repository }}/${{ github.event.repository.name }}-dashboards:${{ github.sha }} \
+            dashboard.json:application/grafana-dashboard+json \
+            alerts.yaml:application/prometheus-alerts+yaml
+        working-directory: monitoring
+
   multiple-backend:
     runs-on: ubuntu-24.04
     needs: build

lib/Config.js

@@ -545,23 +545,7 @@ function bucketNotifAssert(bucketNotifConfig) {
 }
 
 function parseIntegrityChecks(config) {
-    const integrityChecks = {
-        'bucketPutACL': true,
-        'bucketPutCors': true,
-        'bucketPutEncryption': true,
-        'bucketPutLifecycle': true,
-        'bucketPutNotification': true,
-        'bucketPutObjectLock': true,
-        'bucketPutPolicy': true,
-        'bucketPutReplication': true,
-        'bucketPutVersioning': true,
-        'bucketPutWebsite': true,
-        'multiObjectDelete': true,
-        'objectPutACL': true,
-        'objectPutLegalHold': true,
-        'objectPutTagging': true,
-        'objectPutRetention': true,
-    };
+    const integrityChecks = {};
 
     if (config && config.integrityChecks) {
         for (const method in integrityChecks) {

lib/api/api.js

@@ -301,15 +301,20 @@ const api = {
                     }
 
                     const buff = Buffer.concat(post, bodyLength);
+                    return validateMethodChecksumNoChunking(request, buff, log)
+                        .then(error => {
+                            if (error) {
+                                return next(error);
+                            }
 
-                    const err = validateMethodChecksumNoChunking(request, buff, log);
-                    if (err) {
-                        return next(err);
-                    }
-
-                    // Convert array of post buffers into one string
-                    request.post = buff.toString();
-                    return next(null, userInfo, authorizationResults, streamingV4Params, infos);
+                            // Convert array of post buffers into one string
+                            request.post = buff.toString();
+                            return next(null, userInfo, authorizationResults, streamingV4Params, infos);
+                        })
+                        .catch(error => {
+                            log.error('error validating checksums', { error });
+                            next(error);
+                        });
                 });
                 return undefined;
             },