Commit 718fe268 authored by Kamil Trzcinski's avatar Kamil Trzcinski

Improve after review

parent 12bd7810
module Backup module Backup
class Manager class Manager
ARCHIVES_TO_BACKUP = %w{uploads builds artifacts lfs registry} ARCHIVES_TO_BACKUP = %w[uploads builds artifacts lfs registry]
FOLDERS_TO_BACKUP = %w{repositories db} FOLDERS_TO_BACKUP = %w[repositories db]
def pack def pack
# Make sure there is a connection # Make sure there is a connection
...@@ -150,7 +150,7 @@ module Backup ...@@ -150,7 +150,7 @@ module Backup
end end
def skipped?(item) def skipped?(item)
settings[:skipped] && settings[:skipped].include?(item) settings[:skipped] && settings[:skipped].include?(item) || disabled_features.include?(item)
end end
private private
...@@ -160,11 +160,11 @@ module Backup ...@@ -160,11 +160,11 @@ module Backup
end end
def archives_to_backup def archives_to_backup
(ARCHIVES_TO_BACKUP - disabled_features).map{ |name| (name + ".tar.gz") unless skipped?(name) }.compact ARCHIVES_TO_BACKUP.map{ |name| (name + ".tar.gz") unless skipped?(name) }.compact
end end
def folders_to_backup def folders_to_backup
(FOLDERS_TO_BACKUP - disabled_features).reject{ |name| skipped?(name) } FOLDERS_TO_BACKUP.reject{ |name| skipped?(name) }
end end
def disabled_features def disabled_features
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment