Commit 8fc9ff7f authored by Grzegorz Bizon's avatar Grzegorz Bizon

Fix environment deployment platform filter method

parent 2faf991f
...@@ -224,7 +224,7 @@ class Environment < ActiveRecord::Base ...@@ -224,7 +224,7 @@ class Environment < ActiveRecord::Base
end end
def deployment_platform def deployment_platform
project.deployment_platform(environment: self) project.deployment_platform(environment: self.name)
end end
private private
......
...@@ -79,7 +79,7 @@ module Gitlab ...@@ -79,7 +79,7 @@ module Gitlab
def common_query_context(environment, timeframe_start:, timeframe_end:) def common_query_context(environment, timeframe_start:, timeframe_end:)
base_query_context(timeframe_start, timeframe_end).merge({ base_query_context(timeframe_start, timeframe_end).merge({
ci_environment_slug: environment.slug, ci_environment_slug: environment.slug,
kube_namespace: environment.project.deployment_platform&.actual_namespace || '', kube_namespace: environment.deployment_platform&.actual_namespace || '',
environment_filter: %{container_name!="POD",environment="#{environment.slug}"} environment_filter: %{container_name!="POD",environment="#{environment.slug}"}
}) })
end end
......
...@@ -368,6 +368,32 @@ describe Environment do ...@@ -368,6 +368,32 @@ describe Environment do
end end
end end
describe '#deployment_platform' do
context 'when there is a deployment platform for environment' do
let!(:cluster) do
create(:cluster, :provided_by_gcp,
environment_scope: '*', projects: [project])
end
it 'finds a deployment platform' do
expect(environment.deployment_platform).to eq cluster.platform
end
end
context 'when there is no deployment platform for environment' do
it 'returns nil' do
expect(environment.deployment_platform).to be_nil
end
end
it 'checks deployment platforms associated with a project' do
expect(project).to receive(:deployment_platform)
.with(environment: environment.name)
environment.deployment_platform
end
end
describe '#terminals' do describe '#terminals' do
subject { environment.terminals } subject { environment.terminals }
......
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