-
Notifications
You must be signed in to change notification settings - Fork 72
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
ensure that the backup types match when doing incremental backups #891
Conversation
874a918
to
913b212
Compare
Gotta find out how to move that check to "prep" |
913b212
to
2db0caa
Compare
226da9a
to
beadad4
Compare
beadad4
to
5a4b317
Compare
@ehelms ready! |
description 'Check if the incremental backup has the right type' | ||
tags :backup | ||
param :incremental_dir, 'Path to existing backup directory' | ||
param :online_backup, 'Select for online backup', :flag => true, :default => false |
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 the supposition that if this is not set, it defaults to offline backup?
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.
Yes
end | ||
|
||
def run | ||
return unless @incremental_dir |
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.
This parameter is required right? So can this ever be true?
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.
This parameter can be nil
(as in: set to nil
by the caller) when we're creating a non-incremental (=base) backup
No description provided.