Commit 4e457ebf authored by Douglas Barbosa Alexandre's avatar Douglas Barbosa Alexandre

Merge branch 'remove_clusters_cte_feature_flag' into 'master'

Removes clusters_cte feature flag

Closes #64259

See merge request gitlab-org/gitlab-ce!30827
parents 1e5b8087 90aafe78
...@@ -12,19 +12,10 @@ module DeploymentPlatform ...@@ -12,19 +12,10 @@ module DeploymentPlatform
private private
def find_deployment_platform(environment) def find_deployment_platform(environment)
find_platform_kubernetes(environment) || find_platform_kubernetes_with_cte(environment) ||
find_instance_cluster_platform_kubernetes(environment: environment) find_instance_cluster_platform_kubernetes(environment: environment)
end end
def find_platform_kubernetes(environment)
if Feature.enabled?(:clusters_cte)
find_platform_kubernetes_with_cte(environment)
else
find_cluster_platform_kubernetes(environment: environment) ||
find_group_cluster_platform_kubernetes(environment: environment)
end
end
# EE would override this and utilize environment argument # EE would override this and utilize environment argument
def find_platform_kubernetes_with_cte(_environment) def find_platform_kubernetes_with_cte(_environment)
Clusters::ClustersHierarchy.new(self).base_and_ancestors Clusters::ClustersHierarchy.new(self).base_and_ancestors
...@@ -32,18 +23,6 @@ module DeploymentPlatform ...@@ -32,18 +23,6 @@ module DeploymentPlatform
.first&.platform_kubernetes .first&.platform_kubernetes
end end
# EE would override this and utilize environment argument
def find_cluster_platform_kubernetes(environment: nil)
clusters.enabled.default_environment
.last&.platform_kubernetes
end
# EE would override this and utilize environment argument
def find_group_cluster_platform_kubernetes(environment: nil)
Clusters::Cluster.enabled.default_environment.ancestor_clusters_for_clusterable(self)
.first&.platform_kubernetes
end
# EE would override this and utilize environment argument # EE would override this and utilize environment argument
def find_instance_cluster_platform_kubernetes(environment: nil) def find_instance_cluster_platform_kubernetes(environment: nil)
Clusters::Instance.new.clusters.enabled.default_environment Clusters::Instance.new.clusters.enabled.default_environment
......
...@@ -5,7 +5,7 @@ require 'rails_helper' ...@@ -5,7 +5,7 @@ require 'rails_helper'
describe DeploymentPlatform do describe DeploymentPlatform do
let(:project) { create(:project) } let(:project) { create(:project) }
shared_examples '#deployment_platform' do describe '#deployment_platform' do
subject { project.deployment_platform } subject { project.deployment_platform }
context 'with no Kubernetes configuration on CI/CD, no Kubernetes Service' do context 'with no Kubernetes configuration on CI/CD, no Kubernetes Service' do
...@@ -84,20 +84,4 @@ describe DeploymentPlatform do ...@@ -84,20 +84,4 @@ describe DeploymentPlatform do
end end
end end
end end
context 'legacy implementation' do
before do
stub_feature_flags(clusters_cte: false)
end
include_examples '#deployment_platform'
end
context 'CTE implementation' do
before do
stub_feature_flags(clusters_cte: true)
end
include_examples '#deployment_platform'
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