Skip to content

feat(aws-s3tables-alpha): add storage class configuration support #37339

Open
olenarostotskyy wants to merge 14 commits intoaws:mainfrom
olenarostotskyy:olrostob/s3tables-metrics-config
Open

feat(aws-s3tables-alpha): add storage class configuration support #37339
olenarostotskyy wants to merge 14 commits intoaws:mainfrom
olenarostotskyy:olrostob/s3tables-metrics-config

Conversation

@olenarostotskyy
Copy link
Copy Markdown
Contributor

@olenarostotskyy olenarostotskyy commented Mar 24, 2026

Reason for this change

S3 Tables now supports storage class configuration (STANDARD and INTELLIGENT_TIERING) at both the table bucket and table level. This adds CDK support for this feature.

Description of changes

  • Added StorageClass enum with STANDARD and INTELLIGENT_TIERING options
  • Added storageClass property to TableBucketProps for configuring bucket-level storage class
  • Added storageClass property to TableProps for configuring table-level storage class (overrides bucket default)
  • Tables inherit storage class from their parent bucket when not explicitly specified
  • Updated README with documentation and examples
  • Updated rosetta fixture

Describe any new or updated permissions being added

None

Description of how you validated changes

  • Added unit tests for TableBucket with STANDARD, INTELLIGENT_TIERING, and no storage class
  • Added unit tests for Table with STANDARD, INTELLIGENT_TIERING, and no storage class
  • Added integration tests covering:
    • Bucket with STANDARD storage class
    • Bucket with INTELLIGENT_TIERING storage class
    • Table overriding bucket's storage class
    • Table inheriting bucket's storage class
    • Reverse override (bucket INTELLIGENT_TIERING, table STANDARD)
    • Default behavior (no storage class specified)

Checklist


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

Copy link
Copy Markdown
Collaborator

@aws-cdk-automation aws-cdk-automation left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The pull request linter fails with the following errors:

❌ The title scope of the pull request should omit 'aws-' from the name of modified packages. Use 's3tables-alpha' instead of 'aws-s3tables-alpha'.

If you believe this pull request should receive an exemption, please comment and provide a justification. A comment requesting an exemption should contain the text Exemption Request. Additionally, if clarification is needed, add Clarification Request to a comment.

@olenarostotskyy olenarostotskyy changed the title feat(aws-s3tables-alpha) add storage class configuration support feat(aws-s3tables-alpha): add storage class configuration support Mar 24, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 24, 2026

⚠️ Experimental Feature: This security report is currently in experimental phase. Results may include false positives and the rules are being actively refined.
This security report is NOT a review blocker. Please try merge from main to avoid findings unrelated to the PR.


TestsPassed ✅SkippedFailed
Security Guardian Results168 ran168 passed
TestResult
No test annotations available

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 24, 2026

⚠️ Experimental Feature: This security report is currently in experimental phase. Results may include false positives and the rules are being actively refined.
This security report is NOT a review blocker. Please try merge from main to avoid findings unrelated to the PR.


TestsPassed ✅SkippedFailed
Security Guardian Results with resolved templates168 ran168 passed
TestResult
No test annotations available

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

beginning-contributor [Pilot] contributed between 0-2 PRs to the CDK p2

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants