Skip to content
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

feat: add clean schedule and resolve npm audit issues #26

Closed
wants to merge 2 commits into from
Closed

feat: add clean schedule and resolve npm audit issues #26

wants to merge 2 commits into from

Conversation

afittz
Copy link

@afittz afittz commented Jun 18, 2020

Fixes #25

  • Added new CloudFormation parameter called CleanSchedule with a default value of rate(1 hour).

Additional changes:

I'm happy to change/remove anything; just let me know!

@afittz
Copy link
Author

afittz commented Jul 6, 2020

Are there any changes I need to make for this?

@theburningmonk
Copy link
Collaborator

@afittz sorry, missed this when it came in, looks good, maybe add a few examples in the description for the CleanSchedule param so ppl don't have to look at the code to figure out what they need to put in there (not everyone is familiar with the cron syntax for CW Events)

@afittz
Copy link
Author

afittz commented Jul 6, 2020

@theburningmonk Great suggestion! I'll add some examples.

@afittz
Copy link
Author

afittz commented Aug 3, 2020

@theburningmonk Sorry for the delay in getting this cleaned up!!!

```

The CleanSchedule parameter supports any Schedule Expression for a CloudWatch event. The default uses a rate expression that will execute hourly, however you can easily specify any desired number of minutes/hours/days between runs or use a cron expression. Examples: `rate(12 hours)` or `rate(1 day)`. More details on valid options can be found on the [Amazon CloudWatch Events website](https://docs.aws.amazon.com/AmazonCloudWatch/latest/events/ScheduledEvents.html).

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think they're called "EventBridge events" now

@afittz afittz closed this Aug 10, 2021
@arjunsoota
Copy link

Why was this not merged to the main release branch ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature Request] Custom clean up schedule (rate)
4 participants