Skip to content
This repository has been archived by the owner on Jan 19, 2022. It is now read-only.

WIP: Allow logical name to be set on additional buckets #276

Closed

Conversation

robyoung
Copy link
Contributor

@robyoung robyoung commented Jul 6, 2017

AWS allows buckets to have a logical name as well as a physical name. This feature is used by upload_assets to make it easier to perform actions on a kind of bucket within an environment. This is also useful for additional S3 buckets where we need a different bucket per environment.

Backwards compatibility is maintained by introducing an additional yaml key for the logical-name which overrides the resource name. If it's not set the resource name reverts to the bucket name.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.1%) to 57.911% when pulling 8a58f26 on robyoung:s3-logical-resource-id into e873416 on ministryofjustice:master.

AWS allows buckets to have a logical name as well as a physical name.
This feature is used by `upload_assets` to make it easier to perform
actions on a kind of bucket within an environment. This is also useful
for additional S3 buckets where we need a different bucket per
environment.

Backwards compatibility is maintained by introducing an additional yaml
key for the `logical-name` which overrides the resource name. If
it's not set the resource name reverts to the bucket name.
@coveralls
Copy link

Coverage Status

Coverage increased (+0.1%) to 57.911% when pulling 7b916be on robyoung:s3-logical-resource-id into e873416 on ministryofjustice:master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.1%) to 57.911% when pulling 7b916be on robyoung:s3-logical-resource-id into e873416 on ministryofjustice:master.

@robyoung robyoung changed the title Allow logical name to be set on additional buckets WIP: Allow logical name to be set on additional buckets Jul 13, 2017
@robyoung
Copy link
Contributor Author

Apparently the same effect can be achieved by using the cloudformation output.

@robyoung robyoung changed the title WIP: Allow logical name to be set on additional buckets Allow logical name to be set on additional buckets Jul 13, 2017
@robyoung robyoung changed the title Allow logical name to be set on additional buckets WIP: Allow logical name to be set on additional buckets Jul 13, 2017
Copy link

@richerry richerry left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved

Copy link

@richerry richerry left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants