-
Notifications
You must be signed in to change notification settings - Fork 56
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 inconsistency between BACKUP_ITEMS and USE_ #115
Comments
I'm thinking in deprecating
The implementation of this will happen as part of #155 where recap would allow plugins/modules to be written. |
I have implemented the deprecation of Being able to dynamically backup/archive/compress/email is needed in order to add more reports through plugins/modules. That's the reason why it was implemented in there. Documenting in here the tests performed: Testing Backup:
See it backed up Re-running recap, disabling
Working as intended, i.e. dynamically finding the right reports/logs to act upon. Testing sending mail functionalityUsing the previous config:
See it emailed up Re-running recap, adding back
Similar as the first test, this is working as intended, dynamically finding the Testing recaplogThe reports/logs for the day before(
|
Closing this one since it has been merged into |
Back in issue #94 when working on the documentation and configuration of the settings, it was clear that there is a inconsistency between the list of
BACKUP_ITEMS
and theUSE_
variables (#94 (comment)), it's required to be more consistent on those settings, likely to deprecate the use ofBACKUP_ITEMS
or create it in a dynamic way based on theUSE_
variables.The text was updated successfully, but these errors were encountered: