mbox series

[RFC,0/2] LVM: Drop legacy scripts and runtest files

Message ID 20200528110554.10179-1-pvorel@suse.cz
Headers show
Series LVM: Drop legacy scripts and runtest files | expand

Message

Petr Vorel May 28, 2020, 11:05 a.m. UTC 
Hi,

I guess there is no doubt, that new implementation is better thus no
need for legacy version. I hope that dropping legacy fs is ok (if not we
should support them in new API).

Kind regards,
Petr

Petr Vorel (2):
  LVM: Don't require root for generate_lvm_runfile.sh
  LVM: Drop legacy scripts and runtest files

 doc/ltp-run-files.txt                      |   4 +-
 runtest/lvm.part1                          | 217 ---------------------
 runtest/lvm.part2                          |  72 -------
 testcases/misc/lvm/generate_lvm_runfile.sh |   1 -
 testscripts/ltpfslvm.sh                    | 200 -------------------
 testscripts/ltpfsnolvm.sh                  | 150 --------------
 6 files changed, 1 insertion(+), 643 deletions(-)
 delete mode 100644 runtest/lvm.part1
 delete mode 100644 runtest/lvm.part2
 delete mode 100755 testscripts/ltpfslvm.sh
 delete mode 100755 testscripts/ltpfsnolvm.sh

Comments

Petr Vorel May 28, 2020, 11:40 a.m. UTC | #1 
Hi,

growfiles are used also in other runtest files: fs, fs_readonly, scsi_debug.part1.
I wonder how to reduce duplicity for fs and fs_readonly.

runtest/scsi_debug.part1 should be IMHO deleted. And
testscripts/ltp-scsi_debug.sh should be either deleted, or, if the idea is
valid (using scsi_debug kernel module for testing), it should be rewritten.

Other candidate for deleting are ltpfs tests (testcases/kernel/fs/scsi/ltpfs/).
They're not maintained, ltpfsio.sh and ltpfstest is not even installed.

Kind regards,
Petr