s3_bucket - do not use default region as location constraint when creating bucket on ceph cluster

Author: abikouo

changelogs/fragments/20250113-s3_bucket-location-constraints-for-ceph-cluster.yml

@@ -0,0 +1,3 @@
+---
+bugfixes:
+  - s3_bucket - Do not use default region as location constraint when creating bucket on ceph cluster (https://github.com/ansible-collections/amazon.aws/issues/2420).

plugins/module_utils/s3.py

@@ -198,3 +198,9 @@ def _list_objects_v2(client, **params):
 def list_bucket_object_keys(client, bucket, prefix=None, max_keys=None, start_after=None):
     response = _list_objects_v2(client, Bucket=bucket, Prefix=prefix, StartAfter=start_after, MaxKeys=max_keys)
     return [c["Key"] for c in response.get("Contents", [])]
+
+
+def get_s3_bucket_location(module):
+    if module.params.get("ceph") is True:
+        return module.params.get("region")
+    return module.region or "us-east-1"

plugins/modules/s3_bucket.py

@@ -563,7 +563,7 @@
 from ansible_collections.amazon.aws.plugins.module_utils.retries import AWSRetry
 from ansible_collections.amazon.aws.plugins.module_utils.s3 import list_bucket_inventory_configurations
 from ansible_collections.amazon.aws.plugins.module_utils.s3 import s3_extra_params
-from ansible_collections.amazon.aws.plugins.module_utils.s3 import validate_bucket_name
+from ansible_collections.amazon.aws.plugins.module_utils.s3 import validate_bucket_name, get_s3_bucket_location
 from ansible_collections.amazon.aws.plugins.module_utils.tagging import ansible_dict_to_boto3_tag_list
 from ansible_collections.amazon.aws.plugins.module_utils.tagging import boto3_tag_list_to_ansible_dict
 
@@ -1232,9 +1232,7 @@ def create_or_update_bucket(s3_client, module: AnsibleAWSModule):
     """
     name = module.params.get("name")
     object_lock_enabled = module.params.get("object_lock_enabled")
-    # default to US Standard region,
-    # note: module.region will also try to pull a default out of the boto3 configs.
-    location = module.region or "us-east-1"
+    location = get_s3_bucket_location(module)
 
     changed = False
     result = {}