mbox series

[v4,0/5] iotests/297: Cover tests/

Message ID 20210902094017.32902-1-hreitz@redhat.com
Headers show
Series iotests/297: Cover tests/ | expand

Message

Hanna Czenczek Sept. 2, 2021, 9:40 a.m. UTC 
v1: https://lists.nongnu.org/archive/html/qemu-block/2021-03/msg01471.html
v2: https://lists.nongnu.org/archive/html/qemu-block/2021-05/msg00492.html
v3: https://lists.nongnu.org/archive/html/qemu-block/2021-05/msg00569.html


Hi,

Sorry for the long delay, here is v4 to make our lint checking iotest
297 cover the tests/ subdirectory.


v4:
- Fixed the explanatory comment in patch 3 as suggested by Vladimir
- Added patch 4


git-backport-diff against v3:

Key:
[----] : patches are identical
[####] : number of functional differences between upstream/downstream patch
[down] : patch is downstream-only
The flags [FC] indicate (F)unctional and (C)ontextual differences, respectively

001/5:[----] [-C] 'iotests/297: Drop 169 and 199 from the skip list'
002/5:[----] [--] 'migrate-bitmaps-postcopy-test: Fix pylint warnings'
003/5:[0006] [FC] 'migrate-bitmaps-test: Fix pylint warnings'
004/5:[down] 'mirror-top-perms: Fix AbnormalShutdown path'
005/5:[----] [--] 'iotests/297: Cover tests/'


Hanna Reitz (5):
  iotests/297: Drop 169 and 199 from the skip list
  migrate-bitmaps-postcopy-test: Fix pylint warnings
  migrate-bitmaps-test: Fix pylint warnings
  mirror-top-perms: Fix AbnormalShutdown path
  iotests/297: Cover tests/

 tests/qemu-iotests/297                        |  7 +--
 .../tests/migrate-bitmaps-postcopy-test       | 13 +++---
 tests/qemu-iotests/tests/migrate-bitmaps-test | 43 +++++++++++--------
 tests/qemu-iotests/tests/mirror-top-perms     |  2 +-
 4 files changed, 37 insertions(+), 28 deletions(-)

Comments

Hanna Czenczek Sept. 7, 2021, 9:50 a.m. UTC | #1 
On 02.09.21 11:40, Hanna Reitz wrote:
> v1: https://lists.nongnu.org/archive/html/qemu-block/2021-03/msg01471.html
> v2: https://lists.nongnu.org/archive/html/qemu-block/2021-05/msg00492.html
> v3: https://lists.nongnu.org/archive/html/qemu-block/2021-05/msg00569.html
>
>
> Hi,
>
> Sorry for the long delay, here is v4 to make our lint checking iotest
> 297 cover the tests/ subdirectory.

Thanks for the review, applied to my block branch:

https://github.com/XanClic/qemu/commits/block

Hanna