castanhas98 opened a new pull request, #4034:
URL: https://github.com/apache/polaris/pull/4034

   ## Summary
   - add a MinIO integration test matrix that varies `region` and `roleArn` 
independently with `kmsUnavailable=false`
   - assert that only the AWS-shaped `region + roleArn` case fails during 
`loadTable` with the expected KMS-scoping error
   - add a companion test showing the same AWS-shaped config passes when 
`kmsUnavailable=true`
   - refactor table creation/cleanup in the test helper so the failing case 
still cleans up correctly
   
   ## Why
   The existing MinIO coverage exercised normal MinIO vended-credentials 
behavior, but it did not cover the AWS-shaped read-only KMS path that is 
triggered when both `region` and `roleArn` are configured while KMS remains 
enabled.
   
   ## Testing
   - `DOCKER_HOST=unix:///Users/francisco/.colima/default/docker.sock 
TESTCONTAINERS_DOCKER_SOCKET_OVERRIDE=/var/run/docker.sock 
TESTCONTAINERS_HOST_OVERRIDE=127.0.0.1 ./gradlew 
:polaris-runtime-service:intTest --tests 
org.apache.polaris.service.it.RestCatalogMinIOSpecialIT.testCreateTableVendedCredentialsWithAwsShapeTriggersMinioKmsFailure
 --tests 
org.apache.polaris.service.it.RestCatalogMinIOSpecialIT.testCreateTableVendedCredentialsWithAwsShapeAndKmsUnavailablePasses`
   - `./gradlew :polaris-runtime-service:compileIntTestJava`


-- 
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