Merge pull request #3016 from zendesk/grosser/snap

snapshot time for log printing so it is consistent for all pods
zendesk · Oct 23, 2018 · a775423 · a775423
2 parents 971eb81 + 6871937
commit a775423
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/plugins/kubernetes/app/models/kubernetes/deploy_executor.rb b/plugins/kubernetes/app/models/kubernetes/deploy_executor.rb
@@ -144,7 +144,8 @@ def fetch_pods
 
     def show_logs_on_deploy_if_requested(statuses)
       pods = statuses.map(&:pod).compact.select { |p| p.annotations[:'samson/show_logs_on_deploy'] == 'true' }
-      pods.each { |pod| print_pod_details(pod, Time.now, events: false) }
+      log_end = Time.now # here to be consistent for all pods
+      pods.each { |pod| print_pod_details(pod, log_end, events: false) }
     rescue StandardError
       info = ErrorNotifier.notify($!, sync: true)
       @output.puts "Error showing logs: #{info}"