diff mbox series

[yocto-autobuilder-helper] config.json: Switch poky-altcfg to use ipk as the package backend

Message ID 20240304151405.2093292-1-richard.purdie@linuxfoundation.org
State New
Headers show
Series [yocto-autobuilder-helper] config.json: Switch poky-altcfg to use ipk as the package backend | expand

Commit Message

Richard Purdie March 4, 2024, 3:14 p.m. UTC
Since we plan to have better "binary distro" testing on the autobuilder
and we plan to primarily use ipk for this, having ipk based images available
as part of our builds is desireable.

Since we want systemd based images with ipk, it makes sense to leave standard
poky using rpm images but switch poky-altcfg to use ipk.

Whilst late in the release cycle, doing this now for scarthgap gives us a good
platform to build off in the long term too.

Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
---
 config.json | 5 +++++
 1 file changed, 5 insertions(+)

Comments

Tim Orling March 4, 2024, 3:25 p.m. UTC | #1
On Mon, Mar 4, 2024 at 7:14 AM Richard Purdie <
richard.purdie@linuxfoundation.org> wrote:

> Since we plan to have better "binary distro" testing on the autobuilder
> and we plan to primarily use ipk for this, having ipk based images
> available
> as part of our builds is desireable.
>

Definitely agree.

>
> Since we want systemd based images with ipk, it makes sense to leave
> standard
> poky using rpm images but switch poky-altcfg to use ipk.
>

Good idea!

>
> Whilst late in the release cycle, doing this now for scarthgap gives us a
> good
> platform to build off in the long term too.
>

Seems like a good idea to me!

