-
Notifications
You must be signed in to change notification settings - Fork 152
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add new zones to AWS #1157
Add new zones to AWS #1157
Conversation
pkg/blockstorage/awsebs/awsebs.go
Outdated
@@ -736,6 +736,9 @@ func staticRegionToZones(region string) ([]string, error) { | |||
"eu-central-1a", | |||
"eu-central-1b", | |||
"eu-central-1c", | |||
"eu-central-1-wl1-ber-wlz-1", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's an interesting naming scheme.
Is there an official documentation link for these?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The names are defined by AWS. We call the list regions API and these are the values it provides.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is what the AWS CLI spews out:
aws ec2 describe-availability-zones --region eu-central-1 |jq .AvailabilityZones[].ZoneName
"eu-central-1a"
"eu-central-1b"
"eu-central-1c"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@julio-lopez The result varies by the credentials I think. Another reason why this should be dynamic.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
x-references:
aws ec2 describe-availability-zones --all-availability-zones --region eu-central-1 |jq .AvailabilityZones[].ZoneName
"eu-central-1a"
"eu-central-1b"
"eu-central-1c"
"eu-central-1-wl1-ber-wlz-1"
"eu-central-1-wl1-dtm-wlz-1"
"eu-central-1-wl1-muc-wlz-1"
Change Overview
The following PR add the new missing zones to AWS.
Pull request type
Please check the type of change your PR introduces:
Issues
Test Plan