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

Merge branch 'zip_pages_deployments_true' into 'master'

Enable zip_pages_deployments by default

See merge request gitlab-org/gitlab!47763
parents 83d90b96 2aae0765
......@@ -125,7 +125,7 @@ module Projects
end
def create_pages_deployment(artifacts_path, build)
return unless Feature.enabled?(:zip_pages_deployments, project)
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
......@@ -146,10 +146,6 @@ module Projects
project.id,
deployment.id
)
rescue => e
# we don't want to break current pages deployment process if something goes wrong
# TODO: remove this rescue as part of https://gitlab.com/gitlab-org/gitlab/-/issues/245308
Gitlab::ErrorTracking.track_and_raise_for_dev_exception(e)
end
def latest?
......
---
title: Store pages content in zip format
merge_request: 47763
author:
type: added
......@@ -5,4 +5,4 @@ rollout_issue_url: https://gitlab.com/gitlab-org/gitlab/-/issues/245308
milestone: '13.5'
type: development
group: group::release management
default_enabled: false
default_enabled: true
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