helm_upgrade_logs 0.3.3 → 0.3.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c152efac3e0107333fb8a9101cb06f9eede40e5bfeb747b3815e3aca8a0d02b7
-  data.tar.gz: c6ef07655615775aebbe6bb347873905b9d5a43dc5a1ef3fcbfbfeaeb28c2d77
+  metadata.gz: ff847ee96c1dbced9f3dbd122b6245f695fe599462d7e1899d1dd09e483051cf
+  data.tar.gz: a3b10b6e66cfbb71ddeb5490f9073fd868868cf9b0afb3e35c620f01ee17e6ad
 SHA512:
-  metadata.gz: 8aa9f87875f45d174c337314988546e66d4af1bcb279b471a278eb7d826d4eb21362a9a060466abb7e86e27a9902cb9778593f7102e4fbbd82103a72686e68ed
-  data.tar.gz: 2dc02dc6e520c3997957885f68d180a88889132fbf74e3f4f8f01331b92cf3f6ee382c7e98da87557bde798e2854db7eaa5b28ff25c6670c41a7606fc2b94bdb
+  metadata.gz: adb0f2c697a023259f264cb6af3580a63f44448b025cb91400a8733b34ed3731e656807f40f52a376ac6688eb0fb287a73ec2b87c4cda23f257bfa30e8e797e4
+  data.tar.gz: '09bcd240fd9519e938c1fe0112907e922a57f21d2a67215f1f42270a8f67a6fecc5501cadd3968a07abaf4f2c73e358b77b79e95db976d2e44d132e51ac57ed0'

data/CHANGELOG.md

@@ -1,3 +1,7 @@
+## [0.3.4]
+
+- Only log to folder if `helm_upgrade_logs_error_msg` env variable set
+
 ## [0.3.3]
 
 - Run force kill on processes to ensure they stop

data/exe/helm_upgrade_logs

@@ -31,7 +31,7 @@ rescue Errno::ECHILD
   raise HelmUpgradeLogs::Error, "Failed to find logs before helm finished"
 end
 
-FileUtils.mkdir_p UPGRADE_LOG_FOLDER
+FileUtils.mkdir_p(UPGRADE_LOG_FOLDER) if ENV["helm_upgrade_logs_error_msg"]
 
 while Process.waitpid(@helm_pid, Process::WNOHANG).nil?
   pods = read_pods - pods_before_upgrade
@@ -46,11 +46,13 @@ while Process.waitpid(@helm_pid, Process::WNOHANG).nil?
       std_out_pid = Process.spawn(
         add_ns("kubectl logs #{pod} -f --all-containers --prefix --ignore-errors=true --timestamps=true")
       )
+      @pod_pids["#{pod}_stdout"] = std_out_pid
+      next unless ENV["helm_upgrade_logs_error_msg"]
+
       logfile_pid = Process.spawn(
         add_ns("kubectl logs #{pod} -f --all-containers --prefix --ignore-errors=true --timestamps=true"),
         out: "#{UPGRADE_LOG_FOLDER}/#{pod.gsub("/", "_")}.log"
       )
-      @pod_pids["#{pod}_stdout"] = std_out_pid
       @pod_pids["#{pod}_fileout"] = logfile_pid
     end
   end
@@ -59,12 +61,20 @@ end
 puts "[INFO] Cleaning up loggers after helm upgrade"
 helm_status = $CHILD_STATUS.exitstatus
 Process.kill("KILL", event_pid)
+Process.wait event_pid
 Process.kill("KILL", service_pid)
-puts @pod_pids if ENV["helm_upgrade_logs_debug"] == "true"
+Process.wait service_pid
+
+if ENV["helm_upgrade_logs_debug"] == "true"
+  puts "[INFO] Log processes"
+  puts @pod_pids
+end
 @pod_pids.each do |_pod, pid|
   puts "Terminating #{_pod} #{pid}" if ENV["helm_upgrade_logs_debug"] == "true"
   Process.kill("KILL", pid)
+  Process.wait pid
 end
+puts `ps` if ENV["helm_upgrade_logs_debug"] == "true"
 
 if ENV["helm_upgrade_logs_error_msg"]
   pod_logs_files = Dir.glob("#{UPGRADE_LOG_FOLDER}/*.log")

data/lib/helm_upgrade_logs/version.rb

@@ -2,5 +2,5 @@
 
 module HelmUpgradeLogs
   # @return [String] Version of helm upgrade logs
-  VERSION = "0.3.3"
+  VERSION = "0.3.4"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: helm_upgrade_logs
 version: !ruby/object:Gem::Version
-  version: 0.3.3
+  version: 0.3.4
 platform: ruby
 authors:
 - Samuel Garratt
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2022-06-29 00:00:00.000000000 Z
+date: 2022-06-30 00:00:00.000000000 Z
 dependencies: []
 description: Basic wrapper around helm and kubectl to allow easy debugging of a helm
   release. All arguments after a usual helm upgrade are used as normal