Commit eacb7223 authored by Igor Drozdov's avatar Igor Drozdov

Revert "Merge branch 'id-remove-lfs-check-ff' into 'master'"

This reverts merge request !38677
parent 15b0ac26
......@@ -7,6 +7,10 @@ module Gitlab
ERROR_MESSAGE = 'LFS objects are missing. Ensure LFS is properly set up or try a manual "git lfs push --all".'
def validate!
# This feature flag is used for disabling integrify check on some envs
# because these costy calculations may cause performance issues
return unless Feature.enabled?(:lfs_check, default_enabled: true)
return unless project.lfs_enabled?
return if skip_lfs_integrity_check
......
......@@ -27,6 +27,18 @@ RSpec.describe Gitlab::Checks::LfsCheck do
allow(project).to receive(:lfs_enabled?).and_return(true)
end
context 'with lfs_check feature disabled' do
before do
stub_feature_flags(lfs_check: false)
end
it 'skips integrity check' do
expect_any_instance_of(Gitlab::Git::LfsChanges).not_to receive(:new_pointers)
subject.validate!
end
end
context 'deletion' do
let(:changes) { { oldrev: oldrev, ref: ref } }
......
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