dennishuo commented on code in PR #2672: URL: https://github.com/apache/polaris/pull/2672#discussion_r2377639090
########## CHANGELOG.md: ########## @@ -40,6 +40,8 @@ request adding CHANGELOG notes for breaking (!) changes and possibly other secti - Added a Management API endpoint to reset principal credentials, controlled by the `ENABLE_CREDENTIAL_RESET` (default: true) feature flag. +- Added support for S3-compatible storage that does not have STS (use `stsUavailable: true` in catalog storage configuration) Review Comment: Could be worth expanding this sentence with a bit more of what's in the yaml for easy reference if people are reading the notes without wanting to read the full spec change -- specifically to clarify that it disables credential vending rather than vending some kind of root credentials after skipping STS subscoping. How does the behavior compare to setting `SKIP_CREDENTIAL_SUBSCOPING_INDIRECTION=true`? -- 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]
