Merge ~barryprice/swoffsite-charms/+git/swoffsite-charm:master into ~swoffsite-charmers/swoffsite-charms/+git/swoffsite-charm:master

Proposed by Barry Price
Status: Merged
Merged at revision: 105670e7790d6d49a2c42c7e2a859e2161d9f03d
Proposed branch: ~barryprice/swoffsite-charms/+git/swoffsite-charm:master
Merge into: ~swoffsite-charmers/swoffsite-charms/+git/swoffsite-charm:master
Diff against target: 17 lines (+2/-2)
1 file modified
swoffsite/swoffsite/cli.py (+2/-2)
Reviewer Review Type Date Requested Status
🤖 Canonical IS Review Bot (community) Approve
swoffsite Charmers Pending
Review via email: mp+453407@code.launchpad.net

Commit message

Temporarily patch our bundled swoffsite to not use underscores in S3 bucket names (forbidden since 2018)

To post a comment you must log in.
Revision history for this message
🤖 Canonical IS Review Bot (canonical-is-reviewbot) wrote :

Proxy approval for jsimpso

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
diff --git a/swoffsite/swoffsite/cli.py b/swoffsite/swoffsite/cli.py
index a468c6b..4260969 100644
--- a/swoffsite/swoffsite/cli.py
+++ b/swoffsite/swoffsite/cli.py
@@ -35,10 +35,10 @@ from .pool import FakePool, Pool
35def s3_bucket_name():35def s3_bucket_name():
36 '''Calculate a unique bucket name for the specified AWS key and OS tenant.36 '''Calculate a unique bucket name for the specified AWS key and OS tenant.
37 '''37 '''
38 return '_'.join([38 return '-'.join([
39 'swoffsite',39 'swoffsite',
40 os.environ['OS_TENANT_NAME'],40 os.environ['OS_TENANT_NAME'],
41 os.environ['AWS_ACCESS_KEY_ID']]).lower()41 os.environ['AWS_ACCESS_KEY_ID']]).lower().replace('_', '-')
4242
4343
44def parse_args():44def parse_args():

Subscribers

People subscribed via source and target branches