All of lore.kernel.org
 help / color / mirror / Atom feed
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



             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.