-
Notifications
You must be signed in to change notification settings - Fork 4
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
Write patch extraction metadata directly to STAC API #228
Conversation
@kvantricht , @cbutsko, not ready to be merged until issues above are finished, but the code is ready for review. |
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 cool! Not easy for me to understand what happens just from code. Do we need tests for this functionality? Once this thing is kind of operational, we should sit together to have a strategy for ingesting previous extractions as well.
…ting to authentication code flow #213
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.
Very clean code!! I'll approve and let's see how it goes in practice.
STAC_API_USERNAME
andSTAC_API_PASSWORD
. If these don't exist, authentication will happen via authentication code flow, which will require manual intervention for each batch job, which is not very practicalworldcereal_sentinelx_patch_extractions
. We're in talks with the DevOps team to only give PUT, POST and DELETE permissions to the 'wcextractions' account, so that other users cannot accidentally overwrite the STAC metadata. To be seen if this is possible/bulk_items/
endpoint. DevOps team is aware.