Commit 4943065b authored by Marius Bobin's avatar Marius Bobin

Remove expand_names_for_cross_pipeline_artifacts feature flag

It removes the feature flag, enabling the behavior by default.
parent 025f778f
...@@ -2164,10 +2164,7 @@ build_job: ...@@ -2164,10 +2164,7 @@ build_job:
``` ```
Environment variables support for `project:`, `job:`, and `ref` was [introduced](https://gitlab.com/gitlab-org/gitlab/-/issues/202093) Environment variables support for `project:`, `job:`, and `ref` was [introduced](https://gitlab.com/gitlab-org/gitlab/-/issues/202093)
in GitLab 13.3. This is under development, but it is ready for production use. It is deployed in GitLab 13.3. [Feature flag removed](https://gitlab.com/gitlab-org/gitlab/-/issues/235761) in GitLab 13.4.
behind the `ci_expand_names_for_cross_pipeline_artifacts` feature flag, which is **disabled by default**.
[GitLab administrators with access to the GitLab Rails console](../../administration/feature_flags.md)
can enable it for your instance.
For example: For example:
......
...@@ -52,9 +52,7 @@ module EE ...@@ -52,9 +52,7 @@ module EE
def build_cross_dependency_relationship_fragment(dependency, search_scope) def build_cross_dependency_relationship_fragment(dependency, search_scope)
args = dependency.values_at(:job, :ref, :project) args = dependency.values_at(:job, :ref, :project)
if ::Gitlab::Ci::Features.expand_names_for_cross_pipeline_artifacts?(project) args = args.map { |value| ExpandVariables.expand(value, processable_variables) }
args = args.map { |value| ExpandVariables.expand(value, processable_variables) }
end
dep_id = search_scope.max_build_id_by(*args) dep_id = search_scope.max_build_id_by(*args)
model_class.id_in(dep_id) model_class.id_in(dep_id)
......
...@@ -105,14 +105,6 @@ RSpec.describe Ci::BuildDependencies do ...@@ -105,14 +105,6 @@ RSpec.describe Ci::BuildDependencies do
end end
it { is_expected.to contain_exactly(dependency) } it { is_expected.to contain_exactly(dependency) }
context 'with the feature flag disabled' do
before do
stub_feature_flags(ci_expand_names_for_cross_pipeline_artifacts: false)
end
it { is_expected.to be_empty }
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