fixes npe issue with --stats

seqeralabs · Sep 30, 2024 · bf77e72 · bf77e72
1 parent 0c1ac90
commit bf77e72
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/src/main/java/io/seqera/tower/cli/commands/runs/ViewCmd.java b/src/main/java/io/seqera/tower/cli/commands/runs/ViewCmd.java
@@ -33,6 +33,8 @@
 import io.seqera.tower.model.Workflow;
 import io.seqera.tower.model.WorkflowLoad;
 import io.seqera.tower.model.WorkflowQueryAttribute;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 import picocli.CommandLine;
 
 import java.util.ArrayList;
@@ -56,6 +58,7 @@
 )
 public class ViewCmd extends AbstractRunsCmd {
 
+    private static final Logger log = LoggerFactory.getLogger(ViewCmd.class);
     @CommandLine.Option(names = {"-i", "--id"}, description = "Pipeline run identifier.", required = true)
     public String id;
 
@@ -144,7 +147,8 @@ protected Response exec() throws ApiException {
 
             Map<String, Object> stats = new HashMap<>();
             if (opts.stats) {
-                stats.put("wallTime", TimeUnit.MILLISECONDS.toSeconds(workflow.getDuration()) / 60D);
+                if( workflow.getDuration() != null )
+                    stats.put("wallTime", TimeUnit.MILLISECONDS.toSeconds(workflow.getDuration()) / 60D);
                 if(progress.getWorkflowProgress() != null) {
                     stats.put("cpuTime", TimeUnit.MILLISECONDS.toMinutes(progress.getWorkflowProgress().getCpuTime()) / 60D);
                     stats.put("totalMemory", progress.getWorkflowProgress().getMemoryRss() / 1024 / 1024 / 1024D);