[PATCH] container-image-busybox: add a reference container image

[PATCH] container-image-busybox: add a reference container image

[Chen Qi]

Add a small busybox based container image.

Signed-off-by: Chen Qi <Qi.Chen@windriver.com>
---
 meta/recipes-core/images/container-image-busybox.bb | 8 ++++++++
 1 file changed, 8 insertions(+)
 create mode 100644 meta/recipes-core/images/container-image-busybox.bb

diff --git a/meta/recipes-core/images/container-image-busybox.bb b/meta/recipes-core/images/container-image-busybox.bb
new file mode 100644
index 0000000000..a4d63b0eea
--- /dev/null
+++ b/meta/recipes-core/images/container-image-busybox.bb
@@ -0,0 +1,8 @@
+SUMMARY = "A small busybox based container image."
+LICENSE = "MIT"
+
+IMAGE_INSTALL = "busybox"
+IMAGE_LINGUAS = ""
+IMAGE_FSTYPES = "container"
+
+inherit core-image
-- 
2.17.1

Re: [PATCH] container-image-busybox: add a reference container image

[Bruce Ashfield]

On Tue, Jul 23, 2019 at 10:39 PM Chen Qi <Qi.Chen@windriver.com> wrote:
>
> Add a small busybox based container image.

This really isn't a great container reference image. It isn't
manipulating many of
the things that you should be in a container image (linux-dummy, etc).

Have you seen the reference images that have either been presented at the
various ELC conferences and/or the ones that are in meta-virtualization ?
Or the ones from Togan labs, meta-overc, etc ?

IMHO, It would be better to collaborate on this in a non-core layer
for this release and
then consolidate things with the OCI image types, and when we sort out
if/what/when
makes sense from meta-virt into oe-core.

This is a topic of many opinions and different implementations and deserves some
cycles spent on documenting the choices, what we expect the base image to
do, etc.

Cheers,

Bruce

>
> Signed-off-by: Chen Qi <Qi.Chen@windriver.com>
> ---
>  meta/recipes-core/images/container-image-busybox.bb | 8 ++++++++
>  1 file changed, 8 insertions(+)
>  create mode 100644 meta/recipes-core/images/container-image-busybox.bb
>
> diff --git a/meta/recipes-core/images/container-image-busybox.bb b/meta/recipes-core/images/container-image-busybox.bb
> new file mode 100644
> index 0000000000..a4d63b0eea
> --- /dev/null
> +++ b/meta/recipes-core/images/container-image-busybox.bb
> @@ -0,0 +1,8 @@
> +SUMMARY = "A small busybox based container image."
> +LICENSE = "MIT"
> +
> +IMAGE_INSTALL = "busybox"
> +IMAGE_LINGUAS = ""
> +IMAGE_FSTYPES = "container"
> +
> +inherit core-image
> --
> 2.17.1
>
> --
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core



-- 
- Thou shalt not follow the NULL pointer, for chaos and madness await
thee at its end
- "Use the force Harry" - Gandalf, Star Trek II

Re: [PATCH] container-image-busybox: add a reference container image

[Khem Raj]

On Tue, Jul 23, 2019 at 8:03 PM Bruce Ashfield <bruce.ashfield@gmail.com> wrote:
>
> On Tue, Jul 23, 2019 at 10:39 PM Chen Qi <Qi.Chen@windriver.com> wrote:
> >
> > Add a small busybox based container image.
>
> This really isn't a great container reference image. It isn't
> manipulating many of
> the things that you should be in a container image (linux-dummy, etc).
>
> Have you seen the reference images that have either been presented at the
> various ELC conferences and/or the ones that are in meta-virtualization ?
> Or the ones from Togan labs, meta-overc, etc ?
>
> IMHO, It would be better to collaborate on this in a non-core layer
> for this release and
> then consolidate things with the OCI image types, and when we sort out
> if/what/when
> makes sense from meta-virt into oe-core.

I think this is very much suitable for OE-Core, and I would like us to
have container reference image
in core, however, I agree there is prework that should be includes
from prior work in other layers and
we should consolidate all that into a reference usable image in oe-core.

