Changeset 38198

Timestamp:

Apr 27, 2015, 2:06:39 PM (11 years ago)

Author:

watersc1

Message:

Correctly parse job_id values from SLURM (which is also clean for MOAB). If no job_id exists, exit to a fail state that prevents infinite requeues.

File:

• trunk/ippScripts/scripts/sc_remote_exec.pl (modified) (1 diff)

trunk/ippScripts/scripts/sc_remote_exec.pl

@@ -183 +183 @@
 }
 if ($#{ $ssh_exec_stdout } != -1) { # Parse the output
-    $job_id = ${ $ssh_exec_stdout }[0];
-    chomp($job_id);
+    my $line = ${ $ssh_exec_stdout }[0];
+    chomp($line);
+    my @line_split = split /\s+/, $line;
+    $job_id = $line_split[-1];
     $job_id =~ s/\s+//g;
 }
 
 unless(defined($job_id)) { # If we don't have a job_id from this command, it didn't run correctly.
-    &my_die("No job_id returned.  Sorry.", $remote_id, $PS_EXIT_PROG_ERROR);
+    &my_die("No job_id returned.  Sorry.", $remote_id, $PS_EXIT_PROG_ERROR, "exec_fail");
 }