async-container 0.27.1 → 0.27.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c475605b174dc9677a89310e85b7ca28269b084469b51c6f7f5226ca39d26340
-  data.tar.gz: ccbdcaa3da1c785abdadefc3ece4ce6552ea6c418a0218229e04ab53c51b2637
+  metadata.gz: 7d43fc927cb1564c1ed256976873b66245386fe54d11467935184c1f38da9d96
+  data.tar.gz: 3738e9b3009bd87388fe23c34085d99a4ba384fe8ef274465ec732a8398f6b7b
 SHA512:
-  metadata.gz: 1f17a77ff0978894a46b5b18b4ac7e2fc2bf451d13c2c4c4a94c122cc205df90b248007ad2b1d6d4655b89257bb94d745a57aa23ed833702f811afe62dc64261
-  data.tar.gz: 7415b0fb06ccaeea5645f0cc43e098f8ea1f684fc3cc51b1e5dd22dd1b3be68eba864af8f32fe93d960af3e6205a490453c48189730371a19cf0f873cb738155
+  metadata.gz: 14700cf9cb8952affd49569383348715fa357660b809f033a1a0d345a03f3b16145ec868b07022df1bece5224008a49ce78441d86113ef18c94530247fadf104
+  data.tar.gz: 7665c5a66280b04bd835e9708e81fff08ab5116ab07886fae1aa3c3b155bef6011167d10bcdb63b08c3a877d5df795dd3901740a0e46ac0a433ff7cf024feac1

data/lib/async/container/controller.rb

@@ -91,11 +91,11 @@ module Async
 			# Start the container unless it's already running.
 			def start
 				unless @container
-					Console.info(self) {"Controller starting..."}
+					Console.info(self, "Controller starting...")
 					self.restart
 				end
 				
-				Console.info(self) {"Controller started..."}
+				Console.info(self, "Controller started...")
 			end
 			
 			# Stop the container if it's running.
@@ -111,9 +111,9 @@ module Async
 				if @container
 					@notify&.restarting!
 					
-					Console.debug(self) {"Restarting container..."}
+					Console.info(self, "Restarting container...")
 				else
-					Console.debug(self) {"Starting container..."}
+					Console.info(self, "Starting container...")
 				end
 				
 				container = self.create_container
@@ -127,13 +127,14 @@ module Async
 				end
 				
 				# Wait for all child processes to enter the ready state.
-				Console.debug(self, "Waiting for startup...")
+				Console.info(self, "Waiting for startup...")
 				container.wait_until_ready
-				Console.debug(self, "Finished startup.")
+				Console.info(self, "Finished startup.")
 				
 				if container.failed?
 					@notify&.error!("Container failed to start!")
 					
+					Console.info(self, "Stopping failed container...")
 					container.stop(false)
 					
 					raise SetupError, container
@@ -145,7 +146,7 @@ module Async
 				container = nil
 				
 				if old_container
-					Console.debug(self, "Stopping old container...")
+					Console.info(self, "Stopping old container...")
 					old_container&.stop(@graceful_stop)
 				end
 				
@@ -168,11 +169,9 @@ module Async
 				end
 				
 				# Wait for all child processes to enter the ready state.
-				Console.debug(self, "Waiting for startup...")
-				
+				Console.info(self, "Waiting for startup...")
 				@container.wait_until_ready
-				
-				Console.debug(self, "Finished startup.")
+				Console.info(self, "Finished startup.")
 				
 				if @container.failed?
 					@notify.error!("Container failed to reload!")

data/lib/async/container/generic.rb

@@ -139,12 +139,18 @@ module Async
 			# Stop the children instances.
 			# @parameter timeout [Boolean | Numeric] Whether to stop gracefully, or a specific timeout.
 			def stop(timeout = true)
+				Console.info(self, "Stopping container...", timeout: timeout, caller: caller_locations)
 				@running = false
 				@group.stop(timeout)
 				
 				if @group.running?
-					Console.warn(self) {"Group is still running after stopping it!"}
+					Console.warn(self, "Group is still running after stopping it!")
+				else
+					Console.info(self, "Group has stopped.")
 				end
+			rescue => error
+				Console.error(self, "Error while stopping container!", exception: error)
+				raise
 			ensure
 				@running = true
 			end
@@ -165,7 +171,7 @@ module Async
 				name ||= UNNAMED
 				
 				if mark?(key)
-					Console.debug(self) {"Reusing existing child for #{key}: #{name}"}
+					Console.debug(self, "Reusing existing child.", child: {key: key, name: name})
 					return false
 				end
 				
@@ -199,10 +205,10 @@ module Async
 						end
 						
 						if status.success?
-							Console.debug(self) {"#{child} exited with #{status}"}
+							Console.info(self, "Child exited successfully.", status: status, running: @running)
 						else
 							@statistics.failure!
-							Console.error(self, status: status)
+							Console.error(self, "Child exited with error!", status: status, running: @running)
 						end
 						
 						if restart
@@ -211,6 +217,11 @@ module Async
 							break
 						end
 					end
+				rescue => error
+					Console.error(self, "Failure during child process management!", exception: error, running: @running)
+					raise
+				ensure
+					Console.info(self, "Child process management loop exited.", running: @running)
 				end.resume
 				
 				return true

data/lib/async/container/version.rb

@@ -5,6 +5,6 @@
 
 module Async
 	module Container
-		VERSION = "0.27.1"
+		VERSION = "0.27.2"
 	end
 end

data/readme.md

@@ -26,6 +26,10 @@ Please see the [project documentation](https://socketry.github.io/async-containe
 
 Please see the [project releases](https://socketry.github.io/async-container/releases/index) for all releases.
 
+### v0.27.2
+
+  - More logging, especially around failure cases.
+
 ### v0.27.1
 
   - Log caller and timeout when waiting on a child instance to exit, if it blocks.

data/releases.md

@@ -1,5 +1,9 @@
 # Releases
 
+## v0.27.2
+
+  - More logging, especially around failure cases.
+
 ## v0.27.1
 
   - Log caller and timeout when waiting on a child instance to exit, if it blocks.

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: async-container
 version: !ruby/object:Gem::Version
-  version: 0.27.1
+  version: 0.27.2
 platform: ruby
 authors:
 - Samuel Williams
@@ -105,7 +105,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.6.9
+rubygems_version: 3.7.2
 specification_version: 4
 summary: Abstract container-based parallelism using threads and processes where appropriate.
 test_files: []