Path: csiph.com!fu-berlin.de!bofh.it!news.nic.it!robomod From: Mike Waterman Newsgroups: linux.debian.bugs.dist,linux.debian.kernel Subject: Bug#1078607: Feature Request: Add `auto` option for VRF table id management Date: Wed, 14 Aug 2024 15:30:01 +0200 Message-ID: References: X-Original-To: =?UTF-8?Q?Daniel_Gr=C3=B6ber?= , Mike Waterman , 1078607@bugs.debian.org X-Mailbox-Line: From debian-bugs-dist-request@lists.debian.org Wed Aug 14 13:21:10 2024 Old-Return-Path: X-Spam-Flag: NO X-Spam-Score: -1.46 Reply-To: Mike Waterman , 1078607@bugs.debian.org Resent-To: debian-bugs-dist@lists.debian.org Resent-Cc: Debian Kernel Team X-Debian-Pr-Message: followup 1078607 X-Debian-Pr-Package: iproute2 X-Debian-Pr-Keywords: upstream X-Debian-Pr-Source: iproute2 X-Forwarded-Encrypted: i=1; AJvYcCVcfGVb0NiitWIX9zeY065Au/vTVKUDaGRRfZBLhglWC0aehay1zEfJfz0FpzxjZAMLHyDzN4KyK+QaQyWLW9g4fE5STVUSuw== X-Gm-Message-State: AOJu0YzoTWMYnXUdoI9sdv6zNo9hai8rXeqKOZk8YzRBRCrBz2ODWrto LIdcI0/s2lzIZw5uX8hbfvHKg+kHfjD+yTlglXUgnYhakvkPXzqGU6YF28MI25An859/84I1JdW FDLJ/NHS7MtJpgZ5uN1kEgFX4jiYYXmvNbzpxXg== X-Google-SMTP-Source: AGHT+IF9UV8ziTrtsP84UX1aPdMJBfhQDWqyopJaNhUHMQu5EhZrPuCPbxxtvXdTJ4i2tu2L/ZLf7z5eYbRRlo7fvXg= X-Received: by 2002:a05:6820:1688:b0:5d6:1082:4f4d with SMTP id 006d021491bc7-5da7c739e23mr3162131eaf.4.1723641422200; Wed, 14 Aug 2024 06:17:02 -0700 (PDT) MIME-Version: 1.0 Content-Type: multipart/alternative; boundary="000000000000b0de39061fa48a21" X-Debian-Message: from BTS X-Mailing-List: archive/latest/1852423 List-ID: List-URL: Approved: robomod@news.nic.it Lines: 132 Organization: linux.* mail to news gateway Sender: robomod@news.nic.it X-Original-Date: Wed, 14 Aug 2024 09:16:51 -0400 X-Original-Message-ID: X-Original-References: <20240814084401.prfehzuw225ws2mi@House.clients.dxld.at> <20240814084401.prfehzuw225ws2mi@House.clients.dxld.at> Xref: csiph.com linux.debian.bugs.dist:1208358 linux.debian.kernel:83493 --000000000000b0de39061fa48a21 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Thanks, Daniel! I'm setting up to submit to netdev now. I shortened the request as you suggested. On Wed, Aug 14, 2024 at 4:44=E2=80=AFAM Daniel Gr=C3=B6ber wrote: > Hi Mike, > > On Tue, Aug 13, 2024 at 01:41:32PM -0400, Mike Waterman wrote: > > > It'd be great to do this: > > > > ``` > > auto vrfexample > > iface vrfexample > > vrf-table auto > > > > auto vrfexample2 > > iface vrfexample2 > > vrf-table auto > > ``` > > > > Or this: > > > > ``` > > ip link add vrfexample type vrf table auto > > ip link add vrfexample2 type vrf table auto > > ``` > > If the ability to specify 'auto' in iproute2 isn't an absolute requiremen= t > and iproute2 upstream rejects the idea then implementing this in ifupdown= * > would also be possible. > > I maintain ifupdown-ng in Debian (and am involved upstream) which already > has rudimentary [VRF support] I'd be happy to help you work on this, but > the first step really is to ask the iproute2 maintainer, > Stephen Hemminger , to consider this feature. > > [VRF support]: > https://github.com/ifupdown-ng/ifupdown-ng/blob/main/executor-scripts/lin= ux/vrf > > Please CC me and the netdev mailinglist: netdev@vger.kernel.org and be > sure > to include "iproute2" in the mail subject. That list serves many projects > :) > > I'd also recommend trimming your feature request down to the essentials o= f > what affects iproute2, kernel maintainers are busy people. Don't waste > their attention. I can help you draft something if you want. > > --Daniel > --000000000000b0de39061fa48a21 Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable
Thanks, Daniel! I'm sett= ing up to submit to netdev now. I shortened the request as you=C2=A0suggest= ed.


On Wed, Aug 14, 2024 at 4:44=E2=80=AFAM Dani= el Gr=C3=B6ber <dxld@darkboxed.org= > wrote:
= Hi Mike,

On Tue, Aug 13, 2024 at 01:41:32PM -0400, Mike Waterman wrote:

> It'd be great to do this:
>
> ```
> auto vrfexample
> iface vrfexample
>=C2=A0 =C2=A0 =C2=A0vrf-table auto
>
> auto vrfexample2
> iface vrfexample2
>=C2=A0 =C2=A0 =C2=A0vrf-table auto
> ```
>
> Or this:
>
> ```
> ip link add vrfexample type vrf table auto
> ip link add vrfexample2 type vrf table auto
> ```

If the ability to specify 'auto' in iproute2 isn't an absolute = requirement
and iproute2 upstream rejects the idea then implementing this in ifupdown*<= br> would also be possible.

I maintain ifupdown-ng in Debian (and am involved upstream) which already has rudimentary [VRF support] I'd be happy to help you work on this, bu= t
the first step really is to ask the iproute2 maintainer,
Stephen Hemminger <stephen@networkplumber.org>, to consider this feature.
[VRF support]: https:= //github.com/ifupdown-ng/ifupdown-ng/blob/main/executor-scripts/linux/vrf

Please CC me and the netdev mailinglist:
netdev@vger.kernel.org and be sure
to include "iproute2" in the mail subject. That list serves many = projects :)

I'd also recommend trimming your feature request down to the essentials= of
what affects iproute2, kernel maintainers are busy people. Don't waste<= br> their attention. I can help you draft something if you want.

--Daniel
--000000000000b0de39061fa48a21--