Commit 92b86887 authored by Jarka Košanová's avatar Jarka Košanová

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

Remove lfs_check feature flag

Closes #233550

See merge request gitlab-org/gitlab!38677
parents 4c485a25 5f234f8d
...@@ -7,7 +7,6 @@ module Gitlab ...@@ -7,7 +7,6 @@ module Gitlab
ERROR_MESSAGE = 'LFS objects are missing. Ensure LFS is properly set up or try a manual "git lfs push --all".' ERROR_MESSAGE = 'LFS objects are missing. Ensure LFS is properly set up or try a manual "git lfs push --all".'
def validate! def validate!
return unless Feature.enabled?(:lfs_check, default_enabled: true)
return unless project.lfs_enabled? return unless project.lfs_enabled?
return if skip_lfs_integrity_check return if skip_lfs_integrity_check
......
...@@ -27,18 +27,6 @@ RSpec.describe Gitlab::Checks::LfsCheck do ...@@ -27,18 +27,6 @@ RSpec.describe Gitlab::Checks::LfsCheck do
allow(project).to receive(:lfs_enabled?).and_return(true) allow(project).to receive(:lfs_enabled?).and_return(true)
end 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 context 'deletion' do
let(:changes) { { oldrev: oldrev, ref: ref } } 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