Commit 95f81a81 authored by Sincheol (David) Kim's avatar Sincheol (David) Kim

Merge branch 'sh-remove-lfs-auto-link-feature-flag' into 'master'

Remove feature flag for auto-linking LFS objects in forks

See merge request gitlab-org/gitlab!78592
parents 91fb24fc dd5724c4
...@@ -155,7 +155,6 @@ module Repositories ...@@ -155,7 +155,6 @@ module Repositories
end end
def should_auto_link? def should_auto_link?
return false unless Feature.enabled?(:lfs_auto_link_fork_source, project, default_enabled: :yaml)
return false unless project.forked? return false unless project.forked?
# Sanity check in case for some reason the user doesn't have access to the parent # Sanity check in case for some reason the user doesn't have access to the parent
......
---
name: lfs_auto_link_fork_source
introduced_by_url: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/75972
rollout_issue_url: https://gitlab.com/gitlab-org/gitlab/-/issues/348243
milestone: '14.6'
type: development
group: group::source code
default_enabled: true
...@@ -546,14 +546,6 @@ RSpec.describe 'Git LFS API and storage' do ...@@ -546,14 +546,6 @@ RSpec.describe 'Git LFS API and storage' do
expect(lfs_object.reload.projects.pluck(:id)).to match_array([other_project.id, project.id]) expect(lfs_object.reload.projects.pluck(:id)).to match_array([other_project.id, project.id])
end end
context 'when feature flag is disabled' do
before do
stub_feature_flags(lfs_auto_link_fork_source: false)
end
it_behaves_like 'batch upload with existing LFS object'
end
end end
context 'when user does not have access to parent' do context 'when user does not have access to parent' do
......
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