[PULL v2 0/7] Block jobs patches for 2024-04-29

[PULL v2 0/7] Block jobs patches for 2024-04-29

[Vladimir Sementsov-Ogievskiy]

The following changes since commit ad10b4badc1dd5b28305f9b9f1168cf0aa3ae946:

  Merge tag 'pull-error-2024-05-27' of https://repo.or.cz/qemu/armbru into staging (2024-05-27 06:40:42 -0700)

are available in the Git repository at:

  https://gitlab.com/vsementsov/qemu.git tags/pull-block-jobs-2024-04-29-v2

for you to fetch changes up to a149401048481247bcbaf6035a7a1308974fb464:

  iotests/pylintrc: allow up to 10 similar lines (2024-05-28 15:52:15 +0300)

----------------------------------------------------------------
Block jobs patches for 2024-04-29

v2: add "iotests/pylintrc: allow up to 10 similar lines" to fix
    check-python-minreqs

- backup: discard-source parameter
- blockcommit: Reopen base image as RO after abort

----------------------------------------------------------------
Alexander Ivanov (1):
      blockcommit: Reopen base image as RO after abort

Vladimir Sementsov-Ogievskiy (6):
      block/copy-before-write: fix permission
      block/copy-before-write: support unligned snapshot-discard
      block/copy-before-write: create block_copy bitmap in filter node
      qapi: blockdev-backup: add discard-source parameter
      iotests: add backup-discard-source
      iotests/pylintrc: allow up to 10 similar lines

 block/backup.c                                     |   5 +-
 block/block-copy.c                                 |  12 ++-
 block/copy-before-write.c                          |  39 +++++++--
 block/copy-before-write.h                          |   1 +
 block/mirror.c                                     |  11 ++-
 block/replication.c                                |   4 +-
 blockdev.c                                         |   2 +-
 include/block/block-common.h                       |   2 +
 include/block/block-copy.h                         |   2 +
 include/block/block_int-global-state.h             |   2 +-
 qapi/block-core.json                               |   4 +
 tests/qemu-iotests/257.out                         | 112 +++++++++++++-------------
 tests/qemu-iotests/pylintrc                        |   2 +-
 tests/qemu-iotests/tests/backup-discard-source     | 152 ++++++++++++++++++++++++++++++++++++
 tests/qemu-iotests/tests/backup-discard-source.out |   5 ++
 15 files changed, 282 insertions(+), 73 deletions(-)
 create mode 100755 tests/qemu-iotests/tests/backup-discard-source
 create mode 100644 tests/qemu-iotests/tests/backup-discard-source.out


Alexander Ivanov (1):
  blockcommit: Reopen base image as RO after abort

Vladimir Sementsov-Ogievskiy (6):
  block/copy-before-write: fix permission
  block/copy-before-write: support unligned snapshot-discard
  block/copy-before-write: create block_copy bitmap in filter node
  qapi: blockdev-backup: add discard-source parameter
  iotests: add backup-discard-source
  iotests/pylintrc: allow up to 10 similar lines

 block/backup.c                                |   5 +-
 block/block-copy.c                            |  12 +-
 block/copy-before-write.c                     |  39 ++++-
 block/copy-before-write.h                     |   1 +
 block/mirror.c                                |  11 +-
 block/replication.c                           |   4 +-
 blockdev.c                                    |   2 +-
 include/block/block-common.h                  |   2 +
 include/block/block-copy.h                    |   2 +
 include/block/block_int-global-state.h        |   2 +-
 qapi/block-core.json                          |   4 +
 tests/qemu-iotests/257.out                    | 112 ++++++-------
 tests/qemu-iotests/pylintrc                   |   2 +-
 .../qemu-iotests/tests/backup-discard-source  | 152 ++++++++++++++++++
 .../tests/backup-discard-source.out           |   5 +
 15 files changed, 282 insertions(+), 73 deletions(-)
 create mode 100755 tests/qemu-iotests/tests/backup-discard-source
 create mode 100644 tests/qemu-iotests/tests/backup-discard-source.out

-- 
2.34.1

[PULL v2 7/7] iotests/pylintrc: allow up to 10 similar lines

[Vladimir Sementsov-Ogievskiy]

We want to have similar QMP objects in different tests. Reworking these
objects to make common parts by calling some helper functions doesn't
seem good. It's a lot more comfortable to see the whole QAPI request in
one place.

So, let's increase the limit, to unblock further commit
"iotests: add backup-discard-source"

Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Signed-off-by: Vladimir Sementsov-Ogievskiy <vsementsov@yandex-team.ru>
---
 tests/qemu-iotests/pylintrc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tests/qemu-iotests/pylintrc b/tests/qemu-iotests/pylintrc
index de2e0c2781..05b75ee59b 100644
--- a/tests/qemu-iotests/pylintrc
+++ b/tests/qemu-iotests/pylintrc
@@ -55,4 +55,4 @@ max-line-length=79
 
 [SIMILARITIES]
 
-min-similarity-lines=6
+min-similarity-lines=10
-- 
2.34.1

Re: [PULL v2 0/7] Block jobs patches for 2024-04-29

[Richard Henderson]

On 5/28/24 06:57, Vladimir Sementsov-Ogievskiy wrote:
> The following changes since commit ad10b4badc1dd5b28305f9b9f1168cf0aa3ae946:
> 
>    Merge tag 'pull-error-2024-05-27' ofhttps://repo.or.cz/qemu/armbru  into staging (2024-05-27 06:40:42 -0700)
> 
> are available in the Git repository at:
> 
>    https://gitlab.com/vsementsov/qemu.git  tags/pull-block-jobs-2024-04-29-v2
> 
> for you to fetch changes up to a149401048481247bcbaf6035a7a1308974fb464:
> 
>    iotests/pylintrc: allow up to 10 similar lines (2024-05-28 15:52:15 +0300)
> 
> ----------------------------------------------------------------
> Block jobs patches for 2024-04-29
> 
> v2: add "iotests/pylintrc: allow up to 10 similar lines" to fix
>      check-python-minreqs
> 
> - backup: discard-source parameter
> - blockcommit: Reopen base image as RO after abort

Applied, thanks.  Please update https://wiki.qemu.org/ChangeLog/9.1 as appropriate.


r~