Return if not nimbus (#2039)

* Return if not nimbus * Await ensuring Nimbus connection
dappnode · Sep 25, 2024 · 780df8f · 780df8f
1 parent 6c45ad8
commit 780df8f
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/packages/dappmanager/src/calls/packageInstall.ts b/packages/dappmanager/src/calls/packageInstall.ts
@@ -29,7 +29,7 @@ export async function packageInstall({
     options
   });
 
-  ensureNimbusConnection(reqName);
+  await ensureNimbusConnection(reqName);
 }
 
 /**
@@ -41,9 +41,10 @@ export async function packageInstall({
  *
  * TODO: Remove this once all Nimbus packages are multiservice
  */
-function ensureNimbusConnection(dnpName: string): void {
+async function ensureNimbusConnection(dnpName: string): Promise<void> {
   if (!dnpName.includes("nimbus")) {
     logs.debug("Not a Nimbus package, skipping network reconnection");
+    return;
   }
 
   logs.info("Ensuring Nimbus services are connected to the staker network");
@@ -65,5 +66,5 @@ function ensureNimbusConnection(dnpName: string): void {
   }
 
   // Not awaited
-  consensus.persistSelectedConsensusIfInstalled(network);
+  await consensus.persistSelectedConsensusIfInstalled(network);
 }