storj / edge

Storj edge services (including multi-tenant, S3-compatible server to interact with the Storj network)
GNU Affero General Public License v3.0
53 stars 18 forks source link

Add `GetBucketLocation` tests to release checklist #409

Closed amwolff closed 7 months ago

amwolff commented 7 months ago

We need to add some tests to ensure GetBucketLocation is working correctly before we make a release. This is one of the endpoints where E2E tests correctness is dependent on whether there were necessary dependency bumps made before making a release. We should design a simple test that allows us to ensure that this endpoint is working with the scheduled satellite release.

This could be something as simple as:

  1. Create a bucket
  2. Call GetBucketLocation
    • aws --endpoint-url https://s3.storjapi.io s3api get-bucket-location --bucket demo-bucket
  3. Ensure that the created bucket has the necessary location constraint as per the documentation.

The acceptance criteria are to put the exact steps in the release checklist template.

pwilloughby commented 7 months ago

You can also check with the aws cli if that's useful. aws --endpoint-url https://s3.storjapi.io s3api get-bucket-location --bucket demo-bucket

amwolff commented 7 months ago

oh, yep, awscli is likely easier. I'll update the issue. Thanks!

neo-cypher commented 7 months ago

https://build.dev.storj.tools/job/edge-gerrit/7156/