public inbox for passt-dev@passt.top
 help / color / mirror / code / Atom feed
From: Stefano Brivio <sbrivio@redhat.com>
To: passt-dev@passt.top
Subject: [PATCH 0/2] test: Fix two issues made apparent by new command dispatch
Date: Sat, 17 Sep 2022 01:55:32 +0200	[thread overview]
Message-ID: <20220916235534.2399997-1-sbrivio@redhat.com> (raw)

[-- Attachment #1: Type: text/plain, Size: 1118 bytes --]

This is based on pending series by David, but they're logically separated
fixes, so I'm sending them as a different series.

Patch 2/2 adds some 'sleep 1' commands that were previously missing, which
actually goes against the intention of speeding up those tests. However,
without those, a number of tests fail quite frequently due to races
between starting server and client, and I could never get a clean run
with the new command dispatch mechanism otherwise.

We can probably find a more elegant (and time-saving) mechanism to
make sure the server is ready (based e.g. on parsing its output), but
at the moment I'm just trying to make the test suite work again for
me in its entirety.

Stefano Brivio (2):
  test/perf: Check for /sbin/sysctl with which(1), not simply sysctl
  test/passt_in_ns: Consistent sleep commands before starting socat
    client

 test/passt_in_ns/tcp | 29 +++++++++++++++++++++++++----
 test/passt_in_ns/udp | 18 ++++++++++++++++--
 test/perf/passt_tcp  |  4 ++--
 test/perf/passt_udp  |  2 +-
 test/perf/pasta_tcp  |  2 +-
 5 files changed, 45 insertions(+), 10 deletions(-)

-- 
2.35.1


             reply	other threads:[~2022-09-16 23:55 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-09-16 23:55 Stefano Brivio [this message]
2022-09-16 23:55 ` [PATCH 1/2] test/perf: Check for /sbin/sysctl with which(1), not simply sysctl Stefano Brivio
2022-09-17  3:27   ` David Gibson
2022-09-17  8:44     ` Stefano Brivio
2022-09-17  9:51       ` David Gibson
2022-09-16 23:55 ` [PATCH 2/2] test/passt_in_ns: Consistent sleep commands before starting socat client Stefano Brivio
2022-09-17  3:32   ` David Gibson
2022-09-17  8:44     ` Stefano Brivio
2022-09-17 10:19       ` David Gibson
2022-09-17 11:28         ` Stefano Brivio
2022-09-19  1:48           ` David Gibson
2022-09-19  6:41             ` Stefano Brivio
2022-09-19  7:00               ` David Gibson
2022-09-19  8:24                 ` Stefano Brivio

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=20220916235534.2399997-1-sbrivio@redhat.com \
    --to=sbrivio@redhat.com \
    --cc=passt-dev@passt.top \
    /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.
Code repositories for project(s) associated with this public inbox

	https://passt.top/passt

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for IMAP folder(s).