Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Support for Wavelength Zones as valid Availability Zones attributes #1966

Closed
rhbelson opened this issue Apr 2, 2021 · 2 comments
Closed

Support for Wavelength Zones as valid Availability Zones attributes #1966

rhbelson opened this issue Apr 2, 2021 · 2 comments

Comments

@rhbelson
Copy link

rhbelson commented Apr 2, 2021

cfn-lint version: (cfn-lint --version) Latest (v0.48.1)

Description of issue. When creating CloudFormation template for resources in a Wavelength Zones, I get a warning "You must specify a valid allowed value for [Attribute] (us-west-2-wl1-sea-wlz-1). Valid values are ["af-south-1a","af-south-1b"....]"

I'm requesting support that the full list of Wavelength Zones be added to the list of valid availability zones, which are:

  • us-east-1-wl-bos-wlz-1
  • us-east-1-wl-nyc-wlz-1
  • us-east-1-wl-was-wlz-1
  • us-east-1-wl-atl-wlz-1
  • us-east-1-wl-mia-wlz-1
  • us-east-1-wl-dfw-wlz-1
  • us-west-2-wl-sfo-wlz-1
  • us-west-2-wl-las-wlz-1
  • us-west-2-wl-den-wlz-1
  • us-west-2-wl-sea-wlz-1
  • ap-northeast-1-wl1-nrt-wlz-1
  • ap-northeast-2-wl1-cjj-wlz-1
  • ap-northeast-1-wl1-kix-wlz-1
@PatMyron
Copy link
Contributor

PatMyron commented Apr 2, 2021 •
edited
Loading

list being sourced from https://github.com/aws-cloudformation/cfn-python-lint/blob/master/src/cfnlint/data/ExtendedSpecs/all/03_value_types.json

Available Wavelength zones are probably going to be too frequently updated to manually maintain. Which property type?

Also can workaround with resource-level ignores until this is fixed

@rhbelson
Copy link
Author

rhbelson commented Apr 5, 2021

Who maintains 03_value_types.json? Just wanted to make sure available Wavelength Zones were matched to the available Local Zones, which also expects to change throughout the year.

In terms of property type, this would be the AvailabilityZone attribute for a subnet resource -- is this what you had in mind?

@PatMyron PatMyron mentioned this issue May 11, 2021
Merged
PatMyron added a commit that referenced this issue Jun 9, 2021
@PatMyron
switching AvailabilityZone from AllowedValues to AllowedPatternRegex
e6f5f2e 
fixes #1966
https://aws.amazon.com/blogs/aws/now-open-third-availability-zone-in-the-aws-china-beijing-region/
@PatMyron PatMyron mentioned this issue Jun 9, 2021
Merged
kddejong pushed a commit that referenced this issue Jun 10, 2021
@PatMyron
switching AvailabilityZone from AllowedValues to AllowedPatternRegex (#…
b41fba2 
…2040)

fixes #1966
https://aws.amazon.com/blogs/aws/now-open-third-availability-zone-in-the-aws-china-beijing-region/
kddejong added a commit to kddejong/cfn-python-lint that referenced this issue Feb 28, 2022
@kddejong @PatMyron
PRs into main (aws-cloudformation#2041)
acde0f8 
* Lambda runtime deprecation updates (aws-cloudformation#2033)

https://docs.aws.amazon.com/lambda/latest/dg/runtime-support-policy.html
https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-resource-lambda-function.html#cfn-lambda-function-runtime

* switching AvailabilityZone from AllowedValues to AllowedPatternRegex

fixes aws-cloudformation#1966
https://aws.amazon.com/blogs/aws/now-open-third-availability-zone-in-the-aws-china-beijing-region/

Co-authored-by: Pat Myron <PatMyron@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@PatMyron @rhbelson