-
Notifications
You must be signed in to change notification settings - Fork 332
Add --no-sts to CLI #2855
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add --no-sts to CLI #2855
Conversation
client/python/cli/constants.py
Outdated
| "(Only for S3) The STS endpoint to use when connecting to STS" | ||
| ) | ||
| STS_UNAVAILABLE = ( | ||
| "(Only for S3) Indicates that STS is not available" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| "(Only for S3) Indicates that STS is not available" | |
| "(Only for S3 compatible object stores) Indicates that STS is not available" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It could be used with AWS too. How about (Only for S3) Indicates that Polaris should not use STS (e.g. if STS is not available)?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Message updated. PTAL.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
makes sense, thanks @dimas-b !
Following up on apache#2672, add new `--no-sts` option to CLI to allow configuring `stsUnavailable` in `AwsStorageConfigInfo`
Following up on #2672, add new
--no-stsoption to CLI to allow configuringstsUnavailableinAwsStorageConfigInfo