Skip to content

test_bucket_create_exists should fail on recreate#9

Merged
alfredodeza merged 1 commit intoceph:masterfrom
gaul:bucket-create-exists
Dec 4, 2014
Merged

test_bucket_create_exists should fail on recreate#9
alfredodeza merged 1 commit intoceph:masterfrom
gaul:bucket-create-exists

Conversation

@gaul
Copy link
Copy Markdown
Contributor

@gaul gaul commented Jul 31, 2014

AWS S3 has two behaviors for recreating a bucket depending if you use
the us-standard or another region:

>>> bucket = conn.create_bucket('gaul-default', location=Location.DEFAULT)
>>> bucket = conn.create_bucket('gaul-default', location=Location.DEFAULT)
>>> bucket = conn.create_bucket('gaul-uswest', location=Location.USWest)
>>> bucket = conn.create_bucket('gaul-uswest', location=Location.USWest)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/lib/python2.7/dist-packages/boto/s3/connection.py", line 499, in create_bucket
    response.status, response.reason, body)
boto.exception.S3CreateError: S3CreateError: 409 Conflict
<?xml version="1.0" encoding="UTF-8"?>
<Error><Code>BucketAlreadyOwnedByYou</Code><Message>Your previous request to create the named bucket succeeded and you already own it.</Message><BucketName>gaul-uswest</BucketName><RequestId>24B6DC3170365CD7</RequestId><HostId>hUynMTyqc9WZFxAJ2RFK6P7BqmmeHHlMl9xL2NOy56xBUnOZCAlHqGvtMeGeAfVs</HostId></Error>

Additional discussion:

https://issues.apache.org/jira/browse/JCLOUDS-334

AWS S3 has two behaviors for recreating a bucket depending if you use
the us-standard or another region:

>>> bucket = conn.create_bucket('gaul-default', location=Location.DEFAULT)
>>> bucket = conn.create_bucket('gaul-default', location=Location.DEFAULT)
>>> bucket = conn.create_bucket('gaul-uswest', location=Location.USWest)
>>> bucket = conn.create_bucket('gaul-uswest', location=Location.USWest)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/lib/python2.7/dist-packages/boto/s3/connection.py", line 499, in create_bucket
    response.status, response.reason, body)
boto.exception.S3CreateError: S3CreateError: 409 Conflict
<?xml version="1.0" encoding="UTF-8"?>
<Error><Code>BucketAlreadyOwnedByYou</Code><Message>Your previous request to create the named bucket succeeded and you already own it.</Message><BucketName>gaul-uswest</BucketName><RequestId>24B6DC3170365CD7</RequestId><HostId>hUynMTyqc9WZFxAJ2RFK6P7BqmmeHHlMl9xL2NOy56xBUnOZCAlHqGvtMeGeAfVs</HostId></Error>

Additional discussion:

https://issues.apache.org/jira/browse/JCLOUDS-334
gaul added a commit to gaul/s3-tests that referenced this pull request Aug 1, 2014
This test fails on AWS in regions other than us-standard.  See ceph#9 for
more discussion.
gaul added a commit to gaul/s3-tests that referenced this pull request Aug 1, 2014
This test fails on AWS in regions other than us-standard.  See ceph#9 for
more discussion.
@yehudasa
Copy link
Copy Markdown
Member

@andrewgaul sorry, just noticed all these pull requests, for some reason didn't get any notification. Will review these. Thanks, and sorry!

@gaul
Copy link
Copy Markdown
Contributor Author

gaul commented Dec 4, 2014

@yehudasa Can you please review my pull requests? Many of them have been open for over 4 months.

alfredodeza added a commit that referenced this pull request Dec 4, 2014
test_bucket_create_exists should fail on recreate
@alfredodeza alfredodeza merged commit 81dc4f4 into ceph:master Dec 4, 2014
@alfredodeza
Copy link
Copy Markdown
Contributor

@andrewgaul I'm going to revert this test as we do not support the duality that S3 does (other regions vs US Standard).

By default, we will adhere to whatever US Standard complies to.

The specific error documentation states:

Your previous request to create the named bucket succeeded and you already own it. You get this error in all AWS regions except US Standard, us-east-1. In us-east-1 region, you will get 200 OK, but it is no-op (if bucket exists it Amazon S3 will not do anything).

@alfredodeza
Copy link
Copy Markdown
Contributor

Reverted with: abc9e67

@gaul
Copy link
Copy Markdown
Contributor Author

gaul commented Dec 23, 2014

@alfredodeza I reworked this pull request to handle both behaviors in #30.

gaul added a commit to gaul/s3-tests that referenced this pull request Jan 1, 2015
This test fails on AWS in regions other than us-standard.  See ceph#9 for
more discussion.
gaul added a commit to gaul/s3-tests that referenced this pull request Jan 22, 2015
This test fails on AWS in regions other than us-standard.  See ceph#9 for
more discussion.
gaul added a commit to gaul/s3-tests that referenced this pull request Feb 27, 2015
This test fails on AWS in regions other than us-standard.  See ceph#9 for
more discussion.
gaul added a commit to gaul/s3-tests that referenced this pull request Mar 2, 2015
This test fails on AWS in regions other than us-standard.  See ceph#9 for
more discussion.
gaul added a commit to gaul/s3-tests that referenced this pull request Apr 9, 2015
This test fails on AWS in regions other than us-standard.  See ceph#9 for
more discussion.
gaul added a commit to gaul/s3-tests that referenced this pull request May 1, 2015
This test fails on AWS in regions other than us-standard.  See ceph#9 for
more discussion.
gaul added a commit to gaul/s3-tests that referenced this pull request Jul 13, 2015
This test fails on AWS in regions other than us-standard.  See ceph#9 for
more discussion.
gaul added a commit to gaul/s3-tests that referenced this pull request Jul 14, 2015
This test fails on AWS in regions other than us-standard.  See ceph#9 for
more discussion.
gaul added a commit to gaul/s3-tests that referenced this pull request Oct 27, 2015
This test fails on AWS in regions other than us-standard.  See ceph#9 for
more discussion.
gaul added a commit to gaul/s3-tests that referenced this pull request Nov 15, 2015
This test fails on AWS in regions other than us-standard.  See ceph#9 for
more discussion.
gaul added a commit to gaul/s3-tests that referenced this pull request Jan 16, 2016
This test fails on AWS in regions other than us-standard.  See ceph#9 for
more discussion.
gaul added a commit to gaul/s3-tests that referenced this pull request Jan 25, 2017
This test fails on AWS in regions other than us-standard.  See ceph#9 for
more discussion.
gaul added a commit to gaul/s3-tests that referenced this pull request Feb 23, 2017
This test fails on AWS in regions other than us-standard.  See ceph#9 for
more discussion.
gaul added a commit to gaul/s3-tests that referenced this pull request Aug 16, 2017
This test fails on AWS in regions other than us-standard.  See ceph#9 for
more discussion.
gaul added a commit to gaul/s3-tests that referenced this pull request Nov 16, 2018
This test fails on AWS in regions other than us-standard.  See ceph#9 for
more discussion.
gaul added a commit to gaul/s3-tests that referenced this pull request Nov 24, 2019
This test fails on AWS in regions other than us-standard.  See ceph#9 for
more discussion.
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.

3 participants