Minor refactoring of Helm and ApplicationData

app/models/clusters/applications/helm.rb

@@ -17,20 +17,7 @@ module Clusters
       before_create :create_keys_and_certs
-      def create_keys_and_certs
-        ca_cert = Gitlab::Kubernetes::Helm::Certificate.generate_root
-        self.ca_key = ca_cert.key_string
-        self.ca_cert = ca_cert.cert_string
-      end
-
-      def ca_cert_obj
-        return unless has_ssl?
-
-        Gitlab::Kubernetes::Helm::Certificate
-          .from_strings(ca_key, ca_cert)
-      end
-
-      def issue_cert
+      def issue_client_cert
         ca_cert_obj.issue
       end
@@ -41,21 +28,42 @@ module Clusters
       end
       def install_command
-        tiller_cert = ca_cert_obj.issue(expires_in: Gitlab::Kubernetes::Helm::Certificate::INFINITE_EXPIRY)
-
         Gitlab::Kubernetes::Helm::InitCommand.new(
           name: name,
-          files: {
-            'ca.pem': ca_cert,
-            'cert.pem': tiller_cert.cert_string,
-            'key.pem': tiller_cert.key_string
-          }
+          files: files
         )
       end
       def has_ssl?
         ca_key.present? && ca_cert.present?
       end
+
+      private
+
+      def files
+        {
+          'ca.pem': ca_cert,
+          'cert.pem': tiller_cert.cert_string,
+          'key.pem': tiller_cert.key_string
+        }
+      end
+
+      def create_keys_and_certs
+        ca_cert = Gitlab::Kubernetes::Helm::Certificate.generate_root
+        self.ca_key = ca_cert.key_string
+        self.ca_cert = ca_cert.cert_string
+      end
+
+      def tiller_cert
+        @tiller_cert ||= ca_cert_obj.issue(expires_in: Gitlab::Kubernetes::Helm::Certificate::INFINITE_EXPIRY)
+      end
+
+      def ca_cert_obj
+        return unless has_ssl?
+
+        Gitlab::Kubernetes::Helm::Certificate
+          .from_strings(ca_key, ca_cert)
+      end
     end
   end
 end

app/models/clusters/concerns/application_data.rb

@@ -15,15 +15,8 @@ module Clusters
         def files
           @files ||= begin
             files = { 'values.yaml': values }
-            if cluster.application_helm.has_ssl?
-              ca_cert = cluster.application_helm.ca_cert
-              helm_cert = cluster.application_helm.issue_cert
-              files.merge!({
-                'ca.pem': ca_cert,
-                'cert.pem': helm_cert.cert_string,
-                'key.pem': helm_cert.key_string
-              })
-            end
+
+            files.merge!(certificate_files) if cluster.application_helm.has_ssl?
             files
           end
@@ -31,6 +24,22 @@ module Clusters
         private
+        def certificate_files
+          {
+            'ca.pem': ca_cert,
+            'cert.pem': helm_cert.cert_string,
+            'key.pem': helm_cert.key_string
+          }
+        end
+
+        def ca_cert
+          cluster.application_helm.ca_cert
+        end
+
+        def helm_cert
+          @helm_cert ||= cluster.application_helm.issue_client_cert
+        end
+
         def chart_values_file
           "#{Rails.root}/vendor/#{name}/values.yaml"
         end

spec/models/clusters/applications/helm_spec.rb

@@ -15,9 +15,9 @@ describe Clusters::Applications::Helm do
     it { is_expected.to contain_exactly(installed_cluster) }
   end
-  describe '#issue_cert' do
+  describe '#issue_client_cert' do
     let(:application) { create(:clusters_applications_helm) }
-    subject { application.issue_cert }
+    subject { application.issue_client_cert }
     it 'returns a new cert' do
       is_expected.to be_kind_of(Gitlab::Kubernetes::Helm::Certificate)