public inbox for passt-dev@passt.top
 help / color / mirror / code / Atom feed
From: Stefano Brivio <sbrivio@redhat.com>
To: Paul Holzinger <pholzing@redhat.com>
Cc: passt-dev@passt.top, David Gibson <david@gibson.dropbear.id.au>
Subject: Re: [PATCH v4] treewide: Introduce 'local mode' for disconnected setups
Date: Wed, 27 Nov 2024 13:49:12 +0100	[thread overview]
Message-ID: <20241127134912.6df3f693@elisabeth> (raw)
In-Reply-To: <293d884b-1a82-41f4-aa21-b9cbe28b3fd4@redhat.com>

On Wed, 27 Nov 2024 13:37:25 +0100
Paul Holzinger <pholzing@redhat.com> wrote:

> On 27/11/2024 05:27, Stefano Brivio wrote:
> > There are setups where no host interface is available or configured
> > at all, intentionally or not, temporarily or not, but users expect
> > (Podman) containers to run in any case as they did with slirp4netns,
> > and we're now getting reports that we broke such setups at a rather
> > alarming rate.
> >
> > To this end, if we don't find any usable host interface, instead of
> > exiting:
> >
> > - for IPv4, use 169.254.2.1 as guest/container address and 169.254.2.2
> >    as default gateway
> >
> > - for IPv6, don't assign any address (forcibly disable DHCPv6), and
> >    use the *first* link-local address we observe to represent the
> >    guest/container. Advertise fe80::1 as default gateway
> >
> > - use 'tap0' as default interface name for pasta
> >
> > Change ifi4 and ifi6 in struct ctx to int and accept a special -1
> > value meaning that no host interface was selected, but the IP family
> > is enabled. The fact that the kernel uses unsigned int values for
> > those is not an issue as 1. one can't create so many interfaces
> > anyway and 2. we otherwise handle those values transparently.
> >
> > Fix a botched conditional in conf_print() to actually skip printing
> > DHCPv6 information if DHCPv6 is disabled (and skip printing NDP
> > information if NDP is disabled).
> >
> > Link: https://github.com/containers/podman/issues/24614
> > Signed-off-by: Stefano Brivio <sbrivio@redhat.com>  
> 
> Just for completeness sake, I know it is already applied. so it doesn't 
> matter.
> 
> Tested-by: Paul Holzinger <pholzing@redhat.com>

No, no, it matters, thanks a lot. The upcoming release is 100% more
tested now. :)

-- 
Stefano


      reply	other threads:[~2024-11-27 12:49 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-11-27  4:27 [PATCH v4] treewide: Introduce 'local mode' for disconnected setups Stefano Brivio
2024-11-27 12:37 ` Paul Holzinger
2024-11-27 12:49   ` Stefano Brivio [this message]

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=20241127134912.6df3f693@elisabeth \
    --to=sbrivio@redhat.com \
    --cc=david@gibson.dropbear.id.au \
    --cc=passt-dev@passt.top \
    --cc=pholzing@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.
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).