helm-wrapper 1.7.2 → 1.7.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 0faf78c64a0493547119ad800a1e61c689a219334117171fb933dbc27381cf76
-  data.tar.gz: cdaa62c9e7f163e4b85ce57a0e4b324a6c15223d3a304da6da14361337058abc
+  metadata.gz: 10fea1f8e96fea9ba1aebbbab0ff1677082ab65e6203865423dd5306d5731227
+  data.tar.gz: f7f59f2ad0dd69bb40f5eff053c4d307dec467e85b916642887eb19135cef8f6
 SHA512:
-  metadata.gz: 1fba730588a8ef8e9e7e846c8eae9c89c5c48bdd272d762c950596c39adb0de59f792d56e0c1cf873a453696d8c935c70f1f45be5b2919a56205aa8b0d2ac344
-  data.tar.gz: '0836f6da655a96ba46019767c6adce3df1e5202519cac699af70cec2d83c7d97971a3cadd3f4b8fb6f811d0ae3ee82574c7a6b7aa1214dc0ed847956895ab0de'
+  metadata.gz: f2213f807e1c09d55fa671fbc4d1521fea4a13e567a1006dd16dfb73a667708117841301e0b664e7214d24c5c16ebaf2dab920a2e6a056b88254134d2f0e8d98
+  data.tar.gz: 265cc10cf924f79cbec060bb0389e56f7c792e766a3f991859183cdead17063aacc0b472884a85192a417931c2570382b97b83bfc5f6c0b9b9158d21b2f64486

data/lib/helm-wrapper/shared/runner.rb

@@ -5,36 +5,29 @@ require 'open3'
 ###############################################################################
 
 module HelmWrapper
-
-###############################################################################
+  #############################################################################
 
   module Shared
-
-###############################################################################
+    ###########################################################################
 
     class Runner
-
-###############################################################################
+      #########################################################################
 
       include HelmWrapper::Shared::Logging
 
-###############################################################################
+      #########################################################################
 
-      @@repo_actions = [ "registry", "repo" ]
+      @@repo_actions = %w[registry repo]
 
-###############################################################################
+      #########################################################################
 
       @auths_attempted = false
 
-###############################################################################
+      #########################################################################
 
-      attr_reader :auths
-      attr_reader :binary
-      attr_reader :chart
-      attr_reader :config
-      attr_reader :repos
+      attr_reader :auths, :binary, :chart, :config, :repos
 
-###############################################################################
+      #########################################################################
 
       def initialize(binary:, chart:, config: nil)
         @binary = binary
@@ -45,10 +38,10 @@ module HelmWrapper
         @repos = false
       end
 
-###############################################################################
+      #########################################################################
 
-      def init_auths()
-        logger.fatal("Cannot initialise authenticators without a valid configuration!") if @config.nil?
+      def init_auths
+        logger.fatal('Cannot initialise authenticators without a valid configuration!') if @config.nil?
 
         @auths_attempted = true
 
@@ -57,81 +50,84 @@ module HelmWrapper
         @auths = true
       end
 
-###############################################################################
+      #########################################################################
 
       def init_repos(force: false)
-        ENV["HELM_EXPERIMENTAL_OCI"] = "true"
-
         @chart.oci.each_with_index do |oci_repo, oci_index|
-          logger.info("Logging into OCI repository: #{oci_repo["name"]}")
+          logger.info("Logging into OCI repository: #{oci_repo['name']}")
 
-          username = from_environment(variable: oci_repo["username"])
-          password = from_environment(variable: oci_repo["password"])
+          username = from_environment(variable: oci_repo['username'])
+          password = from_environment(variable: oci_repo['password'])
 
           @chart.oci_active(active: true, index: oci_index)
 
-          parameters = Array.new
-          parameters.append("login")
-          parameters.append("\"#{oci_repo["url"]}\"")
+          parameters = []
+          parameters.append('login')
+          parameters.append("\"#{oci_repo['url']}\"")
           parameters.append("--username=\"#{username}\"")
-          parameters.append("--password-stdin")
-          logger.fatal("Failed to login to Helm OCI repository: #{oci_repo["name"]}, url: #{oci_repo["url"]}") unless run(action: "registry", parameters: parameters, stdin: password)
+          parameters.append('--password-stdin')
+          logger.fatal("Failed to login to Helm OCI repository: #{oci_repo['name']}, url: #{oci_repo['url']}") unless run(
+            action: 'registry', parameters: parameters, stdin: password
+          )
         end
 
