ci-fixes, on dumpLogs map before allSettled

paritytech · Oct 1, 2024 · 55b482c · 55b482c
1 parent b0ec05d
commit 55b482c
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/javascript/packages/orchestrator/src/network.ts b/javascript/packages/orchestrator/src/network.ts
@@ -168,10 +168,11 @@ export class Network {
       [],
     );
 
-    const dumpsPromises = this.relay.concat(paraNodes).map((node) => {
-      this.client.dumpLogs(this.tmpDir, node.name);
-    });
-    await Promise.allSettled(dumpsPromises);
+    const dumpsNodes = this.relay.concat(paraNodes);
+    await Promise.allSettled(dumpsNodes.map((node) => {
+        this.client.dumpLogs(this.tmpDir, node.name);
+      })
+    );
 
     if (showLogPath)
       new CreateLogTable({ colWidths: [20, 100] }).pushToPrint([

diff --git a/javascript/packages/orchestrator/src/providers/k8s/kubeClient.ts b/javascript/packages/orchestrator/src/providers/k8s/kubeClient.ts
@@ -750,6 +750,7 @@ export class KubeClient extends Client {
     // We should read it from host filesystem to ensure we are reading all the logs.
 
     // First get the logs files to check if we need to read from disk or not
+    debugLogs("getting logFiles for:", podName);
     const logFiles = await this.gzippedLogFiles(podName);
     debugLogs("logFiles", logFiles);
     let logs = "";