Commit ee202720 authored by Kamil Trzciński's avatar Kamil Trzciński

Merge branch 'remove-zip-deployments-ff' into 'master'

Remove zip_pages_deployment feature flag

See merge request gitlab-org/gitlab!47633
parents 38f7c0a8 db9f61c7
......@@ -125,8 +125,6 @@ module Projects
end
def create_pages_deployment(artifacts_path, build)
return unless Feature.enabled?(:zip_pages_deployments, project, default_enabled: true)
# we're using the full archive and pages daemon needs to read it
# so we want the total count from entries, not only "public/" directory
# because it better approximates work we need to do before we can serve the site
......
---
name: zip_pages_deployments
introduced_by_url: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/42834
rollout_issue_url: https://gitlab.com/gitlab-org/gitlab/-/issues/245308
milestone: '13.5'
type: development
group: group::release management
default_enabled: true
......@@ -105,16 +105,6 @@ RSpec.describe Projects::UpdatePagesService do
end
end
it 'does not create deployment when zip_pages_deployments feature flag is disabled' do
stub_feature_flags(zip_pages_deployments: false)
expect do
expect(execute).to eq(:success)
end.not_to change { project.pages_deployments.count }
expect(project.pages_metadatum.reload.pages_deployment_id).to be_nil
end
it 'limits pages size' do
stub_application_setting(max_pages_size: 1)
expect(execute).not_to eq(:success)
......
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