fix(seaweedfs): do not swallow NoSuchBucket 404 during quota disable

The quota-disable 404 tolerance treated every 404/405 as an absent quota
extension. SeaweedFS also returns 404 with a NoSuchBucket error body when
the bucket does not exist, so a missing remote bucket with quota 0 was
reported as success and the database quota was updated, leaving CloudStack
and S3 inconsistent. Exclude NoSuchBucket responses from the tolerance so
only the extension-not-available case is swallowed.

Author: chrislusf

plugins/storage/object/seaweedfs/src/main/java/org/apache/cloudstack/storage/datastore/util/SeaweedFSObjectStoreUtil.java

@@ -296,8 +296,15 @@ public static void setBucketQuotaViaS3Extension(String s3Url, String accessKey,
             // available (404/405), tolerate the failure for quota 0 so basic
             // bucket CRUD works on deployments without the extension. A
             // positive quota still requires the extension and must fail.
+            //
+            // Distinguish "extension not available" from "bucket not found":
+            // SeaweedFS returns a standard S3 NoSuchBucket error (with
+            // <Code>NoSuchBucket</Code> in the body) when the bucket does not
+            // exist, which must NOT be swallowed — it indicates CloudStack and
+            // S3 are out of sync.
             if (sizeGiB == 0 && e.getMessage() != null
-                    && (e.getMessage().contains("status 404") || e.getMessage().contains("status 405"))) {
+                    && (e.getMessage().contains("status 404") || e.getMessage().contains("status 405"))
+                    && !e.getMessage().contains("NoSuchBucket")) {
                 org.apache.logging.log4j.LogManager.getLogger(SeaweedFSObjectStoreUtil.class)
                         .warn("SeaweedFS quota extension not available for bucket {}; skipping quota disable (quota is already off by default)", bucketName);
                 return;