Merge pull request #298 from nix-community/failed-db

only initialze failedbuilddb once

Author: Mic92

buildbot_nix/__init__.py

@@ -1470,6 +1470,9 @@ def activate(self) -> Generator[Any, object, Any]:
         yield super().activate()
 
 
+DB = None
+
+
 class NixConfigurator(ConfiguratorBase):
     """Janitor is a configurator which create a Janitor Builder with all needed Janitor steps"""
 
@@ -1530,9 +1533,11 @@ def configure(self, config: dict[str, Any]) -> None:
                 )
             )
 
-        db = FailedBuildDB(Path("failed_builds.dbm"))
-        # Hacky but we have no other hooks just now to run code on shutdown.
-        atexit.register(lambda: db.close())
+        global DB  # noqa: PLW0603
+        if DB is None:
+            DB = FailedBuildDB(Path("failed_builds.dbm"))
+            # Hacky but we have no other hooks just now to run code on shutdown.
+            atexit.register(lambda: DB.close() if DB is not None else None)
 
         for project in projects:
             config_for_project(
@@ -1545,7 +1550,7 @@ def configure(self, config: dict[str, Any]) -> None:
                 eval_lock,
                 [x.to_buildstep() for x in self.config.post_build_steps],
                 self.config.job_report_limit,
-                failed_builds_db=db,
+                failed_builds_db=DB,
                 outputs_path=self.config.outputs_path,
             )