Skip to content

Unified Backups: Refactor tasks

Gabriel Mazetto requested to merge 427337-unified-backups-refactor-tasks into master

What does this MR do and why?

  • Refactors and extract Struct based TaskDefinition into Backup::Tasks subclasses
  • Rename previous Backup::Task based classes into Backup::Targets::Target based classes
  • A handful of Rubocop fixes that are no longer ignored due to moving/renaming code around
  • Other methods renamed to be more descriptive and more code extracted from Backup::Manager

MR acceptance checklist

Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

How to set up and validate locally

There is no intended change in behavior, so testing out a regular backup and restore among with custom ENV variable should be sufficient

Related to #427337 (closed)

Edited by Gabriel Mazetto

Merge request reports