acannistra / planet-snowcover

Can we use Planet high-resolution imagery and machine learning to assess snow cover?
https://planet-snowcover.readthedocs.io/en/latest/
MIT License
20 stars 7 forks source link

Modify Docker image on DockerHub to use AWS Metadata Service for credentials #26

Open acannistra opened 4 years ago

acannistra commented 4 years ago

The current workflow for building the Docker image that is used in Sagemaker and in Terraform deployments "bakes in" a set of credentials, which breaks for anyone using the image on infrastructure that isn't ours.

Possible avenues for a fix:

cc: @ajijohn thanks for the report about this.

acannistra commented 4 years ago

Confirmed that aws cli within docker container can access EC2 IAM roles.

acannistra commented 4 years ago

Working on this in #25 [045be20]. Removing dependencies on credentials in dockerfile. This likely changes the tutorial contents' dependency on having a configured profile.