Skip to content

took out asserts#16

Open
wgiddens wants to merge 1 commit into
StreetVoice:masterfrom
wgiddens:patch-1
Open

took out asserts#16
wgiddens wants to merge 1 commit into
StreetVoice:masterfrom
wgiddens:patch-1

Conversation

@wgiddens

@wgiddens wgiddens commented Mar 2, 2016

Copy link
Copy Markdown

When running on EC2 the AWS_* variables are set implicitly by IAM. They aren't required.

When running on EC2 the AWS_* variables are set implicitly by IAM. They aren't required.
@tzangms

tzangms commented Mar 20, 2016

Copy link
Copy Markdown
Contributor

Hi @wgiddens

I wan to know more about the AWS_* variables are set implicitly by IAM stuff, could you provide me where the documents is?

I think the asserts should stay, but add some workaround to work on both situation.

@wgiddens

Copy link
Copy Markdown
Author

You can check the boto3 docs here:
http://boto3.readthedocs.org/en/latest/guide/configuration.html#iam-role

You can set up authentication a variety of ways besides the django settings file, but they recommend letting IAM set it for you.

django-storages catches any errors and raises ImproperlyConfigured. Perhaps that is a solution?

@tzangms

tzangms commented Mar 28, 2016

Copy link
Copy Markdown
Contributor

@wgiddens Thanks for this information, I'll figure it out and modify for compatibility.

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.

2 participants