-        if @chart.artefact.length > 0 then
+        if @chart.artefact.length > 0
 
           @chart.artefact.each_with_index do |artefact_repo, artefact_index|
-            logger.info("Adding artefact repository: #{artefact_repo["name"]}")
+            logger.info("Adding artefact repository: #{artefact_repo['name']}")
 
-            username = artefact_repo["username"].nil? ? nil : from_environment(variable: artefact_repo["username"])
-            password = artefact_repo["password"].nil? ? nil : from_environment(variable: artefact_repo["password"])
+            username = artefact_repo['username'].nil? ? nil : from_environment(variable: artefact_repo['username'])
+            password = artefact_repo['password'].nil? ? nil : from_environment(variable: artefact_repo['password'])
 
             @chart.artefact_active(active: true, index: artefact_index)
 
-            parameters = Array.new
-            parameters.append("add")
-            parameters.append("\"#{artefact_repo["name"]}\"")
-            parameters.append("\"#{artefact_repo["url"]}\"")
+            parameters = []
+            parameters.append('add')
+            parameters.append("\"#{artefact_repo['name']}\"")
+            parameters.append("\"#{artefact_repo['url']}\"")
             parameters.append("--username=\"#{username}\"") unless username.nil?
             parameters.append("--password=\"#{password}\"") unless password.nil?
-            parameters.append("--force-update") if force
-            logger.fatal("Failed to add Helm repository: #{artefact_repo["name"]}, url: #{artefact_repo["url"]}") unless run(action: "repo", parameters: parameters)
+            parameters.append('--force-update') if force
+            logger.fatal("Failed to add Helm repository: #{artefact_repo['name']}, url: #{artefact_repo['url']}") unless run(
+              action: 'repo', parameters: parameters
+            )
           end
 
-          logger.info("Updating artefact repositories...")
-          logger.fatal("Failed to update Helm artefact repositories!") unless run(action: "repo", parameters: Array.new([ "update" ]))
+          logger.info('Updating artefact repositories...')
+          logger.fatal('Failed to update Helm artefact repositories!') unless run(action: 'repo',
+                                                                                  parameters: Array.new(['update']))
         end
 
         @repos = true
       end
 
-###############################################################################
+      #########################################################################
 
       def clean(repos: true)
         clean_repos if repos
         clean_auths if @auths_attempted
       end
 
-###############################################################################
+      #########################################################################
 
-      def delete()
-        logger.fatal("Cannot Helm delete without a valid configuration!") if @config.nil?
-        logger.fatal("Cannot Helm delete before initialising authenticators!") unless auths
+      def delete
+        logger.fatal('Cannot Helm delete without a valid configuration!') if @config.nil?
+        logger.fatal('Cannot Helm delete before initialising authenticators!') unless auths
 
-        parameters = Array.new
+        parameters = []
         parameters.append("--namespace=\"#{@config.namespace}\"")
         parameters.append("\"#{@config.release}\"")
 
-        logger.fatal("Helm delete failed!") unless run(action: "delete", parameters: parameters)
+        logger.fatal('Helm delete failed!') unless run(action: 'delete', parameters: parameters)
       end
 
-###############################################################################
+      #########################################################################
 
-      def template()
-        logger.fatal("Cannot Helm template without a valid configuration!") if @config.nil?
-        logger.fatal("Cannot Helm template before initialising repositories!") unless repos
+      def template
+        logger.fatal('Cannot Helm template without a valid configuration!') if @config.nil?
+        logger.fatal('Cannot Helm template before initialising repositories!') unless repos
 
-        parameters = Array.new
+        parameters = []
         parameters.append("--namespace=\"#{@config.namespace}\"")
         parameters.append("\"#{@config.release}\"")
         parameters.append("\"#{@chart.name}\"")
@@ -139,72 +135,72 @@ module HelmWrapper
         parameters.concat(variable_files)
         parameters.concat(variable_strings)
 
-        logger.fatal("Helm template failed!") unless run(action: "template", parameters: parameters)
+        logger.fatal('Helm template failed!') unless run(action: 'template', parameters: parameters)
       end
 
-###############################################################################
+      #########################################################################
 
       def upgrade(install: true)
-        logger.fatal("Cannot Helm upgrade without a valid configuration!") if @config.nil?
-        logger.fatal("Cannot Helm upgrade before initialising authenticators!") unless auths
-        logger.fatal("Cannot Helm upgrade before initialising repositories!") unless repos
+        logger.fatal('Cannot Helm upgrade without a valid configuration!') if @config.nil?
+        logger.fatal('Cannot Helm upgrade before initialising authenticators!') unless auths
+        logger.fatal('Cannot Helm upgrade before initialising repositories!') unless repos
 
