was: [PATCH] qtest/migration: Add a test for the analyze-migration script https://lore.kernel.org/r/20230927214756.14117-1-faro...@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
