From 4411353300cf8219d2b899785bc5103c549ba8cf Mon Sep 17 00:00:00 2001 From: GitLab Bot Date: Tue, 14 Jan 2020 06:07:51 +0000 Subject: [PATCH] Add latest changes from gitlab-org/gitlab@master --- .../clusters/concerns/application_core.rb | 2 +- .../cluster_application_initial_status.rb | 24 ++++++++++++++++++- ...ster_application_status_shared_examples.rb | 10 -------- 3 files changed, 24 insertions(+), 12 deletions(-) diff --git a/app/models/clusters/concerns/application_core.rb b/app/models/clusters/concerns/application_core.rb index f6431f5bac3..b94f2b15846 100644 --- a/app/models/clusters/concerns/application_core.rb +++ b/app/models/clusters/concerns/application_core.rb @@ -15,7 +15,7 @@ module Clusters def set_initial_status return unless not_installable? - self.status = status_states[:installable] if cluster&.application_helm_available? + self.status = status_states[:installable] if cluster&.application_helm_available? || Feature.enabled?(:managed_apps_local_tiller) end def can_uninstall? diff --git a/spec/support/shared_examples/models/cluster_application_initial_status.rb b/spec/support/shared_examples/models/cluster_application_initial_status.rb index 9775d87953c..030974c9aa0 100644 --- a/spec/support/shared_examples/models/cluster_application_initial_status.rb +++ b/spec/support/shared_examples/models/cluster_application_initial_status.rb @@ -6,8 +6,30 @@ shared_examples 'cluster application initial status specs' do subject { described_class.new(cluster: cluster) } + context 'local tiller feature flag is disabled' do + before do + stub_feature_flags(managed_apps_local_tiller: false) + end + + it 'sets a default status' do + expect(subject.status_name).to be(:not_installable) + end + end + + context 'local tiller feature flag is enabled' do + before do + stub_feature_flags(managed_apps_local_tiller: true) + end + + it 'sets a default status' do + expect(subject.status_name).to be(:installable) + end + end + context 'when application helm is scheduled' do before do + stub_feature_flags(managed_apps_local_tiller: false) + create(:clusters_applications_helm, :scheduled, cluster: cluster) end @@ -16,7 +38,7 @@ shared_examples 'cluster application initial status specs' do end end - context 'when application is scheduled' do + context 'when application helm is installed' do before do create(:clusters_applications_helm, :installed, cluster: cluster) end diff --git a/spec/support/shared_examples/models/cluster_application_status_shared_examples.rb b/spec/support/shared_examples/models/cluster_application_status_shared_examples.rb index a6653f89377..4bca37a4cd0 100644 --- a/spec/support/shared_examples/models/cluster_application_status_shared_examples.rb +++ b/spec/support/shared_examples/models/cluster_application_status_shared_examples.rb @@ -1,16 +1,6 @@ # frozen_string_literal: true shared_examples 'cluster application status specs' do |application_name| - describe '#status' do - let(:cluster) { create(:cluster, :provided_by_gcp) } - - subject { described_class.new(cluster: cluster) } - - it 'sets a default status' do - expect(subject.status_name).to be(:not_installable) - end - end - describe '#status_states' do let(:cluster) { create(:cluster, :provided_by_gcp) }