Bug #7313

[Crunch] [UX] Job log should warn user more clearly when no output is recorded

Added by Bryan Cosca almost 4 years ago. Updated almost 4 years ago.

Status:
Resolved
Priority:
Normal
Assigned To:
-
Category:
Crunch
Target version:
Start date:
09/11/2015
Due date:
% Done:

100%

Estimated time:
Story points:
-

Description

Comparing job: qr1hi-8i9sb-maixhdz21fh9z0n to job: qr1hi-8i9sb-h0z9l6pwcwyhy9h
Comparing scripts: https://workbench.qr1hi.arvadosapi.com/repositories/qr1hi-s0uqq-nft5jq6b5kpvo9t/blob/28e05a2d1d398bf3cb39198f29f513b8dbbfde73/crunch_scripts/varscan.py to https://workbench.qr1hi.arvadosapi.com/repositories/qr1hi-s0uqq-nft5jq6b5kpvo9t/blob/f1648526332bd9c09c6915a5cd16edf065ad5dea/crunch_scripts/varscan.py (to their respective jobs)

the only difference in job in the first job is that I forgot to write data to keep. The log for not writing data to keep was

child 4218 on compute4.1 exit 0 success=false

There needs to be some indication saying that you need to write to keep in order to succeed. This log error is way too vague.

Associated revisions

Revision 714c555b (diff)
Added by Brett Smith almost 4 years ago

7313: crunch-job reports an error when a task doesn't record state.

Closes #7313.

Revision aa041461 (diff)
Added by Brett Smith over 3 years ago

7313: Fix sprintf formatter in crunch-job warning message.

Refs #7313.

History

#1 Updated by Brett Smith almost 4 years ago

  • Subject changed from [SDK] [UX] Not uploading any data to keep shows vague log to [Crunch] [UX] Job log should warn user more clearly when no output is recorded
  • Category set to Crunch
  • Target version set to Arvados Future Sprints

#2 Updated by Tom Clegg almost 4 years ago

Just before or after logging this line

child 4218 on compute4.1 exit 0 success=false

crunch-job should also say

ERROR: Task process exited 0, but never updated its task record to indicate success and record its output.

#3 Updated by Brett Smith almost 4 years ago

  • Status changed from New to Resolved
  • % Done changed from 0 to 100

Applied in changeset arvados|commit:714c555bda26a6a27fad7caef382d1d6705ad215.

#4 Updated by Brett Smith almost 4 years ago

  • Target version changed from Arvados Future Sprints to 2015-12-02 sprint

Also available in: Atom PDF