Skip to content
This repository was archived by the owner on Jun 13, 2024. It is now read-only.

Ignore/Add cn-* region(s) #53

Closed
sopel opened this issue Feb 9, 2014 · 0 comments
Closed

Ignore/Add cn-* region(s) #53

sopel opened this issue Feb 9, 2014 · 0 comments
Assignees
Milestone

Comments

@sopel
Copy link
Owner

sopel commented Feb 9, 2014

Similar to #50, the upstream addition of region cn-north-1 within boto 2.22.0 now triggers runtime failures due to this region requiring a dedicated account.

While this issue is about cn-*right now, it confirms my hunch back then that any dedicated handling will not scale, and a region neutral handling is also confirmed as the current upstream position in boto/boto#1951 (comment).

  • Given the project at hand, I obviously disagree with the notion of cross region looping being that uncommon still, and would expect it to be used more frequently over time due to region failover and load balancing scenarios; in particular, all sorts of AWS account house keeping and backup scenarios are best done cross region from the get go I think.

So in hindsight the include/only solution derived from the AWS Tools for Windows PowerShell turns out to be short sighted, and also inconsistent with the existing --region parameter in the first place:

  -r REGION, --region REGION
                        A region substring selector (e.g. 'us-west')

A better solution might be to expand this current substring handling to regular expressions, which would yield utmost flexibility, yet remain simple enough and explicit if so desired (e.g. (us-east-1|us-west-1)) - the default here could then just be the universally accessible ones (i.e. exclude gov and cn regions).

@sopel sopel added this to the 1.x milestone Feb 9, 2014
@sopel sopel added bug labels Feb 9, 2014
@sopel sopel self-assigned this Feb 9, 2014
@sopel sopel modified the milestones: 1.3.0, 1.x Feb 9, 2014
sopel added a commit that referenced this issue Feb 26, 2014
sopel added a commit that referenced this issue Feb 26, 2014
@sopel sopel closed this as completed in 7e97a73 Feb 26, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant