Add support for DynamoDB TimeToLive

[rbranche]

Is your feature request related to a problem?
There doesn't seem to be a way through ACK to create a TimeToLiveSpecification on a DynamoDB table.

Describe the solution you'd like
Add a new CRD to update TimeToLiveSpecifications

https://docs.aws.amazon.com/amazondynamodb/latest/APIReference/API_UpdateTimeToLive.html

apiVersion: dynamodb.services.k8s.aws/v1alpha1
kind: TableTimeToLive
metadata:
  name: service-timetolive
spec:
    tableName: "service_table_name"
    timeToLiveSpecification:
        attributeName: "TTL"
        enabled: true

Note: the dynamodb-controller types.go already includes the TimeToLiveSpecification, but it needs to be updated to include the "Enabled" key.

https://github.com/aws-controllers-k8s/dynamodb-controller/blob/d25c74aadb1fde6b17f93c7121230f0550cd19aa/apis/v1alpha1/types.go#L652

Describe alternatives you've considered
This specification could also be in Table itself as TimeToLiveSpecification.

[RedbackThomson]

The TimeToLiveSpecification was not included as part of the Spec, by default, because it is not included as a parameter in the CreateTable method. I see that there are two methods for setting and describing this value, on tables:

• UpdateTimeToLive
• DescribeTimeToLive

This will need to be supported through custom hooks, probably in the same way S3 supports the additional Put* fields. That is, after describing a Table, a custom hook for describing the TTL and putting it into the Spec. Also, when updating a Table, checking the diff for TTL and calling the Update SDK.

[RedbackThomson]

@a-hilaly Have any updates on this?

[a-hilaly]

I will have to support update operations before adding this one - also some changes in the e2e tests.

[ack-bot]

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/aws-controllers-k8s/community.
/lifecycle stale

[a-hilaly]

/lifecycle frozen

[fbozic]

Hi team,

Do you maybe have any info when this is going to be released? This is the last feature we are missing before we can start using ACK in production.

Also I would be happy to help by contributing if that is okay.

[trajakovic]

Well, TTL is show stopper for us as well, so wondering if it possible to add this

[RedbackThomson]

@fbozic Looking into it in this sprint. If it is straightforward I should have something out soon

[RedbackThomson]

@fbozic @trajakovic Added support in v0.1.3

[fbozic]

@RedbackThomson thank you very much, this was fast 😄