fix(rds/docdb)!: ParameterGroups remove ApplyMethod isUpToDate check #1709
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description of your changes
RDS:
DBParameterGroup
,DBClusterParametergroup
and DocDBDBClusterParametergroup
CustomParameters
with only fieldsapplyMethod
,parameterName
andparameterValue
(terraform does it the same way)parameterValue
Fixes #1708
Breaking changes ToDos/notes for users:
for resources RDS:
DBParameterGroup
,DBClusterParametergroup
and DocDBDBClusterParametergroup
claims need to be checked:parameters
no longer uses the previously read-only fieldsallowedValues
,applyType
,dataType
,description
,isModifiable
,minimumEngineVersion
,source
andsupportedEngineModes
parameters
requires the only use of all 3 the fieldsapplyMethod
,parameterName
andparameterValue
I have:
make reviewable test
to ensure this PR is ready for review.How has this code been tested
Manually