Deposit icinga checks should return a single status line and set of performance data
The deposit end to end icinga checks return two sets of status and perfdata values.
e.g.
DEPOSIT OK - Deposit took 21.28s and succeeded.
| 'load_time' = 11.87s
| 'total_time' = 21.28s
| 'upload_time' = 0.46s
| 'validation_time' = 8.95s
DEPOSIT OK - Deposit Metadata update took 1.14s and succeeded.
| 'total_time' = 22.42s
| 'update_time' = 1.14s
This format confuses icinga, which is unable to record the performance data. We should either separate the two checks, or combine their output in a single one.
Migrated from T4010 (view on Phabricator)