Commit 32d1e07b authored by Alishan Ladhani's avatar Alishan Ladhani

Remove deployment_approvals feature flag

Changelog: added
EE: true
parent fb071844
---
name: deployment_approvals
introduced_by_url: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/74932
rollout_issue_url: https://gitlab.com/gitlab-org/gitlab/-/issues/347342
milestone: '14.6'
type: development
group: group::release
default_enabled: false
...@@ -448,7 +448,10 @@ curl --header "PRIVATE-TOKEN: <your_access_token>" "https://gitlab.example.com/a ...@@ -448,7 +448,10 @@ curl --header "PRIVATE-TOKEN: <your_access_token>" "https://gitlab.example.com/a
## Approve or reject a blocked deployment **(PREMIUM)** ## Approve or reject a blocked deployment **(PREMIUM)**
> [Introduced](https://gitlab.com/gitlab-org/gitlab/-/issues/343864) in GitLab 14.7 [with a flag](../administration/feature_flags.md) named `deployment_approvals`. Disabled by default. This feature is not ready for production use. > - [Introduced](https://gitlab.com/gitlab-org/gitlab/-/issues/343864) in GitLab 14.7 [with a flag](../administration/feature_flags.md) named `deployment_approvals`. Disabled by default.
> - [Feature flag removed](https://gitlab.com/gitlab-org/gitlab/-/issues/347342) in GitLab 14.8.
See [Deployment Approvals](../ci/environments/deployment_approvals.md) for more information about this feature.
```plaintext ```plaintext
POST /projects/:id/deployments/:deployment_id/approval POST /projects/:id/deployments/:deployment_id/approval
......
...@@ -7,7 +7,8 @@ description: Require approvals prior to deploying to a Protected Environment ...@@ -7,7 +7,8 @@ description: Require approvals prior to deploying to a Protected Environment
# Deployment approvals **(PREMIUM)** # Deployment approvals **(PREMIUM)**
> [Introduced](https://gitlab.com/gitlab-org/gitlab/-/issues/343864) in GitLab 14.7 with a flag named `deployment_approvals`. Disabled by default. > - [Introduced](https://gitlab.com/gitlab-org/gitlab/-/issues/343864) in GitLab 14.7 with a flag named `deployment_approvals`. Disabled by default.
> - [Feature flag removed](https://gitlab.com/gitlab-org/gitlab/-/issues/347342) in GitLab 14.8.
WARNING: WARNING:
This feature is in an alpha stage and subject to change without prior notice. This feature is in an alpha stage and subject to change without prior notice.
......
...@@ -80,8 +80,6 @@ module EE ...@@ -80,8 +80,6 @@ module EE
end end
def needs_approval? def needs_approval?
return false unless ::Feature.enabled?(:deployment_approvals, project, default_enabled: :yaml)
required_approval_count > 0 required_approval_count > 0
end end
......
...@@ -11,7 +11,7 @@ module EE ...@@ -11,7 +11,7 @@ module EE
end end
resource :projects, requirements: ::API::API::NAMESPACE_OR_PROJECT_REQUIREMENTS do resource :projects, requirements: ::API::API::NAMESPACE_OR_PROJECT_REQUIREMENTS do
desc 'Approve or reject a blocked deployment' do desc 'Approve or reject a blocked deployment' do
detail 'This feature is gated behind the :deployment_approvals feature flag.' detail 'This feature was introduced in GitLab 14.8.'
success ::API::Entities::Deployments::Approval success ::API::Entities::Deployments::Approval
end end
params do params do
......
...@@ -259,14 +259,6 @@ RSpec.describe Environment, :use_clean_rails_memory_store_caching do ...@@ -259,14 +259,6 @@ RSpec.describe Environment, :use_clean_rails_memory_store_caching do
let(:required_approval_count) { 1 } let(:required_approval_count) { 1 }
it { is_expected.to be_truthy } it { is_expected.to be_truthy }
context 'and deployment_approvals feature flag turned off' do
before do
stub_feature_flags(deployment_approvals: false)
end
it { is_expected.to be_falsey }
end
end end
context 'with no approvals required' do context 'with no approvals required' 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