build-buddy 1.14.2 → 1.14.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 14eca58f4d3f477090e879ccc9daa6c2e6cd4ed7
-  data.tar.gz: 3f8b38b895cb96d779792147621e9531cad37a40
+  metadata.gz: 96abcbb3d97494bf58764afb274d6ca984bb733e
+  data.tar.gz: bad604376666183553e37a5ae3dc2255c8f59272
 SHA512:
-  metadata.gz: 3969cc949092d261451d018c8e76611990d1c33a1ebc2592c553a4fb57b7742062ca3645539e8a1e1c416993474bd37fc0a181e8b367e380cfe02cd13a6e69d4
-  data.tar.gz: 51abbb4a39671b7f381b4cdf50c66fc731ed6f6effc693931ae9a45c230591552c0bf36ce5741c36509ce86b6ebda497f6bcd654d9ddd766fe0fbe59920c9cb8
+  metadata.gz: b897835e9014316508518e51a48c12a11b836961589559573959db82704b106a7f05658f46111f7afa3d22867e2950dc29650c9278f571aa8d6cf2667a10d5d7
+  data.tar.gz: b5d0e85f7a661cf6fc8ccbcaad1dd422117289915dc9fa0a94a0c94f588338b6fb2d60e0c628afaf09f34c12766cf18082991744f36b9d651a4ced110b95a211

data/lib/build_buddy.rb

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

data/lib/build_buddy/recorder.rb

@@ -22,7 +22,7 @@ module BuildBuddy
       @mongo[:builds].indexes.create_one({:create_time => -1}, :name => "reverse_order")
     end
 
-    def record_build_data(build_data)
+    def record_build_data_and_start_build(build_data)
       builds = @mongo[:builds]
       begin
         # Do this to prevent build _id's from being sequential and so reduce risk
@@ -32,6 +32,8 @@ module BuildBuddy
       rescue Mongo::Error::OperationFailure => e
         retry if e.to_s.start_with?('E11000') # Duplicate key error
       end
+
+      Celluloid::Actor[:builder].async.start_build(build_data)
     end
 
     def update_build_data(build_data)
@@ -39,8 +41,7 @@ module BuildBuddy
         return
       end
 
-      builds = @mongo[:builds]
-      builds.replace_one({ :_id => build_data._id }, build_data.to_h)
+      @mongo[:builds].replace_one({ :_id => build_data._id }, build_data.to_h)
     end
 
     def get_build_data(id)
@@ -69,4 +70,4 @@ module BuildBuddy
       uri
     end
   end
-end
+end

data/lib/build_buddy/scheduler.rb

@@ -78,8 +78,6 @@ module BuildBuddy
       if i != nil
         build_data = @build_queue[i]
         @build_queue.delete_at(i)
-        build_data.stopped_by = slack_user_name
-        Celluloid::Actor[:recorder].async.record_build_data(build_data)
         return :in_queue
       end
 
@@ -96,8 +94,7 @@ module BuildBuddy
         elsif @build_queue.length > 0  # Then, check if there are any builds waiting to go
           build_data = @build_queue.pop()
           @active_build = build_data
-          Celluloid::Actor[:recorder].async.record_build_data(build_data)
-          Celluloid::Actor[:builder].async.start_build(build_data)
+          Celluloid::Actor[:recorder].async.record_build_data_and_start_build(build_data)
         else # Otherwise, stop the timer until we get a build queued.
           @build_timer.cancel
           @build_timer = nil
@@ -122,4 +119,4 @@ module BuildBuddy
       @build_queue.clone
     end
   end
-end
+end

data/lib/build_buddy/slacker.rb

@@ -105,8 +105,9 @@ module BuildBuddy
                    when :not_found
                      "I could not find a queued or active build with that identifier"
                    end
+        info "Build #{bb_id} was stopped by #{slack_user_name}"
       else
-        response = "You must specify the 24 digit hexadecimal build identifier. It can be an active build or a build in the queue."
+        response = "You must specify the build identifier. It can be an active build or a build in the queue."
       end
 
       response

metadata

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