diff options
author | Thong Kuah <tkuah@gitlab.com> | 2019-07-01 11:24:20 +0300 |
---|---|---|
committer | Thong Kuah <tkuah@gitlab.com> | 2019-07-08 00:13:21 +0300 |
commit | 1668f40f430c656ed9c20898605db21a66cb5937 (patch) | |
tree | d26ae7edb98df825116cd767dd1f85f2c5a452a6 /spec/models/deployment_spec.rb | |
parent | 72339077f7d79eeb9d0612c55236aa4aa6da4084 (diff) |
Remove fallback to project.deployment_platform
This improves query performance of
MergeRequestsController#ci_environments_status a lot.
However this means old deployments that deployed to kubernetes clusters
with prometheus installations will no longer show performance metrics as
we cannot backfill cluster_id from deployment_platform with certainty
(clusters may be edited/added/deleted, which changes the results of
deployment_platform).
Diffstat (limited to 'spec/models/deployment_spec.rb')
-rw-r--r-- | spec/models/deployment_spec.rb | 12 |
1 files changed, 0 insertions, 12 deletions
diff --git a/spec/models/deployment_spec.rb b/spec/models/deployment_spec.rb index 713fb647708..a926e1913a7 100644 --- a/spec/models/deployment_spec.rb +++ b/spec/models/deployment_spec.rb @@ -341,18 +341,6 @@ describe Deployment do it { is_expected.to be_truthy } end - - context 'fallback deployment platform' do - let(:cluster) { create(:cluster, :provided_by_user, environment_scope: '*', projects: [deployment.project]) } - let!(:prometheus) { create(:clusters_applications_prometheus, :installed, cluster: cluster) } - - before do - expect(deployment.project).to receive(:deployment_platform).and_return(cluster.platform) - expect(cluster.application_prometheus).to receive(:can_query?).and_return(true) - end - - it { is_expected.to be_truthy } - end end end |