Whamcloud - gitweb
LU-8950 tests: do not use make_custom_file_for_progress()
Do not use make_custom_file_for_progress() in sanity-hsm in tests:
12c, 26, 27b, 28, 31b, 31c, 33, 34, 35, 36, 54, 55, 56, 60, 62, 71,
104, 200, 202, 221, 223b, 225, 251, 252, 407
Before this patch make_custom_file_for_progress() was used to create
big files (5MB to 40 MB). There were several use cases for that:
- in combination with the --bandwidth option of lhsmtool_posix, it
allows synchronizing HSM operations and other things;
- to log enough "progress events", HSM operations need to last long
enough (the --bandwidth option comes in handy too).
It needed to be removed because:
- archiving and restoring big files at a 1MB/s rate takes too much
time (admittedly, that was the point);
- there are other way to have HSM operations occur concurently. For
example: suspending copytools to delay request processing.
- make_custom_file_for_progress() does not correctly reflect what
the function does and should be used for. Wherever big file are
still needed, the patch uses create_file() instead.
Removing the need to archive or restore big files at a limited rate
represents quite a speed-up on sanity-hsm.
Test-Parameters: trivial testlist=sanity-hsm,sanity-hsm
Signed-off-by: Quentin Bouget <quentin.bouget@cea.fr>
Change-Id: Ib9eb7599f9e16d8790630c69c9d1c7be3df416a1
Reviewed-on: https://review.whamcloud.com/24394
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Jean-Baptiste Riaux <riaux.jb@intel.com>
Reviewed-by: Andreas Dilger <adilger@whamcloud.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>