-
Notifications
You must be signed in to change notification settings - Fork 40
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[DBSnapshot] create new resource DBSnapshot #227
base: master
Are you sure you want to change the base?
[DBSnapshot] create new resource DBSnapshot #227
Conversation
"description": "An example resource schema demonstrating some basic constructs and validation rules.", | ||
"sourceUrl": "https://github.com/aws-cloudformation/aws-cloudformation-rpdk.git", | ||
"definitions": { | ||
"Tag": { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I believe we need to mark resource as taggable according to the resource schema: https://github.com/aws-cloudformation/cloudformation-resource-schema/blob/master/src/main/resources/schema/provider.definition.schema.v1.json#L121
@@ -0,0 +1,4 @@ | |||
{ | |||
"DBSnapshotIdentifier": "dbsnapshot-contract-test", | |||
"DBInstanceIdentifier": "postgresql-3" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I believe contract tests will require this instance to exist. Can we include template for that instance in this PR?
.translateToServiceRequest(Function.identity()) | ||
// this skips the call and goes directly to stabilization | ||
.makeServiceCall(EMPTY_CALL) | ||
.stabilize((resourceModel, response, proxyInvocation, model, callbackContext) -> isStabilized(resourceModel, proxyInvocation)).progress(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is handleErrors
call omitted intentionally here?
"read": { | ||
"permissions": [ | ||
"rds:DescribeDBSnapshots", | ||
"rds:ListTagsForResource" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think I see any invocations of ListTagsForResource
. Am I missing something?
Description of changes:
This changes contains DBSnapshot resource implementation following most recent AWS RDS Documentation: https://docs.aws.amazon.com/AmazonRDS/latest/UserGuide/USER_CreateSnapshot.html
Which includes
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.