Introduce CI/CD variables collection class

1 files changed, 9 insertions, 11 deletions

diff --git a/app/models/project_services/kubernetes_service.rb b/app/models/project_services/kubernetes_service.rb
index 8ba07173c74..5c1a1063baa 100644
--- a/app/models/project_services/kubernetes_service.rb
+++ b/app/models/project_services/kubernetes_service.rb
@@ -100,19 +100,17 @@ class KubernetesService < DeploymentService
   def predefined_variables
     config = YAML.dump(kubeconfig)
 
-    variables = [
-      { key: 'KUBE_URL', value: api_url, public: true },
-      { key: 'KUBE_TOKEN', value: token, public: false },
-      { key: 'KUBE_NAMESPACE', value: actual_namespace, public: true },
-      { key: 'KUBECONFIG', value: config, public: false, file: true }
-    ]
-
-    if ca_pem.present?
-      variables << { key: 'KUBE_CA_PEM', value: ca_pem, public: true }
-      variables << { key: 'KUBE_CA_PEM_FILE', value: ca_pem, public: true, file: true }
+    variables = Gitlab::Ci::Variables::Collection.new.tap do |collection|
+      collection.append(key: 'KUBE_URL', value: api_url, public: true)
+      collection.append(key: 'KUBE_TOKEN', value: token, public: false)
+      collection.append(key: 'KUBE_NAMESPACE', value: actual_namespace, public: true)
+      collection.append(key: 'KUBECONFIG', value: config, public: false, file: true)
+
+      collection.append(key: 'KUBE_CA_PEM', value: ca_pem, public: true)
+      collection.append(key: 'KUBE_CA_PEM_FILE', value: ca_pem, public: true, file: true)
     end
 
-    variables
+    variables.to_hash
   end
 
   # Constructs a list of terminals from the reactive cache