diff options
| author | Stan Hu <stanhu@gmail.com> | 2019-05-10 06:53:43 -0500 |
|---|---|---|
| committer | Stan Hu <stanhu@gmail.com> | 2019-05-10 06:55:53 -0500 |
| commit | 36b1a2d7d08e1615f13fe839cc8251ba76ea87c6 (patch) | |
| tree | 3b210a73fe1842352dcf403e68329810d58b5501 | |
| parent | b575b303434577ab7d409a8c7321c0bce96d0994 (diff) | |
| download | gitlab-ce-sh-revert-full-gc-after-import.tar.gz | |
Don't run full gc in AfterImportServicesh-revert-full-gc-after-import
Pull mirrors would run the `Projects::AfterImportService`, which would
force a `git gc` each time it finished. This is overkill and not
necessary now that we have refs packed more frequently
(https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/27826).
Closes https://gitlab.com/gitlab-org/gitlab-ee/issues/11556
| -rw-r--r-- | app/services/projects/after_import_service.rb | 2 | ||||
| -rw-r--r-- | spec/services/projects/after_import_service_spec.rb | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/app/services/projects/after_import_service.rb b/app/services/projects/after_import_service.rb index afb9048e87b..bbdde4408d2 100644 --- a/app/services/projects/after_import_service.rb +++ b/app/services/projects/after_import_service.rb @@ -9,7 +9,7 @@ module Projects end def execute - Projects::HousekeepingService.new(@project, :gc).execute do + Projects::HousekeepingService.new(@project).execute do repository.delete_all_refs_except(RESERVED_REF_PREFIXES) end rescue Projects::HousekeepingService::LeaseTaken => e diff --git a/spec/services/projects/after_import_service_spec.rb b/spec/services/projects/after_import_service_spec.rb index 95c11f71c5e..51d3fd18881 100644 --- a/spec/services/projects/after_import_service_spec.rb +++ b/spec/services/projects/after_import_service_spec.rb @@ -15,7 +15,7 @@ describe Projects::AfterImportService do describe '#execute' do before do allow(Projects::HousekeepingService) - .to receive(:new).with(project, :gc).and_return(housekeeping_service) + .to receive(:new).with(project).and_return(housekeeping_service) allow(housekeeping_service) .to receive(:execute).and_yield |