Re: [oe-core][PATCH 2/2] packagegroup-core-boot: recommend ifupdown
> -Original Message- > From: Alexandre Belloni > Sent: Friday, April 12, 2024 3:22 AM > To: Slater, Joseph > Cc: Ross Burton ; Richard Purdie > ; OE-core c...@lists.openembedded.org>; MacLeod, Randy > > Subject: Re: [oe-core][PATCH 2/2] packagegroup-core-boot: recommend > ifupdown > > On 11/04/2024 14:58:40+, Joe Slater via lists.openembedded.org wrote: > > > > > > > -Original Message- > > > From: Ross Burton > > > Sent: Thursday, April 11, 2024 7:21 AM > > > To: Richard Purdie ; Slater, > > > Joseph > > > Cc: OE-core ; MacLeod, > > > Randy > > > Subject: Re: [oe-core][PATCH 2/2] packagegroup-core-boot: recommend > > > ifupdown > > > > > > On 11 Apr 2024, at 10:33, Richard Purdie via lists.openembedded.org > > > wrote: > > > >> --- > > > >> a/meta/recipes-core/init-ifupdown/init-ifupdown-1.0/qemuall/inter > > > >> faces > > > >> +++ /dev/null > > > >> @@ -1,5 +0,0 @@ > > > >> -# /etc/network/interfaces -- configuration file for ifup(8), > > > >> ifdown(8) > > > >> - > > > >> -# The loopback interface > > > >> -auto lo > > > >> -iface lo inet loopback > > > > > > > > Is this patch meant to touch ifupdown as well as the packagegroup? > > > > > > I suspect not, given that runqemu manually brings up network devices > > > and the last thing we want is DHCP running again. > > > > > > Joe, please remove this chunk. > > > > This was accidental since I removed the qemuall/interfaces in order to use > > the > main one with qemu. I'm sending new patches. Maybe, though, we don't really > need the qemuall stuff. > > This broke all the tests of this build: > > https://autobuilder.yoctoproject.org/typhoon/#/builders/83/builds/6786 The V2 patch doesn't change the interfaces file and should not cause any issues. Joe > > > > > Joe > > > > > > Ross > > > > > > > > > > -- > Alexandre Belloni, co-owner and COO, Bootlin Embedded Linux and Kernel > engineering https://bootlin.com -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#198176): https://lists.openembedded.org/g/openembedded-core/message/198176 Mute This Topic: https://lists.openembedded.org/mt/105454861/21656 Group Owner: openembedded-core+ow...@lists.openembedded.org Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
Re: [oe-core][PATCH 2/2] packagegroup-core-boot: recommend ifupdown
On 11/04/2024 14:58:40+, Joe Slater via lists.openembedded.org wrote: > > > > -Original Message- > > From: Ross Burton > > Sent: Thursday, April 11, 2024 7:21 AM > > To: Richard Purdie ; Slater, Joseph > > > > Cc: OE-core ; MacLeod, Randy > > > > Subject: Re: [oe-core][PATCH 2/2] packagegroup-core-boot: recommend > > ifupdown > > > > On 11 Apr 2024, at 10:33, Richard Purdie via lists.openembedded.org > > wrote: > > >> --- > > >> a/meta/recipes-core/init-ifupdown/init-ifupdown-1.0/qemuall/interfaces > > >> +++ /dev/null > > >> @@ -1,5 +0,0 @@ > > >> -# /etc/network/interfaces -- configuration file for ifup(8), ifdown(8) > > >> - > > >> -# The loopback interface > > >> -auto lo > > >> -iface lo inet loopback > > > > > > Is this patch meant to touch ifupdown as well as the packagegroup? > > > > I suspect not, given that runqemu manually brings up network devices and the > > last thing we want is DHCP running again. > > > > Joe, please remove this chunk. > > This was accidental since I removed the qemuall/interfaces in order to use > the main one with qemu. I'm sending new patches. Maybe, though, we don't > really need the qemuall stuff. This broke all the tests of this build: https://autobuilder.yoctoproject.org/typhoon/#/builders/83/builds/6786 > > Joe > > > > Ross > > > -- Alexandre Belloni, co-owner and COO, Bootlin Embedded Linux and Kernel engineering https://bootlin.com -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#198161): https://lists.openembedded.org/g/openembedded-core/message/198161 Mute This Topic: https://lists.openembedded.org/mt/105454861/21656 Group Owner: openembedded-core+ow...@lists.openembedded.org Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
Re: [oe-core][PATCH 2/2] packagegroup-core-boot: recommend ifupdown
On 11 Apr 2024, at 15:58, Joe Slater via lists.openembedded.org wrote: > This was accidental since I removed the qemuall/interfaces in order to use > the main one with qemu. I'm sending new patches. Maybe, though, we don't > really need the qemuall stuff. The last time I looked we did need it, because testimage will manually setup the network interface with known IPs via kernel parameters. The last thing we need is for ifupdown to decide to take over the interface and reconfigure it. Ross -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#198139): https://lists.openembedded.org/g/openembedded-core/message/198139 Mute This Topic: https://lists.openembedded.org/mt/105454861/21656 Group Owner: openembedded-core+ow...@lists.openembedded.org Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
Re: [oe-core][PATCH 2/2] packagegroup-core-boot: recommend ifupdown
> -Original Message- > From: Ross Burton > Sent: Thursday, April 11, 2024 7:21 AM > To: Richard Purdie ; Slater, Joseph > > Cc: OE-core ; MacLeod, Randy > > Subject: Re: [oe-core][PATCH 2/2] packagegroup-core-boot: recommend > ifupdown > > On 11 Apr 2024, at 10:33, Richard Purdie via lists.openembedded.org > wrote: > >> --- a/meta/recipes-core/init-ifupdown/init-ifupdown-1.0/qemuall/interfaces > >> +++ /dev/null > >> @@ -1,5 +0,0 @@ > >> -# /etc/network/interfaces -- configuration file for ifup(8), ifdown(8) > >> - > >> -# The loopback interface > >> -auto lo > >> -iface lo inet loopback > > > > Is this patch meant to touch ifupdown as well as the packagegroup? > > I suspect not, given that runqemu manually brings up network devices and the > last thing we want is DHCP running again. > > Joe, please remove this chunk. This was accidental since I removed the qemuall/interfaces in order to use the main one with qemu. I'm sending new patches. Maybe, though, we don't really need the qemuall stuff. Joe > > Ross -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#198136): https://lists.openembedded.org/g/openembedded-core/message/198136 Mute This Topic: https://lists.openembedded.org/mt/105454861/21656 Group Owner: openembedded-core+ow...@lists.openembedded.org Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
Re: [oe-core][PATCH 2/2] packagegroup-core-boot: recommend ifupdown
On 11 Apr 2024, at 10:33, Richard Purdie via lists.openembedded.org wrote: >> --- a/meta/recipes-core/init-ifupdown/init-ifupdown-1.0/qemuall/interfaces >> +++ /dev/null >> @@ -1,5 +0,0 @@ >> -# /etc/network/interfaces -- configuration file for ifup(8), ifdown(8) >> - >> -# The loopback interface >> -auto lo >> -iface lo inet loopback > > Is this patch meant to touch ifupdown as well as the packagegroup? I suspect not, given that runqemu manually brings up network devices and the last thing we want is DHCP running again. Joe, please remove this chunk. Ross -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#198135): https://lists.openembedded.org/g/openembedded-core/message/198135 Mute This Topic: https://lists.openembedded.org/mt/105454861/21656 Group Owner: openembedded-core+ow...@lists.openembedded.org Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
Re: [oe-core][PATCH 2/2] packagegroup-core-boot: recommend ifupdown
On Wed, 2024-04-10 at 18:17 -0700, Joe Slater via lists.openembedded.org wrote:
> From: Joe Slater
>
> If the distro features sysvinit and pni-names are
> enabled, RRECOMMENDS ifupdown because busybox ifupdown
> will not initialize the renamed interfaces.
>
> Signed-off-by: Joe Slater
> ---
> .../init-ifupdown/init-ifupdown-1.0/qemuall/interfaces | 5 -
> meta/recipes-core/packagegroups/packagegroup-core-boot.bb | 4 +++-
> 2 files changed, 3 insertions(+), 6 deletions(-)
> delete mode 100644
> meta/recipes-core/init-ifupdown/init-ifupdown-1.0/qemuall/interfaces
>
> diff --git
> a/meta/recipes-core/init-ifupdown/init-ifupdown-1.0/qemuall/interfaces
> b/meta/recipes-core/init-ifupdown/init-ifupdown-1.0/qemuall/interfaces
> deleted file mode 100644
> index 16967763e5..00
> --- a/meta/recipes-core/init-ifupdown/init-ifupdown-1.0/qemuall/interfaces
> +++ /dev/null
> @@ -1,5 +0,0 @@
> -# /etc/network/interfaces -- configuration file for ifup(8), ifdown(8)
> -
> -# The loopback interface
> -auto lo
> -iface lo inet loopback
Is this patch meant to touch ifupdown as well as the packagegroup?
> diff --git a/meta/recipes-core/packagegroups/packagegroup-core-boot.bb
> b/meta/recipes-core/packagegroups/packagegroup-core-boot.bb
> index d96d2f5fff..fecc3334ea 100644
> --- a/meta/recipes-core/packagegroups/packagegroup-core-boot.bb
> +++ b/meta/recipes-core/packagegroups/packagegroup-core-boot.bb
> @@ -37,4 +37,6 @@ RDEPENDS:${PN} = "\
> RRECOMMENDS:${PN} = "\
> ${VIRTUAL-RUNTIME_base-utils-syslog} \
> ${MACHINE_ESSENTIAL_EXTRA_RRECOMMENDS} \
> - ${@bb.utils.contains("DISTRO_FEATURES", "sysvinit", "init-ifupdown", "",
> d)}"
> + ${@bb.utils.contains("DISTRO_FEATURES", "sysvinit", "init-ifupdown", "",
> d)} \
> + ${@bb.utils.contains("DISTRO_FEATURES", "sysvinit pni-names",
> "ifupdown", "", d)} \
> + "
Cheers,
Richard
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#198123):
https://lists.openembedded.org/g/openembedded-core/message/198123
Mute This Topic: https://lists.openembedded.org/mt/105454861/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-
