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

fix(recurringjob): keep snapshots for the backup #1965

Merged
merged 1 commit into from
Nov 28, 2023

Conversation

mantissahz
Copy link
Contributor

Add a new global setting that allows users to keep the retain number of snapshots when the recurring job does a backup.

Ref: longhorn/longhorn#2997

derekbit
derekbit previously approved these changes Jun 8, 2023
Copy link
Member

@derekbit derekbit left a comment

Choose a reason for hiding this comment

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

LGTM

shuo-wu
shuo-wu previously approved these changes Jun 30, 2023
Copy link
Contributor

@shuo-wu shuo-wu left a comment

Choose a reason for hiding this comment

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

LGTM

app/recurring_job.go Show resolved Hide resolved
shuo-wu
shuo-wu previously approved these changes Jun 30, 2023
@mantissahz
Copy link
Contributor Author

@mergify backport v1.5.x

@mergify
Copy link

mergify bot commented Jul 25, 2023

backport v1.5.x

✅ Backports have been created

@mantissahz mantissahz force-pushed the issue2997 branch 2 times, most recently from babace2 to 5039b39 Compare August 8, 2023 08:47
@mantissahz mantissahz force-pushed the issue2997 branch 2 times, most recently from fa84d4c to c80ff96 Compare August 23, 2023 04:15
@mantissahz mantissahz force-pushed the issue2997 branch 2 times, most recently from d036d62 to a471f6c Compare October 5, 2023 01:11
@mantissahz mantissahz force-pushed the issue2997 branch 2 times, most recently from 5f1d613 to 89871f0 Compare October 24, 2023 07:52
app/recurring_job.go Outdated Show resolved Hide resolved
app/recurring_job.go Outdated Show resolved Hide resolved
@mantissahz mantissahz force-pushed the issue2997 branch 2 times, most recently from 92d2891 to 9dfd109 Compare October 24, 2023 13:52
@mantissahz mantissahz requested review from c3y1huang, derekbit, shuo-wu and a team October 24, 2023 13:57
Add a new global setting that allows users to keep the retain
number of snapshots when the recurring job does a backup.

Ref: 2997

Signed-off-by: James Lu <james.lu@suse.com>
Copy link
Member

@innobead innobead left a comment

Choose a reason for hiding this comment

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

LGTM.

@innobead innobead merged commit 16c09fd into longhorn:master Nov 28, 2023
5 checks passed
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.

5 participants