[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v1 5/7] tests/qtest/migration: Print migration incoming error
From: |
Fabiano Rosas |
Subject: |
Re: [PATCH v1 5/7] tests/qtest/migration: Print migration incoming errors |
Date: |
Mon, 27 Nov 2023 17:32:45 -0300 |
Peter Xu <peterx@redhat.com> writes:
> On Mon, Nov 27, 2023 at 12:52:38PM -0300, Fabiano Rosas wrote:
>> >> @@ -118,6 +118,12 @@ void migrate_incoming_qmp(QTestState *to, const char
>> >> *uri, const char *fmt, ...)
>> >>
>> >> rsp = qtest_qmp(to, "{ 'execute': 'migrate-incoming', 'arguments':
>> >> %p}",
>> >> args);
>> >> +
>> >> + if (!qdict_haskey(rsp, "return")) {
>> >> + g_autoptr(GString) s = qobject_to_json_pretty(QOBJECT(rsp),
>> >> true);
>> >> + g_test_message("%s", s->str);
>> >> + }
>> >
>> > This traps the "migrate-incoming" command only (which, afaiu, only setup
>> > the listening), would this capture the incoming error?
>>
>> This is about the migrate-incoming only. We could replace "incoming
>> migration" with "qmp_migrate_incoming" in the commit message to clarify.
>
> Ah.. Did you ever see this failure in any of your runs in these tests? I
> think it means you hit the assertion right below this part, but I'm just
> curious how, as the URIs in the test cases are pretty constant.
Yes, I don't remember what exactly, but we changed the code that parses
the URIs in this release and I'm also working on
file_start_incoming_migration.
- [PATCH v1 2/7] migration/multifd: Remove QEMUFile from where it is not needed, (continued)
[PATCH v1 6/7] tests/qtest/migration: Add a wrapper to print test names, Fabiano Rosas, 2023/11/24
[PATCH v1 7/7] tests/qtest/migration: Use the new migration_test_add, Fabiano Rosas, 2023/11/24