-
Notifications
You must be signed in to change notification settings - Fork 47
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
Added cronjob to schedule regular backup compaction #197
Conversation
Thank you @aaronfern for your contribution. Before I can start building your PR, a member of the organization must set the required label(s) {'reviewed/ok-to-test'}. Once started, you can check the build status in the PR checks section below. |
3569915
to
6167723
Compare
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.
Thanks for the changes @aaronfern! Overall looks very good. Just some minor suggestions below.
/invite @abdasgupta @shreyas-s-rao @ishan16696 |
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.
Thank you for the PR. Here are some feedback.
@aaronfern You have pull request review with status CHANGES_REQUESTED, please check |
@shreyas-s-rao, @ishan16696 You have pull request review open invite, please check |
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.
/lgtm My comments are addressed. @abdasgupta can you please check if this looks fine for you too?
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.
LGTM
How to categorize this PR?
/area backup
/kind enhancement
What this PR does / why we need it:
This PR enhances
etcd druid
to allow it to create a cronjob to schedule regular backup compactions to limit the number of incremental snapshots needed at any point in time for restoration and ultimately improve etcd restoration performance.Cronjob is created based on a configurable etcd spec field
compactionSchedule
Which issue(s) this PR fixes:
Fixes #191
Special notes for your reviewer:
Release note: