feat: Add region field to S3Connection and S3Bucket CRDs

CHANGELOG.md

@@ -11,6 +11,7 @@ All notable changes to this project will be documented in this file.
 ### Added
 
 - Aggregate emitted Kubernetes events on the CustomResources ([#318]).
+- Add the region field to the S3Connection CRD ([#331]).
 
 ### Changed
 
@@ -19,6 +20,7 @@ All notable changes to this project will be documented in this file.
 [#318]: https://github.com/stackabletech/commons-operator/pull/318
 [#320]: https://github.com/stackabletech/commons-operator/pull/320
 [#321]: https://github.com/stackabletech/commons-operator/pull/321
+[#331]: https://github.com/stackabletech/commons-operator/pull/331
 
 ## [24.11.1] - 2025-01-09
 

Cargo.toml

@@ -17,7 +17,7 @@ futures = { version = "0.3", features = ["compat"] }
 serde = { version = "1.0", features = ["derive"] }
 serde_json = "1.0"
 snafu = "0.8"
-stackable-operator = { git = "https://github.com/stackabletech/operator-rs.git", tag = "stackable-operator-0.85.0" }
+stackable-operator = { git = "https://github.com/stackabletech/operator-rs.git", tag = "stackable-operator-0.86.1" }
 strum = { version = "0.26", features = ["derive"] }
 tokio = { version = "1.40", features = ["full"] }
 tracing = "0.1"

deploy/helm/commons-operator/crds/crds.yaml

@@ -366,6 +366,32 @@ spec:
                   minimum: 0.0
                   nullable: true
                   type: integer
+                region:
+                  default:
+                    name: us-east-1
+                  description: |-
+                    AWS service API region used by the AWS SDK when using AWS S3 buckets.
+
+                    This defaults to `us-east-1` and can be ignored if not using AWS S3 buckets.
+
+                    NOTE: This is not the bucket region, and is used by the AWS SDK to construct endpoints for various AWS service APIs. It is only useful when using AWS S3 buckets.
+
+                    When using AWS S3 buckets, you can configure optimal AWS service API connections in the following ways: - From **inside** AWS: Use an auto-discovery source (eg: AWS IMDS). - From **outside** AWS, or when IMDS is disabled, explicity set the region name nearest to where the client application is running from.
+                  oneOf:
+                    - required:
+                        - source
+                    - required:
+                        - name
+                  properties:
+                    name:
+                      description: 'An explicit region, eg: eu-central-1'
+                      type: string
+                    source:
+                      description: Defer region detection to an auto-discovery mechanism.
+                      enum:
+                        - AwsImds
+                      type: string
+                  type: object
                 tls:
                   description: Use a TLS connection. If not specified no TLS will be used.
                   nullable: true
@@ -507,6 +533,32 @@ spec:
                           minimum: 0.0
                           nullable: true
                           type: integer
+                        region:
+                          default:
+                            name: us-east-1
+                          description: |-
+                            AWS service API region used by the AWS SDK when using AWS S3 buckets.
+
+                            This defaults to `us-east-1` and can be ignored if not using AWS S3 buckets.
+
+                            NOTE: This is not the bucket region, and is used by the AWS SDK to construct endpoints for various AWS service APIs. It is only useful when using AWS S3 buckets.
+
+                            When using AWS S3 buckets, you can configure optimal AWS service API connections in the following ways: - From **inside** AWS: Use an auto-discovery source (eg: AWS IMDS). - From **outside** AWS, or when IMDS is disabled, explicity set the region name nearest to where the client application is running from.
+                          oneOf:
+                            - required:
+                                - source
+                            - required:
+                                - name
+                          properties:
+                            name:
+                              description: 'An explicit region, eg: eu-central-1'
+                              type: string
+                            source:
+                              description: Defer region detection to an auto-discovery mechanism.
+                              enum:
+                                - AwsImds
+                              type: string
+                          type: object
                         tls:
                           description: Use a TLS connection. If not specified no TLS will be used.
                           nullable: true