From: Denys Dmytriyenko <denis@denix.org>
To: michael.haener@siemens.com
Cc: openembedded-core@lists.openembedded.org
Subject: Re: [OE-core] [scarthgap][PATCH] oeqa/runtime/ping: don't bother trying to ping localhost
Date: Thu, 30 Oct 2025 17:14:37 -0400 [thread overview]
Message-ID: <20251030211437.GI3125@denix.org> (raw)
In-Reply-To: <20251015204131.14992-1-michael.haener@siemens.com>
On Wed, Oct 15, 2025 at 10:41:31PM +0200, Michael Haener via lists.openembedded.org wrote:
> If SLIRP is being used instead of TAP for networking to the guest then
> the target IP will be localhost. There's no point in pinging localhost
> to see if the target is up but whilst you'd think it is harmless, in
> some containers ping doesn't actually have enough rights to work:
>
> ping: socktype: SOCK_RAW
> ping: socket: Operation not permitted
> ping: => missing cap_net_raw+p capability or setuid?
>
> Look at the target address and if it's localhost or 127.0.0.* return
> immediately.
>
> (Backport from OE-Core rev: a06ef43d2a50e16c32bd6edbdc7b32c3528687d5)
>
> Signed-off-by: Michael Haener <michael.haener@siemens.com>
> ---
> meta/lib/oeqa/runtime/cases/ping.py | 7 +++++++
> test | 0
> 2 files changed, 7 insertions(+)
> create mode 100644 test
>
> diff --git a/meta/lib/oeqa/runtime/cases/ping.py b/meta/lib/oeqa/runtime/cases/ping.py
> index f72460e7f3..bc543f6c41 100644
> --- a/meta/lib/oeqa/runtime/cases/ping.py
> +++ b/meta/lib/oeqa/runtime/cases/ping.py
> @@ -18,6 +18,13 @@ class PingTest(OERuntimeTestCase):
> output = ''
> count = 0
> self.assertNotEqual(len(self.target.ip), 0, msg="No target IP address set")
> +
> + # If the target IP is localhost (because user-space networking is being used),
> + # then there's no point in pinging it.
> + if self.target.ip.startswith("127.0.0.") or self.target.ip in ("localhost", "::1"):
> + print("runtime/ping: localhost detected, not pinging")
> + return
> +
> try:
> while count < 5:
> cmd = 'ping -c 1 %s' % self.target.ip
> diff --git a/test b/test
> new file mode 100644
> index 0000000000..e69de29bb2
Should there be an empty "test" file in the root directory of OE-Core?
--
Denys
next prev parent reply other threads:[~2025-10-30 21:14 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-10-15 20:41 [scarthgap][PATCH] oeqa/runtime/ping: don't bother trying to ping localhost Michael Haener
2025-10-30 21:14 ` Denys Dmytriyenko [this message]
2025-10-30 21:46 ` [OE-core] " Haener, Michael
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=20251030211437.GI3125@denix.org \
--to=denis@denix.org \
--cc=michael.haener@siemens.com \
--cc=openembedded-core@lists.openembedded.org \
/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.