Skip to content

testing patches

[email protected] fio 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest9 patches 8 repliesIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/9.gitignore: ignore zbd test output files.gitignore+1 -00 replies
2/9t/run-fio-tests: a few small improvementst/run-fio-tests.py+31 -120 replies
3/9t/run-fio-tests: detect requirements and skip tests accordinglyt/run-fio-tests.py+126 -30 replies
4/9t/run-fio-tests: improve Windows supportt/run-fio-tests.py+43 -130 replies
5/9t/run-fio-tests: identify test id for debug messagest/run-fio-tests.py+11 -110 replies
6/9t/steadystate_tests: use null ioengine for testst/steadystate_tests.py+3 -180 replies
7/9.travis.yml: run t/run-fio.tests.py as part of build.travis.yml+15 -110 replies
8/9.appveyor.yml: run run-fio-tests.py.appveyor.yml+4 -20 replies
9/9t/run-fio-tests: relax acceptance criterion for t0011t/run-fio-tests.py+2 -10 replies

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.

Read the v1 thread

Cover letter [PATCH 0/9] testing patches

From: Vincent Fu <[email protected]>

Jens, please consider this series of patches related to testing.

The patches improve t/run-fio-tests.py in various ways, most prominently adding support for Windows and macOS.

Also included are travis and appveyor patches that add run-fio-tests.py as a step. Currently both the travis and appveyor build processes complete in less than four minutes. Adding run-fio-tests.py increases this to about 20 minutes for travis and 14 minutes for appveyor.

Also worth noting is that sometimes the VMs these builds run on appear to become overloaded and produce false failures. For instance, builds on the same hash fail here

https://ci.appveyor.com/project/vincentkfu/fio/builds/29342164

but succeed here

https://ci.appveyor.com/project/vincentkfu/fio/builds/29344271

The failure occured because of a timeout running a null ioengine job that reads 10G of data.

It's certainly nice to have these tests automatically run with every commit, but the downside is that the build process is significantly lengthened and some noise is introduced.

Vincent Fu (9):
.gitignore: ignore zbd test output files t/run-fio-tests: a few small improvements t/run-fio-tests: detect requirements and skip tests accordingly t/run-fio-tests: improve Windows support t/run-fio-tests: identify test id for debug messages t/steadystate_tests: use null ioengine for tests .travis.yml: run t/run-fio.tests.py as part of build .appveyor.yml: run run-fio-tests.py
t/run-fio-tests: relax acceptance criterion for t0011

 .appveyor.yml          |   6 +-
 .gitignore             |   1 +
 .travis.yml            |  26 ++---
 t/run-fio-tests.py     | 219 ++++++++++++++++++++++++++++++++++++-----
 t/steadystate_tests.py |  21 +---
 5 files changed, 219 insertions(+), 54 deletions(-)
Signature · 1 line
2.17.1