mirrors/fluentd-ui
1
0
Fork 0
mirror of https://github.com/fluent/fluentd-ui.git synced 2026-05-05 10:56:11 +02:00
Code Issues Projects Releases Wiki Activity

Chamge method name #backup_files -> #backup_files_in_old_order

Browse Source 
to make its sort function clearer, and Refactor #remove_over_backup_files by using it
This commit is contained in:
鳥井 雪 2014-12-17 18:09:04 +09:00
parent a61ba6f506
commit c4eb06f5ba
1 changed files with 11 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
app/models/fluentd/agent/local_common.rb
View File

@ -64,11 +64,15 @@ class Fluentd
end
def backup_files
Dir.glob("#{config_backup_dir}/*").sort
Dir.glob("#{config_backup_dir}/*")
end
def backup_files_in_old_order
backup_files.sort
end
def backup_files_in_new_order
backup_files.reverse
backup_files_in_old_order.reverse
end
private
@ -84,9 +88,11 @@ class Fluentd
def remove_over_backup_files
over_file_count = backup_files.size - MAX_BACKUP_FILE_NUM
#.times do nothing if over_file_count is negative or 0.
over_file_count.times do |i|
FileUtils.rm(backup_files.shift)
return if over_file_count <= 0
backup_files_in_old_order.first(over_file_count).each do |file|
next unless File.exist? file
FileUtils.rm(file)
end
end