-
Notifications
You must be signed in to change notification settings - Fork 5
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
Optional assume role in raster-api for external S3 bucket read permissions #56
Conversation
I stamped this with an ominous 'do not merge' message but the changes themselves are non-breaking. The raster api works as expected when the session tokens are removed from the raster-api environment. The issue only comes up if the edl service is running and adding session tokens to the raster api environment. |
e260808
to
1f2ab9e
Compare
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.
Looks good to me! I'm curious where the value for GDAL_HTTP_RETRY_DELAY
comes from. Seems very specific
I took this (and all or the raster-api updates) from the IMPACT eoAPI but I don't know how @sharkinsspatial and @vincentsarago chose that very specific value. I added a note to the tech debt issue #54 that we should try to explain or remove this setting. |
What
DELTA_RASTER_DATA_ACCESS_ROLE_ARN
will be assumed by raster api to obtain credentials with GetObject permissions for internal and external S3 buckets. This is intended to support reading data from external data providers that are willing to add a limited read only bucket policy to the VEDA raster api data access role.Why
The raster-api needs to stream cloud optimized data from externally hosted buckets. This PR adds the methods needed to assume a role with read permissions for external buckets.
How tested