diff options
author | Patrick J Cherry <patrick@bytemark.co.uk> | 2017-01-31 12:12:23 +0000 |
---|---|---|
committer | Patrick J Cherry <patrick@bytemark.co.uk> | 2017-01-31 12:12:23 +0000 |
commit | 659a651aa294c335e519b4c72bbd052b3199a793 (patch) | |
tree | a36a95dd652717624034c553bf18c7d66156ccf5 /bin/byteback-prune | |
parent | 38c89a9e32a67f96b40d756a5c356f8c31247ab9 (diff) |
First pass with rubocoprubocop
Diffstat (limited to 'bin/byteback-prune')
-rwxr-xr-x | bin/byteback-prune | 20 |
1 files changed, 10 insertions, 10 deletions
diff --git a/bin/byteback-prune b/bin/byteback-prune index a0cd0fb..3fef2c4 100755 --- a/bin/byteback-prune +++ b/bin/byteback-prune @@ -1,4 +1,5 @@ #!/usr/bin/ruby +# frozen_string_literal: true # # Program to free up space on the backup-storage volume, by removing # backups (whether by age, or importance). @@ -17,12 +18,12 @@ opts = Trollop.options do banner "Prune old backup directories to ensure there's enough space" opt :minpercent, 'Start prune when disk has less than this %age free', - :type => :integer, - :default => 5 + type: :integer, + default: 5 opt :maxpercent, 'Stop prune when disk has more than this %age free', - :type => :integer, - :default => 10 + type: :integer, + default: 10 opt :list, 'List backups in pruning order, no other action' @@ -31,8 +32,8 @@ opts = Trollop.options do opt :prune_force, 'Prune the next backup regardless' opt :order, "Order backups by 'age' or 'importance'", - :type => :string, - :default => 'importance' + type: :string, + default: 'importance' opt :verbose, 'Show debugging messages' end @@ -80,7 +81,7 @@ gradient_30m = @df_history.gradient(1800) # Check whether we should still be pruning # @free = @df_history.list.last.percent_free -PRUNING_FLAG = "#{ENV['HOME']}/.byteback.pruning" +PRUNING_FLAG = "#{ENV['HOME']}/.byteback.pruning".freeze if @do_prune_force info('Forcing prune') @@ -127,14 +128,13 @@ unless @do_prune_force # # Do not prune unless at least one snapshot is a week old # - last_week = Time.now - (7*86400) - unless snapshots.any?{|snapshot| last_week > snapshot.time} + last_week = Time.now - (7 * 86_400) + unless snapshots.any? { |snapshot| last_week > snapshot.time } warn('There are no snapshots older than a week. Not pruning.') exit 0 end end - exit 0 unless (@do_prune && File.exist?(PRUNING_FLAG)) || @do_prune_force |