Skip to content
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_data: Download opendrr-boundaries.dump from S3 #121

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

anthonyfok
Copy link
Member

@anthonyfok anthonyfok commented Jun 17, 2021

  • python/Dockerfile: Install zip and aws
  • sample.env: Add placeholders for AWS credentials

Fixes #116

Please see also prerequisite at OpenDRR/boundaries#21

- python/Dockerfile: Install zip and aws
- sample.env: Add AWS credentials [TODO]

Fixes #116
AWS_ACCESS_KEY_ID=
AWS_SECRET_ACCESS_KEY=
AWS_DEFAULT_REGION=ca-central-1

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How long are these credentials valid for? Are they only available to users with access to the AWS environment?

@anthonyfok anthonyfok self-assigned this Jan 17, 2022
@anthonyfok
Copy link
Member Author

Note to self: We came to the realization recently that GitHub release assets can likely do the job.
My hunch is to just close this without merging, but I'll delay that closing and instead revisit and reevaluate this PR at a later date. Moving it into the backlog for now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Switch from Backblaze B2 to Amazon S3
2 participants