Add verbosity to smart-dispatch

SMART-Lab / smartdispatch

An easy to use job launcher for supercomputers with PBS compatible job manager.

Do What The F*ck You Want To Public License

34 stars 18 forks source link

Add verbosity to smart-dispatch #175

Open bouthilx opened 7 years ago

bouthilx commented 7 years ago

It is difficult to debug resuming while important process are taking place in the pbs script automatically built by SmartDispatch.

Adding:

verbose to smart-dispatch/sd-launch-pbs command-line options
debugging prints in epilog (based on verbosity option)

bouthilx commented 7 years ago

@mgermain Small PR I took out of Slurm Backend PR

coveralls commented 7 years ago

Coverage remained the same at 94.532% when pulling 329efce4f71f799f9fdb4b3864ef53af65e94164 on bouthilx:verbose into 9133e15ae91e1e5b0a2b7e90f442c1ebfb623e86 on SMART-Lab:master.

coveralls commented 7 years ago

Coverage remained the same at 94.532% when pulling 502181da6d576160bcae129c04959f9d4860c532 on bouthilx:verbose into 9133e15ae91e1e5b0a2b7e90f442c1ebfb623e86 on SMART-Lab:master.

mgermain commented 7 years ago

After the tiny modifs mentioned this is ready to be merged.

coveralls commented 7 years ago

Coverage remained the same at 94.532% when pulling 65239f5700abcdeece5f76a15ff877ad211a2f72 on bouthilx:verbose into 9133e15ae91e1e5b0a2b7e90f442c1ebfb623e86 on SMART-Lab:master.

bouthilx commented 7 years ago

@mgermain I bit the bullet and reverted the E501 fixes. :stuck_out_tongue_winking_eye: I also changed the debugging logic in the epilog to support passing WARNING/INFO/DEBUG levels to sd-launch-pbs. Previously, it could only pass WARNING (no -v) or DEBUG (-vv).