>
> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
> ---
>  config.json | 5 +++++
>  1 file changed, 5 insertions(+)
>
> diff --git a/config.json b/config.json
> index fd8335f..4d0f258 100644
> --- a/config.json
> +++ b/config.json
> @@ -127,6 +127,7 @@
>              "BUILDINFO" : true,
>              "BUILDHISTORY" : true,
>              "extravars" : [
> +                "PACKAGE_CLASSES = 'package_ipk package_rpm package_deb'",
>                  "IMAGE_INSTALL:append = ' ssh-pregen-hostkeys'"
>              ],
>              "extratools" : "${EXTRATOOLS_URL}",
> @@ -140,6 +141,7 @@
>              "BUILDINFO" : true,
>              "BUILDHISTORY" : true,
>              "extravars" : [
> +                "PACKAGE_CLASSES = 'package_ipk package_rpm package_deb'",
>                  "IMAGE_INSTALL:append = ' ssh-pregen-hostkeys'"
>              ],
>              "extratools" : "${EXTRATOOLS_URL}",
> @@ -198,6 +200,9 @@
>          "altcfg-hw" : {
>              "DISTRO" : "poky-altcfg",
>              "BUILDINFO" : true,
> +            "extravars" : [
> +                "PACKAGE_CLASSES = 'package_ipk package_rpm package_deb'"
> +            ],
>              "step1" : {
>                  "BBTARGETS" : "core-image-full-cmdline core-image-sato
> core-image-sato-sdk"
>              }
> --
> 2.40.1
>
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#62665):
> https://lists.yoctoproject.org/g/yocto/message/62665
> Mute This Topic: https://lists.yoctoproject.org/mt/104723845/924729
> Group Owner: yocto+owner@lists.yoctoproject.org
> Unsubscribe: https://lists.yoctoproject.org/g/yocto/unsub [
> ticotimo@gmail.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>
>
Michael Opdenacker March 4, 2024, 3:34 p.m. UTC | #2
Hi Richard

Many thanks for the patch!

On 3/4/24 at 16:14, Richard Purdie wrote:
> Since we plan to have better "binary distro" testing on the autobuilder
> and we plan to primarily use ipk for this, having ipk based images available
> as part of our builds is desireable.
>
> Since we want systemd based images with ipk, it makes sense to leave standard
> poky using rpm images but switch poky-altcfg to use ipk.
>
> Whilst late in the release cycle, doing this now for scarthgap gives us a good
> platform to build off in the long term too.
>
> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
> ---
>   config.json | 5 +++++
>   1 file changed, 5 insertions(+)
>
> diff --git a/config.json b/config.json
> index fd8335f..4d0f258 100644
> --- a/config.json
> +++ b/config.json
> @@ -127,6 +127,7 @@
>               "BUILDINFO" : true,
>               "BUILDHISTORY" : true,
>               "extravars" : [
> +                "PACKAGE_CLASSES = 'package_ipk package_rpm package_deb'",
>                   "IMAGE_INSTALL:append = ' ssh-pregen-hostkeys'"
>               ],
>               "extratools" : "${EXTRATOOLS_URL}",
> @@ -140,6 +141,7 @@
>               "BUILDINFO" : true,
>               "BUILDHISTORY" : true,
>               "extravars" : [
> +                "PACKAGE_CLASSES = 'package_ipk package_rpm package_deb'",
>                   "IMAGE_INSTALL:append = ' ssh-pregen-hostkeys'"
>               ],
>               "extratools" : "${EXTRATOOLS_URL}",
> @@ -198,6 +200,9 @@
>           "altcfg-hw" : {
>               "DISTRO" : "poky-altcfg",
>               "BUILDINFO" : true,
> +            "extravars" : [
> +                "PACKAGE_CLASSES = 'package_ipk package_rpm package_deb'"
> +            ],
>               "step1" : {
>                   "BBTARGETS" : "core-image-full-cmdline core-image-sato core-image-sato-sdk"
>               }
>
Actually, to use these images for testing package upgrades through 
feeds, we also need enough free space in the images.
Could we also add something like IMAGE_ROOTFS_EXTRA_SPACE = "262144"? 
Thanks again Cheers Michael. --

Michael Opdenacker, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
Richard Purdie March 4, 2024, 3:48 p.m. UTC | #3
On Mon, 2024-03-04 at 16:34 +0100, Michael Opdenacker wrote:
> Hi Richard
> 
> Many thanks for the patch!
> 
> On 3/4/24 at 16:14, Richard Purdie wrote:
> > Since we plan to have better "binary distro" testing on the autobuilder
> > and we plan to primarily use ipk for this, having ipk based images available
> > as part of our builds is desireable.
> > 
> > Since we want systemd based images with ipk, it makes sense to leave standard
> > poky using rpm images but switch poky-altcfg to use ipk.
> > 
> > Whilst late in the release cycle, doing this now for scarthgap gives us a good
> > platform to build off in the long term too.
> > 
> > Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
> > ---
> >   config.json | 5 +++++
> >   1 file changed, 5 insertions(+)
> > 
> > diff --git a/config.json b/config.json
> > index fd8335f..4d0f258 100644
> > --- a/config.json
> > +++ b/config.json
> > @@ -127,6 +127,7 @@
> >               "BUILDINFO" : true,
> >               "BUILDHISTORY" : true,
> >               "extravars" : [
> > +                "PACKAGE_CLASSES = 'package_ipk package_rpm package_deb'",
> >                   "IMAGE_INSTALL:append = ' ssh-pregen-hostkeys'"
> >               ],
> >               "extratools" : "${EXTRATOOLS_URL}",
> > @@ -140,6 +141,7 @@
> >               "BUILDINFO" : true,
> >               "BUILDHISTORY" : true,
> >               "extravars" : [
> > +                "PACKAGE_CLASSES = 'package_ipk package_rpm package_deb'",
> >                   "IMAGE_INSTALL:append = ' ssh-pregen-hostkeys'"
> >               ],
> >               "extratools" : "${EXTRATOOLS_URL}",
> > @@ -198,6 +200,9 @@
> >           "altcfg-hw" : {
> >               "DISTRO" : "poky-altcfg",
> >               "BUILDINFO" : true,
> > +            "extravars" : [
> > +                "PACKAGE_CLASSES = 'package_ipk package_rpm package_deb'"
> > +            ],
> >               "step1" : {
> >                   "BBTARGETS" : "core-image-full-cmdline core-image-sato core-image-sato-sdk"
> >               }
> > 
> Actually, to use these images for testing package upgrades through 
> feeds, we also need enough free space in the images.
> Could we also add something like IMAGE_ROOTFS_EXTRA_SPACE = "262144"?

Setting that "globally" for those templates will potentially break other
images the usual way to do it would be:

IMAGE_ROOTFS_EXTRA_SPACE:append = "+ 102400"

but I still worry about side effects, e.g. for minimal or initramfs images.
Which images do we plan to test? core-image-full-cmdline and what else?

We probably need a set of definitions like:

IMAGE_ROOTFS_EXTRA_SPACE:append:pn-core-image-full-cmdline = " + 262144"

but something like:

IMAGE_ROOTFS_EXTRA_SPACE:append = "${@bb.utils.contains("IMAGE_FEATURES, "package-management", " + 262144", "", d)}"

might work.

Cheers,

Richard
Michael Opdenacker March 4, 2024, 4:02 p.m. UTC | #4
Hi Richard,

On 3/4/24 at 16:48, Richard Purdie wrote:
> On Mon, 2024-03-04 at 16:34 +0100, Michael Opdenacker wrote:
>> Hi Richard
>>
>> Many thanks for the patch!
>>
>> On 3/4/24 at 16:14, Richard Purdie wrote:
>>> Since we plan to have better "binary distro" testing on the autobuilder
>>> and we plan to primarily use ipk for this, having ipk based images available
>>> as part of our builds is desireable.
>>>
>>> Since we want systemd based images with ipk, it makes sense to leave standard
>>> poky using rpm images but switch poky-altcfg to use ipk.
>>>
>>> Whilst late in the release cycle, doing this now for scarthgap gives us a good
>>> platform to build off in the long term too.
>>>
>>> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
>>> ---
>>>    config.json | 5 +++++
>>>    1 file changed, 5 insertions(+)
>>>
>>> diff --git a/config.json b/config.json
>>> index fd8335f..4d0f258 100644
>>> --- a/config.json
>>> +++ b/config.json
>>> @@ -127,6 +127,7 @@
>>>                "BUILDINFO" : true,
>>>                "BUILDHISTORY" : true,
>>>                "extravars" : [
>>> +                "PACKAGE_CLASSES = 'package_ipk package_rpm package_deb'",
>>>                    "IMAGE_INSTALL:append = ' ssh-pregen-hostkeys'"
>>>                ],
>>>                "extratools" : "${EXTRATOOLS_URL}",
>>> @@ -140,6 +141,7 @@
>>>                "BUILDINFO" : true,
>>>                "BUILDHISTORY" : true,
>>>                "extravars" : [
>>> +                "PACKAGE_CLASSES = 'package_ipk package_rpm package_deb'",
>>>                    "IMAGE_INSTALL:append = ' ssh-pregen-hostkeys'"
>>>                ],
>>>                "extratools" : "${EXTRATOOLS_URL}",
>>> @@ -198,6 +200,9 @@
>>>            "altcfg-hw" : {
>>>                "DISTRO" : "poky-altcfg",
>>>                "BUILDINFO" : true,
>>> +            "extravars" : [
>>> +                "PACKAGE_CLASSES = 'package_ipk package_rpm package_deb'"
>>> +            ],
>>>                "step1" : {
>>>                    "BBTARGETS" : "core-image-full-cmdline core-image-sato core-image-sato-sdk"
>>>                }
>>>
>> Actually, to use these images for testing package upgrades through
>> feeds, we also need enough free space in the images.
>> Could we also add something like IMAGE_ROOTFS_EXTRA_SPACE = "262144"?
> Setting that "globally" for those templates will potentially break other
> images the usual way to do it would be:
>
> IMAGE_ROOTFS_EXTRA_SPACE:append = "+ 102400"
>
> but I still worry about side effects, e.g. for minimal or initramfs images.
> Which images do we plan to test? core-image-full-cmdline and what else?
>
> We probably need a set of definitions like:
>
> IMAGE_ROOTFS_EXTRA_SPACE:append:pn-core-image-full-cmdline = " + 262144"
>
> but something like:
>
> IMAGE_ROOTFS_EXTRA_SPACE:append = "${@bb.utils.contains("IMAGE_FEATURES, "package-management", " + 262144", "", d)}"
>
> might work.


