Skip to content

Commit 77da3ba

Browse files
authored
Rename EnablePerformanceInsights to PerformanceInsightsEnabled for CreateDBInstance and ModifyDBInstance api (#77)
Fixes aws-controllers-k8s/community#1251 Rename EnablePerformanceInsights to PerformanceInsightsEnabled for RDS API CreateDBInstance and ModifyDBInstance to avoid conflict with DecribeDBInstance PerformanceInsightsEnabled status Description of changes: Rename EnablePerformanceInsights to PerformanceInsightsEnabled for RDS API CreateDBInstance and ModifyDBInstance to avoid conflict with DecribeDBInstance PerformanceInsightsEnabled status 1. make the change in generator.yaml file to rename EnablePerformanceInsights to PerformanceInsightsEnabled 2. run existing rds controller test and passed By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
1 parent b522378 commit 77da3ba

File tree

9 files changed

+64
-65
lines changed

9 files changed

+64
-65
lines changed
Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,13 @@
11
ack_generate_info:
2-
build_date: "2022-05-11T16:29:30Z"
3-
build_hash: c6efa6ac643edb21219e0763541b2558718b5fe6
4-
go_version: go1.18.1
5-
version: v0.18.4-10-gc6efa6a
6-
api_directory_checksum: e7bbd21f4f975f9cf1e1e804ebd450e8e310023d
2+
build_date: "2022-05-18T21:18:00Z"
3+
build_hash: c651d2bb60694df1f7a5dad823258472a1a6fc8a
4+
go_version: go1.18.2
5+
version: v0.18.4-12-gc651d2b
6+
api_directory_checksum: 932f41dc8e3c84de66ba73cde9eecc2b94c847f3
77
api_version: v1alpha1
88
aws_sdk_go_version: v1.42.0
99
generator_config_info:
10-
file_checksum: 977d8d553f526f7d5b76d8ea9d82e5798de16e39
10+
file_checksum: 6f5ece4103c904b1e0417e817cdc437743a85fb3
1111
original_file_name: generator.yaml
1212
last_modification:
1313
reason: API generation

apis/v1alpha1/db_instance.go

Lines changed: 6 additions & 10 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

apis/v1alpha1/generator.yaml

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -178,6 +178,14 @@ resources:
178178
resource: Key
179179
service_name: kms
180180
path: Status.ACKResourceMetadata.ARN
181+
renames:
182+
operations:
183+
CreateDBInstance:
184+
input_fields:
185+
EnablePerformanceInsights: PerformanceInsightsEnabled
186+
ModifyDBInstance:
187+
input_fields:
188+
EnablePerformanceInsights: PerformanceInsightsEnabled
181189
GlobalCluster:
182190
exceptions:
183191
terminal_codes:

apis/v1alpha1/zz_generated.deepcopy.go

Lines changed: 5 additions & 10 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

config/crd/bases/rds.services.k8s.aws_dbinstances.yaml

Lines changed: 7 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -284,13 +284,6 @@ spec:
284284
for MySQL and PostgreSQL (https://docs.aws.amazon.com/AmazonRDS/latest/UserGuide/UsingWithRDS.IAMDBAuth.html)
285285
in the Amazon RDS User Guide."
286286
type: boolean
287-
enablePerformanceInsights:
288-
description: "A value that indicates whether to enable Performance
289-
Insights for the DB instance. For more information, see Using Amazon
290-
Performance Insights (https://docs.aws.amazon.com/AmazonRDS/latest/UserGuide/USER_PerfInsights.html)
291-
in the Amazon Relational Database Service User Guide. \n This setting
292-
doesn't apply to RDS Custom."
293-
type: boolean
294287
engine:
295288
description: "The name of the database engine to be used for this
296289
instance. \n Not every database engine is available for every Amazon
@@ -468,6 +461,13 @@ spec:
468461
from a DB instance after it is associated with a DB instance. \n
469462
This setting doesn't apply to RDS Custom."
470463
type: string
464+
performanceInsightsEnabled:
465+
description: "A value that indicates whether to enable Performance
466+
Insights for the DB instance. For more information, see Using Amazon
467+
Performance Insights (https://docs.aws.amazon.com/AmazonRDS/latest/UserGuide/USER_PerfInsights.html)
468+
in the Amazon Relational Database Service User Guide. \n This setting
469+
doesn't apply to RDS Custom."
470+
type: boolean
471471
performanceInsightsKMSKeyID:
472472
description: "The Amazon Web Services KMS key identifier for encryption
473473
of Performance Insights data. \n The Amazon Web Services KMS key
@@ -1035,10 +1035,6 @@ spec:
10351035
storageType:
10361036
type: string
10371037
type: object
1038-
performanceInsightsEnabled:
1039-
description: True if Performance Insights is enabled for the DB instance,
1040-
and otherwise false.
1041-
type: boolean
10421038
readReplicaDBClusterIdentifiers:
10431039
description: "Contains one or more identifiers of Aurora DB clusters
10441040
to which the RDS DB instance is replicated as a read replica. For

generator.yaml

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -178,6 +178,14 @@ resources:
178178
resource: Key
179179
service_name: kms
180180
path: Status.ACKResourceMetadata.ARN
181+
renames:
182+
operations:
183+
CreateDBInstance:
184+
input_fields:
185+
EnablePerformanceInsights: PerformanceInsightsEnabled
186+
ModifyDBInstance:
187+
input_fields:
188+
EnablePerformanceInsights: PerformanceInsightsEnabled
181189
GlobalCluster:
182190
exceptions:
183191
terminal_codes:

helm/crds/rds.services.k8s.aws_dbinstances.yaml

Lines changed: 7 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -284,13 +284,6 @@ spec:
284284
for MySQL and PostgreSQL (https://docs.aws.amazon.com/AmazonRDS/latest/UserGuide/UsingWithRDS.IAMDBAuth.html)
285285
in the Amazon RDS User Guide."
286286
type: boolean
287-
enablePerformanceInsights:
288-
description: "A value that indicates whether to enable Performance
289-
Insights for the DB instance. For more information, see Using Amazon
290-
Performance Insights (https://docs.aws.amazon.com/AmazonRDS/latest/UserGuide/USER_PerfInsights.html)
291-
in the Amazon Relational Database Service User Guide. \n This setting
292-
doesn't apply to RDS Custom."
293-
type: boolean
294287
engine:
295288
description: "The name of the database engine to be used for this
296289
instance. \n Not every database engine is available for every Amazon
@@ -468,6 +461,13 @@ spec:
468461
from a DB instance after it is associated with a DB instance. \n
469462
This setting doesn't apply to RDS Custom."
470463
type: string
464+
performanceInsightsEnabled:
465+
description: "A value that indicates whether to enable Performance
466+
Insights for the DB instance. For more information, see Using Amazon
467+
Performance Insights (https://docs.aws.amazon.com/AmazonRDS/latest/UserGuide/USER_PerfInsights.html)
468+
in the Amazon Relational Database Service User Guide. \n This setting
469+
doesn't apply to RDS Custom."
470+
type: boolean
471471
performanceInsightsKMSKeyID:
472472
description: "The Amazon Web Services KMS key identifier for encryption
473473
of Performance Insights data. \n The Amazon Web Services KMS key
@@ -1035,10 +1035,6 @@ spec:
10351035
storageType:
10361036
type: string
10371037
type: object
1038-
performanceInsightsEnabled:
1039-
description: True if Performance Insights is enabled for the DB instance,
1040-
and otherwise false.
1041-
type: boolean
10421038
readReplicaDBClusterIdentifiers:
10431039
description: "Contains one or more identifiers of Aurora DB clusters
10441040
to which the RDS DB instance is replicated as a read replica. For

pkg/resource/db_instance/delta.go

Lines changed: 7 additions & 7 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

pkg/resource/db_instance/sdk.go

Lines changed: 10 additions & 10 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)