async-container-supervisor 0.9.2 → 0.9.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 9013912b7ef22165d93ebf5f6958d26be67b99f601a28b3ac6e7208fc4e8db54
-  data.tar.gz: 2de566c74dc9fabad2bc5ccb0f0bf329581860fbb8e27f14944cefecfacef9b2
+  metadata.gz: 4765d4a1500ef475808f032d7140aacd3beac8e939c68b34c3884d0cee53bef3
+  data.tar.gz: fc6fd02c0376c43685f32a2556122fa7c10d36388411241927f6bd58bf9a741f
 SHA512:
-  metadata.gz: bde00386e4f3faa439e149b872bb22075725f05235769a1b5084760f059e0e1aed7a669dec9232125aa5053b1afdf09a016c444a16918c4da92210169e9e50b0
-  data.tar.gz: 844be24d31216db2cdfb978aaf888d15be0982e3211c384e7239bd564ef08880d5d870d887084319232ce6760594b37974623c11c5686c433c92ae3de960e67d
+  metadata.gz: 43c99bc5e998157859b494039c62e44bbc2469a82824f482a6d5918c658c35a0e173a3f5c910f5a947b530bbbccdf8a0e1a7eb42a14f54ac268f6d4581b63a43
+  data.tar.gz: 8444b007dbdfc4a8ecfd6834cdf5e19d73939421ababec4be2dedc5db5d5d6551c29b9264259966df2d637b162dd0a63f847a7dbefdc683c429ee27a534d3b76

data/lib/async/container/supervisor/memory_monitor.rb

@@ -30,6 +30,9 @@ module Async
 					@options = options
 					
 					@processes = Hash.new{|hash, key| hash[key] = Set.new.compare_by_identity}
+					
+					# Queue to serialize cluster modifications to prevent race conditions:
+					@guard = Mutex.new
 				end
 				
 				# @attribute [Memory::Leak::Cluster] The cluster of processes being monitored.
@@ -50,7 +53,10 @@ module Async
 						
 						if connections.empty?
 							Console.debug(self, "Registering process.", child: {process_id: process_id})
-							self.add(process_id)
+							# Queue the cluster modification to avoid race conditions:
+							@guard.synchronize do
+								self.add(process_id)
+							end
 						end
 						
 						connections.add(connection)
@@ -66,7 +72,9 @@ module Async
 						
 						if connections.empty?
 							Console.debug(self, "Removing process.", child: {process_id: process_id})
-							@cluster.remove(process_id)
+							@guard.synchronize do
+								@cluster.remove(process_id)
+							end
 						end
 					end
 				end
@@ -119,14 +127,16 @@ module Async
 				def run
 					Async do
 						Loop.run(interval: @interval) do
-							# This block must return true if the process was killed.
-							@cluster.check! do |process_id, monitor|
-								Console.error(self, "Memory leak detected!", child: {process_id: process_id}, monitor: monitor)
-								
-								begin
-									memory_leak_detected(process_id, monitor)
-								rescue => error
-									Console.error(self, "Failed to handle memory leak!", child: {process_id: process_id}, exception: error)
+							@guard.synchronize do
+								# This block must return true if the process was killed.
+								@cluster.check! do |process_id, monitor|
+									Console.error(self, "Memory leak detected!", child: {process_id: process_id}, monitor: monitor)
+									
+									begin
+										memory_leak_detected(process_id, monitor)
+									rescue => error
+										Console.error(self, "Failed to handle memory leak!", child: {process_id: process_id}, exception: error)
+									end
 								end
 							end
 						end

data/lib/async/container/supervisor/version.rb

@@ -9,7 +9,7 @@ module Async
 	module Container
 		# @namespace
 		module Supervisor
-			VERSION = "0.9.2"
+			VERSION = "0.9.3"
 		end
 	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-supervisor/releases/index) for all releases.
 
+### v0.9.3
+
+  - Serialize `register`/`remove` and `check!` operations in `MemoryMonitor`.
+
 ### v0.9.1
 
   - Close `Call` queue if asynchronous call fails during dispatch - further messages will fail with `ClosedQueueError`.

data/releases.md

@@ -1,5 +1,9 @@
 # Releases
 
+## v0.9.3
+
+  - Serialize `register`/`remove` and `check!` operations in `MemoryMonitor`.
+
 ## v0.9.1
 
   - Close `Call` queue if asynchronous call fails during dispatch - further messages will fail with `ClosedQueueError`.

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: async-container-supervisor
 version: !ruby/object:Gem::Version
-  version: 0.9.2
+  version: 0.9.3
 platform: ruby
 authors:
 - Samuel Williams
@@ -154,7 +154,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.6.9
+rubygems_version: 4.0.3
 specification_version: 4
 summary: A supervisor for managing multiple container processes.
 test_files: []