Commit 6e3867e1 authored by Shinya Maeda's avatar Shinya Maeda

Remove cd_skipped_deployment_status feature flag

This commit removes the feature flag as it's confirmed
deemed stable.
parent c0f5ca55
...@@ -388,12 +388,8 @@ module Ci ...@@ -388,12 +388,8 @@ module Ci
end end
after_transition any => [:skipped, :canceled] do |build, transition| after_transition any => [:skipped, :canceled] do |build, transition|
if Feature.enabled?(:cd_skipped_deployment_status, build.project) if transition.to_name == :skipped
if transition.to_name == :skipped build.deployment&.skip
build.deployment&.skip
else
build.deployment&.cancel
end
else else
build.deployment&.cancel build.deployment&.cancel
end end
......
---
title: Add skipped status to deployments
merge_request: 51068
author:
type: added
name: cd_skipped_deployment_status
introduced_by_url: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/46614
rollout_issue_url: https://gitlab.com/gitlab-org/gitlab/-/issues/283884
milestone: '13.6'
type: development
group: group::release
default_enabled: false
...@@ -1165,26 +1165,12 @@ RSpec.describe Ci::Build do ...@@ -1165,26 +1165,12 @@ RSpec.describe Ci::Build do
end end
context 'when transits to skipped' do context 'when transits to skipped' do
context 'when cd_skipped_deployment_status is disabled' do before do
before do build.skip!
stub_feature_flags(cd_skipped_deployment_status: false)
build.skip!
end
it 'transits deployment status to canceled' do
expect(deployment).to be_canceled
end
end end
context 'when cd_skipped_deployment_status is enabled' do it 'transits deployment status to skipped' do
before do expect(deployment).to be_skipped
stub_feature_flags(cd_skipped_deployment_status: project)
build.skip!
end
it 'transits deployment status to skipped' do
expect(deployment).to be_skipped
end
end end
end end
......
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