s3 backend us-east-1 boto3 workaround

[ninglesby]

Hey I just started playing around with zou, and it is great! I found a little bug that is pretty easy to fix. I have never done a PR before, is this something I should do or is it small enough that it would be easier for someone else to take care of?

I was getting a LocationConstraint Error when trying to use s3. I found this Boto3 Issue , which seems to indicate that "us-east-1" is a special case and should not be used with LocationConstraint.

file_store.py

    try:
        self.bucket.create(
            CreateBucketConfiguration={
                'LocationConstraint': config.region
            }
        )
    except self.s3.meta.client.exceptions.BucketAlreadyOwnedByYou:
        pass

I changed it to

    try:
        if config.region == "us-east-1":
            self.bucket.create()
        else:
            self.bucket.create(
                CreateBucketConfiguration={
                    'LocationConstraint': config.region
                }
            )
    except self.s3.meta.client.exceptions.BucketAlreadyOwnedByYou:
        pass

And that seemed to do the trick.

[frankrousseau]

Hello @ninglesby,

Thank you for the information. And yes the best is that you make a PR to fix this problem.

[frankrousseau]

I made the fix. Can you confirm that it works?

[ninglesby]

It works, Thanks!