Ouch, good catch.
I'd vote for the second option. When we have package management, it 
makes sense to keep enough space for downloading and applying updates.
Cheers
Michael.
diff mbox series

Patch

diff --git a/config.json b/config.json
index fd8335f..4d0f258 100644
--- a/config.json
+++ b/config.json
@@ -127,6 +127,7 @@ 
             "BUILDINFO" : true,
             "BUILDHISTORY" : true,
             "extravars" : [
+                "PACKAGE_CLASSES = 'package_ipk package_rpm package_deb'",
                 "IMAGE_INSTALL:append = ' ssh-pregen-hostkeys'"
             ],
             "extratools" : "${EXTRATOOLS_URL}",
@@ -140,6 +141,7 @@ 
             "BUILDINFO" : true,
             "BUILDHISTORY" : true,
             "extravars" : [
+                "PACKAGE_CLASSES = 'package_ipk package_rpm package_deb'",
                 "IMAGE_INSTALL:append = ' ssh-pregen-hostkeys'"
             ],
             "extratools" : "${EXTRATOOLS_URL}",
@@ -198,6 +200,9 @@ 
         "altcfg-hw" : {
             "DISTRO" : "poky-altcfg",
             "BUILDINFO" : true,
+            "extravars" : [
+                "PACKAGE_CLASSES = 'package_ipk package_rpm package_deb'"
+            ],
             "step1" : {
                 "BBTARGETS" : "core-image-full-cmdline core-image-sato core-image-sato-sdk"
             }