From a878cf13734bfaa0f1f0db187a4a4aa436057be5 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Alejandro=20Rodr=C3=ADguez?= Date: Tue, 20 Sep 2016 20:39:30 -0300 Subject: Set a low IO priority for storage moves to lower performance impact --- lib/gitlab_projects.rb | 12 +++++++++++- 1 file changed, 11 insertions(+), 1 deletion(-) (limited to 'lib') diff --git a/lib/gitlab_projects.rb b/lib/gitlab_projects.rb index a40cc4f..3d92cf2 100644 --- a/lib/gitlab_projects.rb +++ b/lib/gitlab_projects.rb @@ -316,7 +316,17 @@ class GitlabProjects if wait_for_pushes $logger.info "Syncing project #{@project_name} from <#{full_path}> to <#{new_full_path}>." - system(*%W(rsync -a --delete #{source_path} #{new_full_path})) + + # Set a low IO priority with ionice to not choke the server on moves + rsync_path = 'ionice -c2 -n7 rsync' + result = system(*%W(#{rsync_path} -a --delete --rsync-path="#{rsync_path}" #{source_path} #{new_full_path})) + + unless result + # If the command fails with `ionice` (maybe because we're on a OS X + # development machine), try again without `ionice`. + rsync_path = 'rsync' + system(*%W(#{rsync_path} -a --delete --rsync-path="#{rsync_path}" #{source_path} #{new_full_path})) + end else $logger.error "mv-storage failed: source path <#{full_path}> is waiting for pushes to finish." false -- cgit v1.2.1