-        parameters = Array.new
+        parameters = []
         parameters.append("--namespace=\"#{@config.namespace}\"")
-        parameters.append("--install") if install
+        parameters.append('--install') if install
         parameters.append("\"#{@config.release}\"")
         parameters.append("\"#{@chart.name}\"")
-        parameters.append("--atomic") if @config.atomic
-        parameters.append("--wait") if @config.wait
-        parameters.append("--timeout=\"#{@config.timeout}\"") if (@config.atomic or @config.wait)
+        parameters.append('--atomic') if @config.atomic
+        parameters.append('--wait') if @config.wait
+        parameters.append("--timeout=\"#{@config.timeout}\"") if @config.atomic or @config.wait
         parameters.append("--version=\"#{@chart.version}\"") unless @chart.version.strip.empty?
         parameters.concat(variable_files)
         parameters.concat(variable_strings)
 
-        logger.fatal("Helm upgrade failed!") unless run(action: "upgrade", parameters: parameters)
+        logger.fatal('Helm upgrade failed!') unless run(action: 'upgrade', parameters: parameters)
       end
 
-###############################################################################
+      #########################################################################
 
-      def lint()
-        parameters = Array.new
+      def lint
+        parameters = []
         parameters.append("\"#{@chart.path}\"")
 
-        logger.fatal("Helm validate failed!") unless run(action: "lint", parameters: parameters)
+        logger.fatal('Helm validate failed!') unless run(action: 'lint', parameters: parameters)
       end
 
-###############################################################################
+      #########################################################################
 
-      def package(destination: ".")
-        parameters = Array.new
+      def package(destination: '.')
+        parameters = []
         parameters.append("--destination=\"#{destination}\"")
         parameters.append("\"#{@chart.path}\"")
 
-        logger.fatal("Helm package failed!") unless run(action: "package", parameters: parameters)
+        logger.fatal('Helm package failed!') unless run(action: 'package', parameters: parameters)
       end
 
-###############################################################################
+      #########################################################################
 
-      def push(source: ".", destination:)
-        logger.fatal("Cannot Helm push before initialising repositories!") unless repos
+      def push(destination:, source: '.')
+        logger.fatal('Cannot Helm push before initialising repositories!') unless repos
 
         package_name = "#{@chart.name}-#{@chart.version}.tgz"
         package_path = File.join(source, package_name)
 
         logger.fatal("Must package before pushing! Package: #{package_path} not found!") unless File.file?(package_path)
 
-        parameters = Array.new
+        parameters = []
         parameters.append("\"#{package_path}\"")
         parameters.append("\"#{destination}\"")
 
-        logger.fatal("Helm push failed!") unless run(action: "push", parameters: parameters)
+        logger.fatal('Helm push failed!') unless run(action: 'push', parameters: parameters)
       end
 
-###############################################################################
+      #########################################################################
 
-    private
+      private
 
-###############################################################################
+      #########################################################################
 
       def clean_auths
         @auths = false
@@ -212,86 +208,88 @@ module HelmWrapper
         @auths_attempted = false
       end
 
-###############################################################################
+      #########################################################################
 
       def clean_repos
         @repos = false
 
         @chart.oci.each do |oci_repo|
-          if oci_repo["active"] then
-            logger.info("Logging out of OCI repository: #{oci_repo["name"]}")
+          next unless oci_repo['active']
 
-            parameters = Array.new
-            parameters.append("logout")
-            parameters.append("\"#{oci_repo["url"]}\"")
-            logger.error("Failed to logout of Helm OCI repository: #{oci_repo["name"]}, url: #{oci_repo["url"]}") unless run(action: "registry", parameters: parameters)
-          end
+          logger.info("Logging out of OCI repository: #{oci_repo['name']}")
+
+          parameters = []
+          parameters.append('logout')
+          parameters.append("\"#{oci_repo['url']}\"")
+          logger.error("Failed to logout of Helm OCI repository: #{oci_repo['name']}, url: #{oci_repo['url']}") unless run(
+            action: 'registry', parameters: parameters
+          )
         end
 
         @chart.artefact.each do |artefact_repo|
