singhpk234 commented on code in PR #1506:
URL: https://github.com/apache/polaris/pull/1506#discussion_r2072079148
##########
spec/polaris-management-service.yml:
##########
@@ -938,6 +940,34 @@ components:
format: password
description: Bearer token (input-only)
+ SigV4AuthenticationParameters:
Review Comment:
It is a global but it works for regions **_enabled by default_** not all
regions here is the list of region it works for by default
https://docs.aws.amazon.com/accounts/latest/reference/manage-acct-regions.html
i.e why there is **_We recommend that you use the appropriate AWS STS regional
endpoints_**
Though I agree with @dimas-b that they need to be configured at a different
place rather than here, may be via Polaris configs ? I am not sure if this
today, we had a to deal with a lot of head-aches in EMR-FS for this. Hence
brought out if we are onboarding a new use case on STS we should incorporate
this.
Not a blocker for this pr !
--
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]
