From: Michael Tremer Date: Thu, 6 Feb 2025 21:24:39 +0000 (+0000) Subject: builders: Hash builders by hostname for their stats connections X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=098b4c6f88c373f2e347c59d1cd6cbc7cda789c2;p=pbs.git builders: Hash builders by hostname for their stats connections Signed-off-by: Michael Tremer --- diff --git a/src/buildservice/builders.py b/src/buildservice/builders.py index 12da7d09..54aedb22 100644 --- a/src/buildservice/builders.py +++ b/src/buildservice/builders.py @@ -409,28 +409,24 @@ class BuildersStats(base.Object): def join(self, builder, connection): try: - self.builders[builder].append(connection) + self.builders[builder.name].append(connection) except KeyError: - self.builders[builder] = [connection] + self.builders[builder.name] = [connection] def leave(self, connection): # Find and remove the connection - for builder in self.builders.copy(): + for name in self.builders.copy(): try: - self.builders[builder].remove(connection) + self.builders[name].remove(connection) except ValueError: continue - # Remove the builder if it has no connections left - if not self.builders[builder]: - del self.builders[builder] - async def submit_stats(self, builder, stats): """ Called when a builder sends new stats """ try: - connections = self.builders[builder] + connections = self.builders[builder.name] except KeyError: return