-          if artefact_repo["active"] then
-            if artefact_repo["password"].nil? then
-              logger.info("Not removing artefact repository: #{artefact_repo["name"]}, no credentials stored.")
+          if artefact_repo['active']
+            if artefact_repo['password'].nil?
+              logger.info("Not removing artefact repository: #{artefact_repo['name']}, no credentials stored.")
             else
-              logger.info("Removing artefact repository: #{artefact_repo["name"]}")
-
-              parameters = Array.new
-              parameters.append("remove")
-              parameters.append("\"#{artefact_repo["name"]}\"")
-              logger.error("Failed to remove Helm repository: #{artefact_repo["name"]}, url: #{artefact_repo["url"]}") unless run(action: "repo", parameters: parameters)
+              logger.info("Removing artefact repository: #{artefact_repo['name']}")
+
+              parameters = []
+              parameters.append('remove')
+              parameters.append("\"#{artefact_repo['name']}\"")
+              logger.error("Failed to remove Helm repository: #{artefact_repo['name']}, url: #{artefact_repo['url']}") unless run(
+                action: 'repo', parameters: parameters
+              )
             end
           end
         end
-
-        ENV.delete("HELM_EXPERIMENTAL_OCI")
       end
 
-###############################################################################
+      #########################################################################
 
       def variable_files
-        result = Array.new
+        result = []
 
         @config.variables.files.each do |file|
           result.append("--values=\"#{file}\"")
         end
 
-        return result
+        result
       end
 
-###############################################################################
+      #########################################################################
 
       def variable_strings
-        result = Array.new
+        result = []
 
         @config.variables.values.each do |key, value|
-          result.append("--set=\"#{key.to_s}=#{value}\"")
+          result.append("--set=\"#{key}=#{value}\"")
         end
 
-        return result
+        result
       end
 
-###############################################################################
+      #########################################################################
 
-      def run(action:, parameters: Array.new, stdin: nil)
+      def run(action:, parameters: [], stdin: nil)
         result = false
 
-        parameters.reject! { |item| not item.kind_of?(String) or item.strip.empty? }
+        parameters.reject! { |item| !item.is_a?(String) or item.strip.empty? }
 
-        cmdline = [ "\"#{@binary.path}\"", action ].concat(parameters).join(" ")
+        cmdline = ["\"#{@binary.path}\"", action].concat(parameters).join(' ')
 
         if @@repo_actions.include?(action)
           begin
-            out, status = Open3.capture2e(cmdline, :stdin_data=>stdin)
-            logger.debug("Helm output:")
+            out, status = Open3.capture2e(cmdline, stdin_data: stdin)
+            logger.debug('Helm output:')
             logger.debug(out)
             result = status.success?
-          rescue
+          rescue StandardError
             result = false
           end
         else
           logger.info("Starting Helm, action: #{action}")
 
-          fatal("Running Helm with data for stdin can only be used when configuring repos!") unless stdin.nil?
+          fatal('Running Helm with data for stdin can only be used when configuring repos!') unless stdin.nil?
           puts("\n" + ('#' * 80) + "\n\n")
 
           result = system(cmdline) || false
@@ -299,28 +297,25 @@ module HelmWrapper
           puts("\n" + ('#' * 80) + "\n\n")
         end
 
-        return result
+        result
       end
 
-###############################################################################
+      #########################################################################
 
       def from_environment(variable:)
         logger.fatal("Environment variable: #{variable} does not exist") unless ENV.key?(variable)
         logger.fatal("Environment variable: #{variable} is blank!") if ENV[variable].strip.empty?
 
-        return ENV[variable].strip
+        ENV[variable].strip
       end
 
-###############################################################################
-
+      #########################################################################
     end
 
-###############################################################################
-
+    ###########################################################################
   end
 
-###############################################################################
-
+  #############################################################################
 end
 
 ###############################################################################

data/lib/helm-wrapper/version.rb

@@ -1,13 +1,11 @@
 ###############################################################################
 
 module HelmWrapper
+  #############################################################################
 
-###############################################################################
-
-  VERSION = '1.7.2'
-
-###############################################################################
+  VERSION = '1.7.3'
 
+  #############################################################################
 end
 
 ###############################################################################

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: helm-wrapper
 version: !ruby/object:Gem::Version
-  version: 1.7.2
+  version: 1.7.3
 platform: ruby
 authors:
 - Richard Lees
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2022-07-17 00:00:00.000000000 Z
+date: 2022-12-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
@@ -100,7 +100,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.3.7
+rubygems_version: 3.3.26
 signing_key: 
 specification_version: 4
 summary: A ruby wrapper for managing Helm binaries and chart deployment.