Commit a0be3a2e authored by Stan Hu's avatar Stan Hu

Merge branch 'revert-92b86887' into 'master'

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

See merge request gitlab-org/gitlab!38723
parents ae7ed776 eacb7223
......@@ -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