search

NERSC / shifter

Shifter - Linux Containers for HPC
Other
348 stars 65 forks source link

BugFix: Incorrect error handling #253

Closed agaur-lbl closed 5 years ago

agaur-lbl commented 5 years ago

If we fail to load job data, we should propagate that error code and exit cleanly. In the bug we saw- despite getting incorrect job data, we were not handling the error correctly and allowing shifter to continue setupRoot operation on shared nodes.

This patch, exits the prolog.

Signed-off-by: Aditi Gaur agaur@lbl.gov

coveralls commented 5 years ago

Coverage Status

Coverage remained the same at 64.65% when pulling cd2f5e54520fb39660278deaaf767ae0cff7f0cc on agaur-lbl:prolog_fix into eaf113ce2526843b54865029c7de5acf78bac46c on NERSC:master.