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

Commit 2b52f3a

Browse files
author
Christopher Hein
authored
Merge pull request #53 from christopherhein/feature/52-resourcename
Adding Support For ResourceName w/ Dynamo & S3buckets
2 parents ee15428 + 0b174a4 commit 2b52f3a

File tree

6 files changed

+6
-8
lines changed

6 files changed

+6
-8
lines changed

examples/dynamodb.yaml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,6 @@ kind: DynamoDB
33
metadata:
44
name: dynamodb-table
55
spec:
6-
tableName: dynamodb-table
76
hashAttribute:
87
name: user_id
98
type: S

examples/s3bucket.yaml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,6 @@ kind: S3Bucket
33
metadata:
44
name: chrishein-test-bucket-109
55
spec:
6-
bucketName: chrishein-test-bucket-name-109
76
versioning: false
87
logging:
98
enabled: false

models/dynamodb.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ spec:
2222
type: object
2323
properties:
2424
- key: tableName
25-
type: string
25+
type: resourceName
2626
description: |
2727
TableName is the name of the DynamoDB Table to be created.
2828
structKey: TableName

models/s3bucket.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ spec:
1919
type: object
2020
properties:
2121
- key: bucketName
22-
type: string
22+
type: resourceName
2323
description: |
2424
BucketName is the name of the S3 Bucket to be created.
2525
structKey: BucketName

pkg/operator/dynamodb/cft.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -75,7 +75,7 @@ func (s *Cloudformation) CreateStack() (output *cloudformation.CreateStackOutput
7575
resourceVersion := helpers.CreateParam("ResourceVersion", s.DynamoDB.ResourceVersion)
7676
namespace := helpers.CreateParam("Namespace", s.DynamoDB.Namespace)
7777
clusterName := helpers.CreateParam("ClusterName", s.config.ClusterName)
78-
tableName := helpers.CreateParam("TableName", helpers.Stringify(s.DynamoDB.Spec.TableName))
78+
tableName := helpers.CreateParam("TableName", helpers.Stringify(s.DynamoDB.Name))
7979
rangeAttributeName := helpers.CreateParam("RangeAttributeName", helpers.Stringify(s.DynamoDB.Spec.RangeAttribute.Name))
8080
rangeAttributeType := helpers.CreateParam("RangeAttributeType", helpers.Stringify(s.DynamoDB.Spec.RangeAttribute.Type))
8181
readCapacityUnits := helpers.CreateParam("ReadCapacityUnits", helpers.Stringify(s.DynamoDB.Spec.ReadCapacityUnits))
@@ -134,7 +134,7 @@ func (s *Cloudformation) UpdateStack(updated *awsV1alpha1.DynamoDB) (output *clo
134134
resourceVersion := helpers.CreateParam("ResourceVersion", s.DynamoDB.ResourceVersion)
135135
namespace := helpers.CreateParam("Namespace", s.DynamoDB.Namespace)
136136
clusterName := helpers.CreateParam("ClusterName", s.config.ClusterName)
137-
tableName := helpers.CreateParam("TableName", helpers.Stringify(updated.Spec.TableName))
137+
tableName := helpers.CreateParam("TableName", helpers.Stringify(s.DynamoDB.Name))
138138
rangeAttributeName := helpers.CreateParam("RangeAttributeName", helpers.Stringify(updated.Spec.RangeAttribute.Name))
139139
rangeAttributeType := helpers.CreateParam("RangeAttributeType", helpers.Stringify(updated.Spec.RangeAttribute.Type))
140140
readCapacityUnits := helpers.CreateParam("ReadCapacityUnits", helpers.Stringify(updated.Spec.ReadCapacityUnits))

pkg/operator/s3bucket/cft.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -75,7 +75,7 @@ func (s *Cloudformation) CreateStack() (output *cloudformation.CreateStackOutput
7575
resourceVersion := helpers.CreateParam("ResourceVersion", s.S3Bucket.ResourceVersion)
7676
namespace := helpers.CreateParam("Namespace", s.S3Bucket.Namespace)
7777
clusterName := helpers.CreateParam("ClusterName", s.config.ClusterName)
78-
bucketName := helpers.CreateParam("BucketName", helpers.Stringify(s.S3Bucket.Spec.BucketName))
78+
bucketName := helpers.CreateParam("BucketName", helpers.Stringify(s.S3Bucket.Name))
7979
versioning := helpers.CreateParam("EnableVersioning", helpers.Stringify(s.S3Bucket.Spec.Versioning))
8080
loggingenabled := helpers.CreateParam("EnableLogging", helpers.Stringify(s.S3Bucket.Spec.Logging.Enabled))
8181
loggingprefix := helpers.CreateParam("LoggingPrefix", helpers.Stringify(s.S3Bucket.Spec.Logging.Prefix))
@@ -128,7 +128,7 @@ func (s *Cloudformation) UpdateStack(updated *awsV1alpha1.S3Bucket) (output *clo
128128
resourceVersion := helpers.CreateParam("ResourceVersion", s.S3Bucket.ResourceVersion)
129129
namespace := helpers.CreateParam("Namespace", s.S3Bucket.Namespace)
130130
clusterName := helpers.CreateParam("ClusterName", s.config.ClusterName)
131-
bucketName := helpers.CreateParam("BucketName", helpers.Stringify(updated.Spec.BucketName))
131+
bucketName := helpers.CreateParam("BucketName", helpers.Stringify(s.S3Bucket.Name))
132132
versioning := helpers.CreateParam("EnableVersioning", helpers.Stringify(updated.Spec.Versioning))
133133
loggingenabled := helpers.CreateParam("EnableLogging", helpers.Stringify(updated.Spec.Logging.Enabled))
134134
loggingprefix := helpers.CreateParam("LoggingPrefix", helpers.Stringify(updated.Spec.Logging.Prefix))

0 commit comments

Comments
 (0)