build-buddy 1.14.0 → 1.14.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 7db95cc8816c610a427233d3082ed8209c9a65b6
-  data.tar.gz: 235ad6277e52ad8be9747933133476562af77633
+  metadata.gz: 6ba4c0fb810c99f9f83160fed087cfe0265d4894
+  data.tar.gz: d4beedca033f53d2867182aeeecad3a67c9f409c
 SHA512:
-  metadata.gz: af52e4b43610c3ff26c5d76c99c3b069f7ba393cb374a020e8954b75ea890447a950451f52bd96da1820b3afb0b85ca2eb17d707739ba1cec51d45f84ae63a91
-  data.tar.gz: a82009f93b686be80f338175408dc5d03bc8374189f60985fbd44112e4adce023205733f6b5a94e0e7d03a04d3921688dd0040c0f1081590fa582ffbf04b62a5
+  metadata.gz: 691ce8fe310f1aae0719b7cf4a56effb99b14078948f4fed733b7436c5250a8293f16e58737796d739c8e4f47df10732b0b38dadccd3d057757e3f65c14498a9
+  data.tar.gz: a214d15ceea592f4bf8d632b63c593532e24b3d2b40abb3b127ce982e448e45213f6013d3312cb159eb1be216637997c625ebed6d56d97e18008fcd9abaac2de

data/lib/build_buddy.rb

@@ -9,5 +9,5 @@ require 'build_buddy/recorder'
 require 'build_buddy/build_data'
 
 module BuildBuddy
-  VERSION = "1.14.0"
+  VERSION = "1.14.1"
 end

data/lib/build_buddy/build_data.rb

@@ -54,8 +54,6 @@ module BuildBuddy
     def to_h
       hash = {}
       instance_variables.each {|var| hash[var.to_s.delete("@")] = instance_variable_get(var) }
-      # Remove the bb_id field as it's only meaningful while the build-buddy is running
-      hash.delete(:bb_id)
       hash
     end
 
@@ -82,10 +80,5 @@ module BuildBuddy
     def pull_request_uri
       "https://github.com/#{@repo_full_name}/pull/#{@pull_request}"
     end
-
-    def log_file_name
-      return nil if @start_time.nil?
-      File.join(Config.build_output_dir, "#{@start_time.strftime('%Y%m%d-%H%M%S')}.log")
-    end
   end
 end

data/lib/build_buddy/builder.rb

@@ -159,10 +159,6 @@ source ${BB_BUILD_SCRIPT}
       @watcher.async.watch_pid
     end
 
-    def self.get_log_file_name(build_data)
-      File.join(Config.build_output_dir, build_data._id.to_s, "log.txt")
-    end
-
     def process_done(status)
       @build_data.end_time = Time.now.utc
       @build_data.termination_type = (status.signaled? ? :killed : :exited)
@@ -231,7 +227,9 @@ source ${BB_BUILD_SCRIPT}
       @watcher = nil
 
       # Delete older log directories
-      log_dir_names = Dir.entries(Config.build_output_dir).select { |f| !f.match(/\d{8}-\d{6}\.log/).nil? }.sort()
+      log_dir_names = Dir.entries(Config.build_output_dir)
+        .select {|fn| fn.match(/[0-9a-z]{24}$/)}
+        .sort_by! {|fn| File.mtime(File.join(Config.build_output_dir, fn))}
       while log_dir_names.count > Config.num_saved_build_outputs
         dir_name = log_dir_names.shift
         FileUtils.rm_rf(File.join(Config.build_output_dir, dir_name))

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: build-buddy
 version: !ruby/object:Gem::Version
-  version: 1.14.0
+  version: 1.14.1
 platform: ruby
 authors:
 - John Lyon-smith