Commit 06c45bd2 authored by Maxime Orefice's avatar Maxime Orefice

Remove ci_synchronous_artifact_parsing feature flag

Changelog: other
parent 2543b6b5
...@@ -109,10 +109,6 @@ module Ci ...@@ -109,10 +109,6 @@ module Ci
end end
def parse_artifact(artifact) def parse_artifact(artifact)
unless Feature.enabled?(:ci_synchronous_artifact_parsing, project, default_enabled: true)
return success
end
case artifact.file_type case artifact.file_type
when 'dotenv' then parse_dotenv_artifact(artifact) when 'dotenv' then parse_dotenv_artifact(artifact)
else success else success
......
---
name: ci_synchronous_artifact_parsing
introduced_by_url: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/26247
rollout_issue_url:
milestone: '12.9'
type: development
group: group::release
default_enabled: true
...@@ -175,18 +175,6 @@ RSpec.describe Ci::JobArtifacts::CreateService do ...@@ -175,18 +175,6 @@ RSpec.describe Ci::JobArtifacts::CreateService do
hash_including('key' => 'KEY1', 'value' => 'VAR1', 'source' => 'dotenv'), hash_including('key' => 'KEY1', 'value' => 'VAR1', 'source' => 'dotenv'),
hash_including('key' => 'KEY2', 'value' => 'VAR2', 'source' => 'dotenv')) hash_including('key' => 'KEY2', 'value' => 'VAR2', 'source' => 'dotenv'))
end end
context 'when ci_synchronous_artifact_parsing feature flag is disabled' do
before do
stub_feature_flags(ci_synchronous_artifact_parsing: false)
end
it 'does not call parse service' do
expect(Ci::ParseDotenvArtifactService).not_to receive(:new)
expect(subject[:status]).to eq(:success)
end
end
end end
context 'when artifact_type is metrics' do context 'when artifact_type is metrics' 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