From: Het Gala <het.gala@nutanix.com>
To: qemu-devel@nongnu.org
Cc: thuth@redhat.com, lvivier@redhat.com, pbonzini@redhat.com,
peterx@redhat.com, farosas@suse.de, prerna.saxena@nutanix.com,
Het Gala <het.gala@nutanix.com>
Subject: [PATCH v2 0/3] qtest/migration: Fixes around multifd_tcp_channels_none migration qtest
Date: Tue, 9 Apr 2024 11:00:08 +0000 [thread overview]
Message-ID: <20240409110011.174426-1-het.gala@nutanix.com> (raw)
With the introduction of new patchset to have 'channels' as the start
argument of migrate QAPIs instead of 'uri' (tests/qtest/migration: Add
tests for introducing 'channels' argument in migrate QAPIs), a few minor
issues got went unnoticed, which were caught while trying to introduce
similar qtests in migration-test.c
Fix multifd_tcp_channels_none qtest to actually utilize 'channels' arg
in migrate QAPIs, fix double freeing of addr Qdict and typos in that
patchset.
This patchset is built on top of (tests/qtest/migration: Add tests for
introducing 'channels' argument in migrate QAPIs)
Can find the build pipeline at : https://gitlab.com/galahet/Qemu/-/pipelines/1245462266
v1 --> v2:
---------
1. Split the second patch into different patches - One to deal with double
freeing of Qdict and other to add connect_channels inside
multifd_tcp_channels_none to actually use 'channels' arg.
2. use 'git commit --fixup' to improve commit message as well as
to inform on which commit is the fix meant to be.
Het Gala (3):
fixup! tests/qtest/migration: Add negative tests to validate migration
QAPIs
fixup! tests/qtest/migration: Add migrate_set_ports into migrate_qmp
to update migration port value
fixup! tests/qtest/migration: Add multifd_tcp_plain test using list of
channels instead of uri
tests/qtest/migration-helpers.c | 4 ++--
tests/qtest/migration-test.c | 4 ++--
2 files changed, 4 insertions(+), 4 deletions(-)
--
2.22.3
next reply other threads:[~2024-04-09 11:01 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-04-09 11:00 Het Gala [this message]
2024-04-09 11:00 ` [PATCH v2 1/3] fixup! tests/qtest/migration: Add negative tests to validate migration QAPIs Het Gala
2024-04-09 11:00 ` [PATCH v2 2/3] fixup! tests/qtest/migration: Add migrate_set_ports into migrate_qmp to update migration port value Het Gala
2024-04-09 11:00 ` [PATCH v2 3/3] fixup! tests/qtest/migration: Add multifd_tcp_plain test using list of channels instead of uri Het Gala
2024-04-09 11:56 ` [PATCH v2 0/3] qtest/migration: Fixes around multifd_tcp_channels_none migration qtest Peter Xu
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20240409110011.174426-1-het.gala@nutanix.com \
--to=het.gala@nutanix.com \
--cc=farosas@suse.de \
--cc=lvivier@redhat.com \
--cc=pbonzini@redhat.com \
--cc=peterx@redhat.com \
--cc=prerna.saxena@nutanix.com \
--cc=qemu-devel@nongnu.org \
--cc=thuth@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.