[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v2 0/6] scripts/migration: Fix analyze-migration.py and add a tes
From: |
Fabiano Rosas |
Subject: |
[PATCH v2 0/6] scripts/migration: Fix analyze-migration.py and add a test |
Date: |
Mon, 9 Oct 2023 15:43:20 -0300 |
was: [PATCH] qtest/migration: Add a test for the analyze-migration script
https://lore.kernel.org/r/20230927214756.14117-1-farosas@suse.de
The analyze-migration.py script should be kept in sync with the code
that generates the migration stream. The addition/removal of sections
and flags from the stream can cause the script to break. Issues when
parsing the stream mostly manifest in the form of cryptic python
errors such as:
Looking at git log, it seems that this is a fairly useful script for
people debugging issues with the migration stream. Let's add a test
for it to catch bugs early avoid keeping a broken script in the tree.
People suggested putting this in avocado or a dedicated shell
script. I hope looking at the patches it becomes clear that it's best
to have this along with migration-test. Mostly to avoid duplicating
capabilities code and arch-specific flags.
CI run: https://gitlab.com/farosas/qemu/-/pipelines/1030789937
Fabiano Rosas (5):
migration: Fix analyze-migration.py 'configuration' parsing
migration: Add capability parsing to analyze-migration.py
migration: Fix analyze-migration.py when ignore-shared is used
migration: Fix analyze-migration read operation signedness
tests/qtest/migration: Add a test for the analyze-migration script
Nikolay Borisov (1):
migration: Add the configuration vmstate to the json writer
migration/migration.c | 1 +
migration/savevm.c | 20 ++++++++---
scripts/analyze-migration.py | 67 ++++++++++++++++++++++++++++++++----
tests/qtest/meson.build | 2 ++
tests/qtest/migration-test.c | 60 ++++++++++++++++++++++++++++++++
5 files changed, 139 insertions(+), 11 deletions(-)
--
2.35.3
- [PATCH v2 0/6] scripts/migration: Fix analyze-migration.py and add a test,
Fabiano Rosas <=