Skip to content
This repository was archived by the owner on Nov 7, 2019. It is now read-only.

Commit 9711dd7

Browse files
author
Christopher Hein
authored
Merge pull request #70 from christopherhein/bug/using-s3-name-vs-dynamo
Use Dynamo Variable Name vs S3 for ConfigMap
2 parents 1d1d06a + 09b1b46 commit 9711dd7

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

models/dynamodb.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -99,7 +99,7 @@ spec:
9999
templateKey: TableArn
100100
additionalResources:
101101
configMaps:
102-
- name: s3BucketCM
102+
- name: dynamoCM
103103
data:
104104
- key: tableName
105105
value: "{{.Obj.Output.TableName}}"

pkg/operator/dynamodb/controller.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -263,13 +263,13 @@ func syncAdditionalResources(config *config.Config, s *awsV1alpha1.DynamoDB) (er
263263
resource = resource.DeepCopy()
264264

265265
configmaps := []string{}
266-
s3BucketCMData := map[string]string{
266+
dynamoCMData := map[string]string{
267267
"tableName": "{{.Obj.Output.TableName}}",
268268
"tableARN": "{{.Obj.Output.TableARN}}",
269269
"region": "{{.Config.Region}}",
270270
}
271-
s3BucketCM := helpers.CreateConfigMap(config, s, s.Name, s.Namespace, s3BucketCMData)
272-
configmaps = append(configmaps, s3BucketCM)
271+
dynamoCM := helpers.CreateConfigMap(config, s, s.Name, s.Namespace, dynamoCMData)
272+
configmaps = append(configmaps, dynamoCM)
273273
resource.AdditionalResources.ConfigMaps = configmaps
274274

275275
_, err = clientSet.DynamoDBs(s.Namespace).Update(resource)

0 commit comments

Comments
 (0)