Skip to content

Commit

Permalink
Clean up retention period on rds instance (#3172)
Browse files Browse the repository at this point in the history
  • Loading branch information
kddejong authored Apr 26, 2024
1 parent 95a7b79 commit a9e75ef
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,8 @@
"Engine": {
"pattern": "^(?!aurora).*$",
"type": "string"
}
},
"SourceDBInstanceIdentifier": false
},
"required": [
"Engine"
Expand Down
7 changes: 6 additions & 1 deletion test/unit/rules/resources/test_retentionperiod_rds.py
Original file line number Diff line number Diff line change
Expand Up @@ -50,10 +50,15 @@ def validator():
"name,instance,expected",
[
(
"Valid version",
"Valid retention period",
{"MessageRetentionPeriod": "90"},
[],
),
(
"Valid with SourceDBInstanceIdentifier",
{"SourceDBInstanceIdentifier": "source-db"},
[],
),
(
"Invalid when not specified",
{
Expand Down

0 comments on commit a9e75ef

Please sign in to comment.