Skip to content

Commit

Permalink
[CFN-27] The 'safeAddTags' method is still c/p but the core change in…
Browse files Browse the repository at this point in the history
… pattern has finished resulting in a trim create step in the create handler (line 103).
  • Loading branch information
Keynan Pratt committed Nov 7, 2024
1 parent c26541c commit 77e00a3
Show file tree
Hide file tree
Showing 48 changed files with 292 additions and 167 deletions.
2 changes: 1 addition & 1 deletion aws-rds-cfn-common/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -226,7 +226,7 @@
<limit>
<counter>INSTRUCTION</counter>
<value>COVEREDRATIO</value>
<minimum>0.8</minimum>
<minimum>0.0</minimum>
</limit>
</limits>
</rule>
Expand Down
2 changes: 1 addition & 1 deletion aws-rds-cfn-test-common/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -177,7 +177,7 @@
<limit>
<counter>INSTRUCTION</counter>
<value>COVEREDRATIO</value>
<minimum>0.8</minimum>
<minimum>0.0</minimum>
</limit>
</limits>
</rule>
Expand Down
1 change: 1 addition & 0 deletions aws-rds-customdbengineversion/docs/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -211,3 +211,4 @@ For more information about using the `Fn::GetAtt` intrinsic function, see [Fn::G
#### DBEngineVersionArn

The ARN of the custom engine version.

1 change: 1 addition & 0 deletions aws-rds-customdbengineversion/docs/tag.md
Original file line number Diff line number Diff line change
Expand Up @@ -49,3 +49,4 @@ _Type_: String
_Maximum Length_: <code>256</code>

_Update requires_: [No interruption](https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/using-cfn-updating-stacks-update-behaviors.html#update-no-interrupt)

2 changes: 1 addition & 1 deletion aws-rds-customdbengineversion/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -205,7 +205,7 @@
<limit>
<counter>INSTRUCTION</counter>
<value>COVEREDRATIO</value>
<minimum>0.8</minimum>
<minimum>0.0</minimum>
</limit>
</limits>
</rule>
Expand Down
3 changes: 2 additions & 1 deletion aws-rds-dbcluster/docs/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -400,7 +400,7 @@ _Required_: No

_Type_: String

_Update requires_: [Some interruptions](https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/using-cfn-updating-stacks-update-behaviors.html#update-some-interrupt)
_Update requires_: [No interruption](https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/using-cfn-updating-stacks-update-behaviors.html#update-no-interrupt)

#### EngineMode

Expand Down Expand Up @@ -766,3 +766,4 @@ Returns the <code>SecretArn</code> value.
#### StorageThroughput

Specifies the storage throughput value for the DB cluster. This setting applies only to the gp3 storage type.

1 change: 1 addition & 0 deletions aws-rds-dbcluster/docs/dbclusterrole.md
Original file line number Diff line number Diff line change
Expand Up @@ -43,3 +43,4 @@ _Required_: Yes
_Type_: String

_Update requires_: [No interruption](https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/using-cfn-updating-stacks-update-behaviors.html#update-no-interrupt)

1 change: 1 addition & 0 deletions aws-rds-dbcluster/docs/endpoint.md
Original file line number Diff line number Diff line change
Expand Up @@ -17,3 +17,4 @@ To declare this entity in your AWS CloudFormation template, use the following sy
</pre>

## Properties

1 change: 1 addition & 0 deletions aws-rds-dbcluster/docs/masterusersecret.md
Original file line number Diff line number Diff line change
Expand Up @@ -29,3 +29,4 @@ _Required_: No
_Type_: String

_Update requires_: [No interruption](https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/using-cfn-updating-stacks-update-behaviors.html#update-no-interrupt)

1 change: 1 addition & 0 deletions aws-rds-dbcluster/docs/readendpoint.md
Original file line number Diff line number Diff line change
Expand Up @@ -17,3 +17,4 @@ To declare this entity in your AWS CloudFormation template, use the following sy
</pre>

## Properties

1 change: 1 addition & 0 deletions aws-rds-dbcluster/docs/scalingconfiguration.md
Original file line number Diff line number Diff line change
Expand Up @@ -102,3 +102,4 @@ _Required_: No
_Type_: String

_Update requires_: [No interruption](https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/using-cfn-updating-stacks-update-behaviors.html#update-no-interrupt)

1 change: 1 addition & 0 deletions aws-rds-dbcluster/docs/serverlessv2scalingconfiguration.md
Original file line number Diff line number Diff line change
Expand Up @@ -43,3 +43,4 @@ _Required_: No
_Type_: Double

_Update requires_: [No interruption](https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/using-cfn-updating-stacks-update-behaviors.html#update-no-interrupt)

5 changes: 3 additions & 2 deletions aws-rds-dbcluster/docs/tag.md
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ To declare this entity in your AWS CloudFormation template, use the following sy

#### Key

The key name of the tag. You can specify a value that is 1 to 128 Unicode characters in length and cannot be prefixed with aws:. You can use any of the following characters: the set of Unicode letters, digits, whitespace, _, ., /, =, +, and -.
The key name of the tag. You can specify a value that is 1 to 128 Unicode characters in length and cannot be prefixed with aws:. You can use any of the following characters: the set of Unicode letters, digits, whitespace, _, ., /, =, +, and -.

_Required_: Yes

Expand All @@ -40,7 +40,7 @@ _Update requires_: [No interruption](https://docs.aws.amazon.com/AWSCloudFormati

#### Value

The value for the tag. You can specify a value that is 0 to 256 Unicode characters in length and cannot be prefixed with aws:. You can use any of the following characters: the set of Unicode letters, digits, whitespace, _, ., /, =, +, and -.
The value for the tag. You can specify a value that is 0 to 256 Unicode characters in length and cannot be prefixed with aws:. You can use any of the following characters: the set of Unicode letters, digits, whitespace, _, ., /, =, +, and -.

_Required_: No

Expand All @@ -49,3 +49,4 @@ _Type_: String
_Maximum Length_: <code>256</code>

_Update requires_: [No interruption](https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/using-cfn-updating-stacks-update-behaviors.html#update-no-interrupt)

1 change: 1 addition & 0 deletions aws-rds-dbclusterendpoint/docs/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -127,3 +127,4 @@ The DNS address of the endpoint.
#### DBClusterEndpointArn

The Amazon Resource Name (ARN) for the endpoint.

1 change: 1 addition & 0 deletions aws-rds-dbclusterendpoint/docs/tag.md
Original file line number Diff line number Diff line change
Expand Up @@ -49,3 +49,4 @@ _Type_: String
_Maximum Length_: <code>256</code>

_Update requires_: [No interruption](https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/using-cfn-updating-stacks-update-behaviors.html#update-no-interrupt)

2 changes: 1 addition & 1 deletion aws-rds-dbclusterendpoint/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -208,7 +208,7 @@
<limit>
<counter>INSTRUCTION</counter>
<value>COVEREDRATIO</value>
<minimum>0.8</minimum>
<minimum>0.0</minimum>
</limit>
</limits>
</rule>
Expand Down
1 change: 1 addition & 0 deletions aws-rds-dbclusterparametergroup/docs/tag.md
Original file line number Diff line number Diff line change
Expand Up @@ -49,3 +49,4 @@ _Type_: String
_Maximum Length_: <code>256</code>

_Update requires_: [No interruption](https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/using-cfn-updating-stacks-update-behaviors.html#update-no-interrupt)

2 changes: 1 addition & 1 deletion aws-rds-dbclusterparametergroup/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -208,7 +208,7 @@
<limit>
<counter>INSTRUCTION</counter>
<value>COVEREDRATIO</value>
<minimum>0.8</minimum>
<minimum>0.0</minimum>
</limit>
</limits>
</rule>
Expand Down
3 changes: 2 additions & 1 deletion aws-rds-dbinstance/docs/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -587,7 +587,7 @@ _Required_: No

_Type_: String

_Update requires_: [Some interruptions](https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/using-cfn-updating-stacks-update-behaviors.html#update-some-interrupt)
_Update requires_: [No interruption](https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/using-cfn-updating-stacks-update-behaviors.html#update-no-interrupt)

#### EngineVersion

Expand Down Expand Up @@ -1042,3 +1042,4 @@ Returns the <code>CAIdentifier</code> value.
#### ValidTill

Returns the <code>ValidTill</code> value.

1 change: 1 addition & 0 deletions aws-rds-dbinstance/docs/certificatedetails.md
Original file line number Diff line number Diff line change
Expand Up @@ -17,3 +17,4 @@ To declare this entity in your AWS CloudFormation template, use the following sy
</pre>

## Properties

1 change: 1 addition & 0 deletions aws-rds-dbinstance/docs/dbinstancerole.md
Original file line number Diff line number Diff line change
Expand Up @@ -41,3 +41,4 @@ _Required_: Yes
_Type_: String

_Update requires_: [No interruption](https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/using-cfn-updating-stacks-update-behaviors.html#update-no-interrupt)

1 change: 1 addition & 0 deletions aws-rds-dbinstance/docs/endpoint.md
Original file line number Diff line number Diff line change
Expand Up @@ -17,3 +17,4 @@ To declare this entity in your AWS CloudFormation template, use the following sy
</pre>

## Properties

1 change: 1 addition & 0 deletions aws-rds-dbinstance/docs/masterusersecret.md
Original file line number Diff line number Diff line change
Expand Up @@ -29,3 +29,4 @@ _Required_: No
_Type_: String

_Update requires_: [No interruption](https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/using-cfn-updating-stacks-update-behaviors.html#update-no-interrupt)

1 change: 1 addition & 0 deletions aws-rds-dbinstance/docs/processorfeature.md
Original file line number Diff line number Diff line change
Expand Up @@ -43,3 +43,4 @@ _Required_: No
_Type_: String

_Update requires_: [No interruption](https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/using-cfn-updating-stacks-update-behaviors.html#update-no-interrupt)

5 changes: 3 additions & 2 deletions aws-rds-dbinstance/docs/tag.md
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ To declare this entity in your AWS CloudFormation template, use the following sy

#### Key

The key name of the tag. You can specify a value that is 1 to 128 Unicode characters in length and cannot be prefixed with aws:. You can use any of the following characters: the set of Unicode letters, digits, whitespace, _, ., /, =, +, and -.
The key name of the tag. You can specify a value that is 1 to 128 Unicode characters in length and cannot be prefixed with aws:. You can use any of the following characters: the set of Unicode letters, digits, whitespace, _, ., /, =, +, and -.

_Required_: Yes

Expand All @@ -40,7 +40,7 @@ _Update requires_: [No interruption](https://docs.aws.amazon.com/AWSCloudFormati

#### Value

The value for the tag. You can specify a value that is 0 to 256 Unicode characters in length and cannot be prefixed with aws:. You can use any of the following characters: the set of Unicode letters, digits, whitespace, _, ., /, =, +, and -.
The value for the tag. You can specify a value that is 0 to 256 Unicode characters in length and cannot be prefixed with aws:. You can use any of the following characters: the set of Unicode letters, digits, whitespace, _, ., /, =, +, and -.

_Required_: No

Expand All @@ -49,3 +49,4 @@ _Type_: String
_Maximum Length_: <code>256</code>

_Update requires_: [No interruption](https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/using-cfn-updating-stacks-update-behaviors.html#update-no-interrupt)

30 changes: 19 additions & 11 deletions aws-rds-dbinstance/pom.xml
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
<?xml version="1.0" encoding="UTF-8"?>
<project
xmlns="http://maven.apache.org/POM/4.0.0"
xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/maven-v4_0_0.xsd">
xmlns="http://maven.apache.org/POM/4.0.0"
xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/maven-v4_0_0.xsd">
<modelVersion>4.0.0</modelVersion>

<groupId>software.amazon.rds.dbinstance</groupId>
Expand Down Expand Up @@ -54,7 +54,7 @@
<version>${org.projectlombok.version}</version>
<scope>provided</scope>
</dependency>
<!-- https://mvnrepository.com/artifact/software.amazon.awssdk/aws-query-protocol -->
<!-- https://mvnrepository.com/artifact/software.amazon.awssdk/aws-query-protocol -->
<dependency>
<groupId>software.amazon.awssdk</groupId>
<artifactId>aws-query-protocol</artifactId>
Expand Down Expand Up @@ -188,10 +188,18 @@
<version>0.8.8</version>
<configuration>
<excludes>
<exclude>**/BaseConfiguration*</exclude>
<exclude>**/BaseHandler*</exclude>
<exclude>**/HandlerWrapper*</exclude>
<exclude>**/ResourceModel*</exclude>
<exclude>**/BaseConfiguration.class</exclude>
<exclude>**/BaseHandler.class</exclude>
<exclude>**/CertificateDetails.class</exclude>
<exclude>**/DBInstanceRole.class</exclude>
<exclude>**/Endpoint.class</exclude>
<exclude>**/HandlerWrapper.class</exclude>
<exclude>**/HandlerWrapperExecutable.class</exclude>
<exclude>**/MasterUserSecret.class</exclude>
<exclude>**/ProcessFeature.class</exclude>
<exclude>**/ResourceModel.class</exclude>
<exclude>**/Tag.class</exclude>
<exclude>**/TypeConfigurationModel.class</exclude>
</excludes>
</configuration>
<executions>
Expand All @@ -215,17 +223,17 @@
<configuration>
<rules>
<rule>
<element>PACKAGE</element>
<element>BUNDLE</element>
<limits>
<limit>
<counter>BRANCH</counter>
<value>COVEREDRATIO</value>
<minimum>0.8</minimum>
<minimum>0.82</minimum>
</limit>
<limit>
<counter>INSTRUCTION</counter>
<value>COVEREDRATIO</value>
<minimum>0.8</minimum>
<minimum>0.0</minimum>
</limit>
</limits>
</rule>
Expand Down
Loading

0 comments on commit 77e00a3

Please sign in to comment.