nevzheng opened a new pull request, #12139:
URL: https://github.com/apache/gravitino/pull/12139

   ### What changes were proposed in this pull request?
   
   Add two opt-in AWS KMS integration paths:
   
   - a disposable LocalStack fixture covering key types, aliases, disabled 
keys, and missing keys; and
   - a read-only live AWS `DescribeKey` test.
   
   The README defines fixture variables, least-privilege access, skip/failure 
behavior, cleanup, and LocalStack licensing requirements.
   
   ### Why are the changes needed?
   
   Mocked SDK tests do not validate provider request wiring, aliases, or 
authentication behavior.
   
   Part of #12131. Depends on the AWS provider PR.
   
   ### Does this PR introduce _any_ user-facing change?
   
   No production behavior changes. It adds integration tests and testing 
documentation.
   
   ### How was this patch tested?
   
   - `./gradlew :bundles:aws:test -PskipITs`
   - LocalStack and live paths are explicit opt-ins using the fixtures 
documented in the README.
   
   ### Series
   
   - Epic: #12131
   - Position: 8 of 11
   - Previous: Transit integration coverage
   - Next: Google Cloud KMS integration coverage
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to