Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

initial s3 integration #250

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Screek893
Copy link

Use S3 as a storage for reports.

Any S3 compatible bucket is supported.
I used s3fs module, which is a wrapper for boto3.

S3 credentials are configured with env variables (https://s3fs.readthedocs.io/en/latest/index.html#credentials)
for example:

AWS_ACCESS_KEY_ID: 'xxxxxxx'
AWS_SECRET_ACCESS_KEY: 'yyyyyy'
AWS_DEFAULT_REGION: 'region'
AWS_ENDPOINT_URL: '<endpoint_url if it is not AWS S3>'

Add these variables if you want to use s3 as a storage.

STORAGE_TYPE: 's3' # "local" by default
BUCKET_NAME: 'bucket-name'

I am not a python professional. Fell free to review the code and suggest any changes.

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

Successfully merging this pull request may close these issues.

1 participant