>
> This is a topic of many opinions and different implementations and deserves some
> cycles spent on documenting the choices, what we expect the base image to
> do, etc.
>
> Cheers,
>
> Bruce
>
> >
> > Signed-off-by: Chen Qi <Qi.Chen@windriver.com>
> > ---
> >  meta/recipes-core/images/container-image-busybox.bb | 8 ++++++++
> >  1 file changed, 8 insertions(+)
> >  create mode 100644 meta/recipes-core/images/container-image-busybox.bb
> >
> > diff --git a/meta/recipes-core/images/container-image-busybox.bb b/meta/recipes-core/images/container-image-busybox.bb
> > new file mode 100644
> > index 0000000000..a4d63b0eea
> > --- /dev/null
> > +++ b/meta/recipes-core/images/container-image-busybox.bb
> > @@ -0,0 +1,8 @@
> > +SUMMARY = "A small busybox based container image."
> > +LICENSE = "MIT"
> > +
> > +IMAGE_INSTALL = "busybox"
> > +IMAGE_LINGUAS = ""
> > +IMAGE_FSTYPES = "container"
> > +
> > +inherit core-image
> > --
> > 2.17.1
> >
> > --
> > _______________________________________________
> > Openembedded-core mailing list
> > Openembedded-core@lists.openembedded.org
> > http://lists.openembedded.org/mailman/listinfo/openembedded-core
>
>
>
> --
> - Thou shalt not follow the NULL pointer, for chaos and madness await
> thee at its end
> - "Use the force Harry" - Gandalf, Star Trek II
> --
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [PATCH] container-image-busybox: add a reference container image

[Burton, Ross]

Are you aware of container-test-image.bb?  Curious what the point of this is.

Ross

On Wed, 24 Jul 2019 at 03:39, Chen Qi <Qi.Chen@windriver.com> wrote:
>
> Add a small busybox based container image.
>
> Signed-off-by: Chen Qi <Qi.Chen@windriver.com>
> ---
>  meta/recipes-core/images/container-image-busybox.bb | 8 ++++++++
>  1 file changed, 8 insertions(+)
>  create mode 100644 meta/recipes-core/images/container-image-busybox.bb
>
> diff --git a/meta/recipes-core/images/container-image-busybox.bb b/meta/recipes-core/images/container-image-busybox.bb
> new file mode 100644
> index 0000000000..a4d63b0eea
> --- /dev/null
> +++ b/meta/recipes-core/images/container-image-busybox.bb
> @@ -0,0 +1,8 @@
> +SUMMARY = "A small busybox based container image."
> +LICENSE = "MIT"
> +
> +IMAGE_INSTALL = "busybox"
> +IMAGE_LINGUAS = ""
> +IMAGE_FSTYPES = "container"
> +
> +inherit core-image
> --
> 2.17.1
>
> --
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [PATCH] container-image-busybox: add a reference container image

[ChenQi]

On 07/24/2019 05:38 PM, Burton, Ross wrote:
> Are you aware of container-test-image.bb?  Curious what the point of this is.
>
> Ross

Yes. It exists in meta-selftest, having an empty /usr/bin/theapp, for 
the purpose of selftest.

I think we could have some reference image for container. So I added 
this small reference image, from which users could create container and 
use shell.

I expect it to be like busybox:glibc or busybox:musl in Docker Hub, with 
the major difference that we use symlinks to /bin/busybox.nosuid and 
/bin/busybox.suid while official ones use separate binaries.

Regards,
Chen Qi

> On Wed, 24 Jul 2019 at 03:39, Chen Qi <Qi.Chen@windriver.com> wrote:
>> Add a small busybox based container image.
>>
>> Signed-off-by: Chen Qi <Qi.Chen@windriver.com>
>> ---
>>   meta/recipes-core/images/container-image-busybox.bb | 8 ++++++++
>>   1 file changed, 8 insertions(+)
>>   create mode 100644 meta/recipes-core/images/container-image-busybox.bb
>>
>> diff --git a/meta/recipes-core/images/container-image-busybox.bb b/meta/recipes-core/images/container-image-busybox.bb
>> new file mode 100644
>> index 0000000000..a4d63b0eea
>> --- /dev/null
>> +++ b/meta/recipes-core/images/container-image-busybox.bb
>> @@ -0,0 +1,8 @@
>> +SUMMARY = "A small busybox based container image."
>> +LICENSE = "MIT"
>> +
>> +IMAGE_INSTALL = "busybox"
>> +IMAGE_LINGUAS = ""
>> +IMAGE_FSTYPES = "container"
>> +
>> +inherit core-image
>> --
>> 2.17.1
>>
>> --
>> _______________________________________________
>> Openembedded-core mailing list
>> Openembedded-core@lists.openembedded.org
>> http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [PATCH] container-image-busybox: add a reference container image

[Bruce Ashfield]

On Wed, Jul 24, 2019 at 6:00 AM ChenQi <Qi.Chen@windriver.com> wrote:
>
> On 07/24/2019 05:38 PM, Burton, Ross wrote:
> > Are you aware of container-test-image.bb?  Curious what the point of this is.
> >
> > Ross
>
> Yes. It exists in meta-selftest, having an empty /usr/bin/theapp, for
> the purpose of selftest.
>
> I think we could have some reference image for container. So I added
> this small reference image, from which users could create container and
> use shell.
>

And you do realize that such references already exist in several other
layers ? So it would be best to do a study of those options, talk to the
interested folks and then we can propose something to oe-core.

> I expect it to be like busybox:glibc or busybox:musl in Docker Hub, with
> the major difference that we use symlinks to /bin/busybox.nosuid and
> /bin/busybox.suid while official ones use separate binaries.

Have a look at the ELCe presentations from 2018, this has been covered
in great detail at those events.

We can't just drop an image into core without documentation, examples,
test cases, etc. That's what I'm talking about as points of collaboration.

Cheers,

Bruce

>
> Regards,
> Chen Qi
>
> > On Wed, 24 Jul 2019 at 03:39, Chen Qi <Qi.Chen@windriver.com> wrote:
> >> Add a small busybox based container image.
> >>
> >> Signed-off-by: Chen Qi <Qi.Chen@windriver.com>
> >> ---
> >>   meta/recipes-core/images/container-image-busybox.bb | 8 ++++++++
> >>   1 file changed, 8 insertions(+)
> >>   create mode 100644 meta/recipes-core/images/container-image-busybox.bb
> >>
> >> diff --git a/meta/recipes-core/images/container-image-busybox.bb b/meta/recipes-core/images/container-image-busybox.bb
> >> new file mode 100644
> >> index 0000000000..a4d63b0eea
> >> --- /dev/null
> >> +++ b/meta/recipes-core/images/container-image-busybox.bb
> >> @@ -0,0 +1,8 @@
> >> +SUMMARY = "A small busybox based container image."
> >> +LICENSE = "MIT"
> >> +
> >> +IMAGE_INSTALL = "busybox"
> >> +IMAGE_LINGUAS = ""
> >> +IMAGE_FSTYPES = "container"
> >> +
> >> +inherit core-image
> >> --
> >> 2.17.1
> >>
> >> --
> >> _______________________________________________
> >> Openembedded-core mailing list
> >> Openembedded-core@lists.openembedded.org
> >> http://lists.openembedded.org/mailman/listinfo/openembedded-core
>
>
> --
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core



-- 
- Thou shalt not follow the NULL pointer, for chaos and madness await
thee at its end
- "Use the force Harry" - Gandalf, Star Trek II

Re: [PATCH] container-image-busybox: add a reference container image

[Bruce Ashfield]

On Wed, Jul 24, 2019 at 1:06 AM Khem Raj <raj.khem@gmail.com> wrote:
>
> On Tue, Jul 23, 2019 at 8:03 PM Bruce Ashfield <bruce.ashfield@gmail.com> wrote:
> >
> > On Tue, Jul 23, 2019 at 10:39 PM Chen Qi <Qi.Chen@windriver.com> wrote:
> > >
> > > Add a small busybox based container image.
> >
> > This really isn't a great container reference image. It isn't
> > manipulating many of
> > the things that you should be in a container image (linux-dummy, etc).
> >
> > Have you seen the reference images that have either been presented at the
> > various ELC conferences and/or the ones that are in meta-virtualization ?
> > Or the ones from Togan labs, meta-overc, etc ?
> >
> > IMHO, It would be better to collaborate on this in a non-core layer
> > for this release and
> > then consolidate things with the OCI image types, and when we sort out
> > if/what/when
> > makes sense from meta-virt into oe-core.
>
> I think this is very much suitable for OE-Core, and I would like us to
> have container reference image
> in core, however, I agree there is prework that should be includes
> from prior work in other layers and
> we should consolidate all that into a reference usable image in oe-core.

I definitely agree on that something belongs in oe-core, and yes, 100%
agreed that we shouldn't do this ad-hoc and should at a minimum have
documentation and some consolidated images / use cases from the
existing layers with container image types.

Cheers,

Bruce

>
> >
> > This is a topic of many opinions and different implementations and deserves some
> > cycles spent on documenting the choices, what we expect the base image to
> > do, etc.
> >
> > Cheers,
> >
> > Bruce
> >
> > >
> > > Signed-off-by: Chen Qi <Qi.Chen@windriver.com>
> > > ---
> > >  meta/recipes-core/images/container-image-busybox.bb | 8 ++++++++
> > >  1 file changed, 8 insertions(+)
> > >  create mode 100644 meta/recipes-core/images/container-image-busybox.bb
> > >
> > > diff --git a/meta/recipes-core/images/container-image-busybox.bb b/meta/recipes-core/images/container-image-busybox.bb
> > > new file mode 100644
> > > index 0000000000..a4d63b0eea
> > > --- /dev/null
> > > +++ b/meta/recipes-core/images/container-image-busybox.bb
> > > @@ -0,0 +1,8 @@
> > > +SUMMARY = "A small busybox based container image."
> > > +LICENSE = "MIT"
> > > +
> > > +IMAGE_INSTALL = "busybox"
> > > +IMAGE_LINGUAS = ""
> > > +IMAGE_FSTYPES = "container"
> > > +
> > > +inherit core-image
> > > --
> > > 2.17.1
> > >
> > > --
> > > _______________________________________________
> > > Openembedded-core mailing list
> > > Openembedded-core@lists.openembedded.org
> > > http://lists.openembedded.org/mailman/listinfo/openembedded-core
> >
> >
> >
> > --
> > - Thou shalt not follow the NULL pointer, for chaos and madness await
> > thee at its end
> > - "Use the force Harry" - Gandalf, Star Trek II
> > --
> > _______________________________________________
> > Openembedded-core mailing list
> > Openembedded-core@lists.openembedded.org
> > http://lists.openembedded.org/mailman/listinfo/openembedded-core



-- 
- Thou shalt not follow the NULL pointer, for chaos and madness await
thee at its end
- "Use the force Harry" - Gandalf, Star Trek II

Re: [PATCH] container-image-busybox: add a reference container image

[ChenQi]

On 07/24/2019 08:27 PM, Bruce Ashfield wrote:
> On Wed, Jul 24, 2019 at 6:00 AM ChenQi <Qi.Chen@windriver.com> wrote:
>> On 07/24/2019 05:38 PM, Burton, Ross wrote:
>>> Are you aware of container-test-image.bb?  Curious what the point of this is.
>>>
>>> Ross
>> Yes. It exists in meta-selftest, having an empty /usr/bin/theapp, for
>> the purpose of selftest.
>>
>> I think we could have some reference image for container. So I added
>> this small reference image, from which users could create container and
>> use shell.
>>
> And you do realize that such references already exist in several other
> layers ? So it would be best to do a study of those options, talk to the
> interested folks and then we can propose something to oe-core.
>
>> I expect it to be like busybox:glibc or busybox:musl in Docker Hub, with
>> the major difference that we use symlinks to /bin/busybox.nosuid and
>> /bin/busybox.suid while official ones use separate binaries.
> Have a look at the ELCe presentations from 2018, this has been covered
> in great detail at those events.
>
> We can't just drop an image into core without documentation, examples,
> test cases, etc. That's what I'm talking about as points of collaboration.
>
> Cheers,
>
> Bruce

Hi Bruce,

Thanks for your info and advice.
I'll do more investigation.

Regards,
Chen Qi

>> Regards,
>> Chen Qi
>>
>>> On Wed, 24 Jul 2019 at 03:39, Chen Qi <Qi.Chen@windriver.com> wrote:
>>>> Add a small busybox based container image.
>>>>
>>>> Signed-off-by: Chen Qi <Qi.Chen@windriver.com>
>>>> ---
>>>>    meta/recipes-core/images/container-image-busybox.bb | 8 ++++++++
>>>>    1 file changed, 8 insertions(+)
>>>>    create mode 100644 meta/recipes-core/images/container-image-busybox.bb
>>>>
>>>> diff --git a/meta/recipes-core/images/container-image-busybox.bb b/meta/recipes-core/images/container-image-busybox.bb
>>>> new file mode 100644
>>>> index 0000000000..a4d63b0eea
>>>> --- /dev/null
>>>> +++ b/meta/recipes-core/images/container-image-busybox.bb
>>>> @@ -0,0 +1,8 @@
>>>> +SUMMARY = "A small busybox based container image."
>>>> +LICENSE = "MIT"
>>>> +
>>>> +IMAGE_INSTALL = "busybox"
>>>> +IMAGE_LINGUAS = ""
>>>> +IMAGE_FSTYPES = "container"
>>>> +
>>>> +inherit core-image
>>>> --
>>>> 2.17.1
>>>>
>>>> --
>>>> _______________________________________________
>>>> Openembedded-core mailing list
>>>> Openembedded-core@lists.openembedded.org
>>>> http://lists.openembedded.org/mailman/listinfo/openembedded-core
>>
>> --
>> _______________________________________________
>> Openembedded-core mailing list
>> Openembedded-core@lists.openembedded.org
>> http://lists.openembedded.org/mailman/listinfo/openembedded-core
>
>