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.

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

E3691 false positive when specify EngineVersion without minor #3849

Closed
sir-sigurd opened this issue Dec 2, 2024 · 0 comments · Fixed by #3850
Closed

E3691 false positive when specify EngineVersion without minor #3849

sir-sigurd opened this issue Dec 2, 2024 · 0 comments · Fixed by #3850

Comments

@sir-sigurd
Copy link

CloudFormation Lint Version

1.20.1

What operating system are you using?

Mac

Describe the bug

I get

E3691 14 is not one of ['11.22', '11.22-rds.20240418', '11.22-rds.20240509', '11.22-rds.20240808', '12.15', '12.16', '12.17', '12.18', '12.19', '12.20', '12.21', '12.22', '13.11', '13.12', '13.13', '13.14', '13.15', '13.16', '13.17', '13.18', '14.10', '14.11', '14.12', '14.13', '14.14', '14.15', '14.9', '15.10', '15.4', '15.5', '15.6', '15.7', '15.8', '15.9', '16.1', '16.2', '16.3', '16.4', '16.5', '16.6', '17.1', '17.2']
delme-rds-test.yaml:8:7

on the reproduction template

Expected behavior

Not get E3691 because the reproduction template actually works beause RDS API allows to specify only major version, see https://docs.aws.amazon.com/AmazonRDS/latest/UserGuide/PostgreSQL.Concepts.General.DBVersions.html:

If a major version is specified but a minor version is not, Amazon RDS defaults to a recent release of the major version you have specified.

Reproduction template

Resources:
  MyDB:
    Type: 'AWS::RDS::DBInstance'
    Properties:
      DBInstanceClass: db.t3.micro
      AllocatedStorage: 20
      Engine: postgres
      EngineVersion: 14
      MasterUsername: test
      MasterUserPassword: testtest
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant