All of lore.kernel.org
 help / color / mirror / Atom feed
* [OE-Core][PATCH] eudev: Add group sgx to eudev package
@ 2023-07-14 14:09 Alex Kiernan
  2023-07-19 12:30 ` Alexandre Belloni
  0 siblings, 1 reply; 5+ messages in thread
From: Alex Kiernan @ 2023-07-14 14:09 UTC (permalink / raw)
  To: openembedded-core; +Cc: Alex Kiernan

Fix startup warning:

  udevd[171]: specified group 'sgx' unknown

This mirrors the change in bab455cd9b1b ("systemd: add group sgx to udev
package") for systemd-udev.

Signed-off-by: Alex Kiernan <alex.kiernan@gmail.com>
---

 meta/recipes-core/udev/eudev_3.2.12.bb | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/meta/recipes-core/udev/eudev_3.2.12.bb b/meta/recipes-core/udev/eudev_3.2.12.bb
index 572ccecafd0c..4268bcc2c5de 100644
--- a/meta/recipes-core/udev/eudev_3.2.12.bb
+++ b/meta/recipes-core/udev/eudev_3.2.12.bb
@@ -18,7 +18,7 @@ SRC_URI[sha256sum] = "ccdd64ec3c381d3c3ed0e99d2e70d1f62988c7763de89ca7bdffafa5ea
 
 GITHUB_BASE_URI = "https://github.com/eudev-project/eudev/releases"
 
-inherit autotools update-rc.d qemu pkgconfig features_check manpages github-releases
+inherit autotools update-rc.d qemu pkgconfig features_check manpages github-releases useradd
 
 CONFLICT_DISTRO_FEATURES = "systemd"
 
@@ -85,3 +85,6 @@ pkg_postinst:${PN}-hwdb () {
 pkg_prerm:${PN}-hwdb () {
 	rm -f $D${sysconfdir}/udev/hwdb.bin
 }
+
+USERADD_PACKAGES = "${PN}"
+GROUPADD_PARAM:${PN} = "-r sgx"
-- 
2.39.0



^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [OE-Core][PATCH] eudev: Add group sgx to eudev package
  2023-07-14 14:09 [OE-Core][PATCH] eudev: Add group sgx to eudev package Alex Kiernan
@ 2023-07-19 12:30 ` Alexandre Belloni
  2023-07-19 18:47   ` Alex Kiernan
       [not found]   ` <177359505CCD363D.12709@lists.openembedded.org>
  0 siblings, 2 replies; 5+ messages in thread
From: Alexandre Belloni @ 2023-07-19 12:30 UTC (permalink / raw)
  To: Alex Kiernan; +Cc: openembedded-core

Hello,

I had a bit of trouble to find this but this causes the following
oe-selftest failure:

https://autobuilder.yoctoproject.org/typhoon/#/builders/56/builds/2274/steps/14/logs/stdio

2023-07-18 20:56:16,128 - oe-selftest - INFO - gdbserver.GdbServerTest.test_gdb_server (subunit.RemotedTestCase)
2023-07-18 20:56:16,129 - oe-selftest - INFO -  ... ERROR
Stderr:
2023-07-18 20:32:39,581 - oe-selftest - INFO - Adding: "include selftest.inc" in /home/pokybuild/yocto-worker/oe-selftest/build/build-st-836115/conf/local.conf
2023-07-18 20:32:39,582 - oe-selftest - INFO - Adding: "include bblayers.inc" in bblayers.conf
2023-07-18 20:56:16,129 - oe-selftest - INFO - 14: 7/58 224/529 (269.86s) (0 failed) (gdbserver.GdbServerTest.test_gdb_server)
2023-07-18 20:56:16,129 - oe-selftest - INFO - testtools.testresult.real._StringException: Traceback (most recent call last):
  File "/home/pokybuild/yocto-worker/oe-selftest/build/meta/lib/oeqa/selftest/cases/gdbserver.py", line 43, in test_gdb_server
    shutil.unpack_archive(filename, debugfs)
  File "/usr/lib64/python3.11/shutil.py", line 1323, in unpack_archive
    func(filename, extract_dir, **kwargs)
  File "/usr/lib64/python3.11/shutil.py", line 1244, in _unpack_tarfile
    tarobj.extractall(extract_dir, filter=filter)
  File "/usr/lib64/python3.11/tarfile.py", line 2257, in extractall
    self._extract_one(tarinfo, path, set_attrs=not tarinfo.isdir(),
  File "/usr/lib64/python3.11/tarfile.py", line 2324, in _extract_one
    self._handle_fatal_error(e)
  File "/usr/lib64/python3.11/tarfile.py", line 2320, in _extract_one
    self._extract_member(tarinfo, os.path.join(path, tarinfo.name),
  File "/usr/lib64/python3.11/tarfile.py", line 2403, in _extract_member
    self.makefile(tarinfo, targetpath)
  File "/usr/lib64/python3.11/tarfile.py", line 2448, in makefile
    with bltn_open(targetpath, "wb") as target:
         ^^^^^^^^^^^^^^^^^^^^^^^^^^^
PermissionError: [Errno 13] Permission denied: '/tmp/debugfs-j_xgxhkm/./etc/gshadow'

On 14/07/2023 15:09:55+0100, Alex Kiernan wrote:
> Fix startup warning:
> 
>   udevd[171]: specified group 'sgx' unknown
> 
> This mirrors the change in bab455cd9b1b ("systemd: add group sgx to udev
> package") for systemd-udev.
> 
> Signed-off-by: Alex Kiernan <alex.kiernan@gmail.com>
> ---
> 
>  meta/recipes-core/udev/eudev_3.2.12.bb | 5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
> 
> diff --git a/meta/recipes-core/udev/eudev_3.2.12.bb b/meta/recipes-core/udev/eudev_3.2.12.bb
> index 572ccecafd0c..4268bcc2c5de 100644
> --- a/meta/recipes-core/udev/eudev_3.2.12.bb
> +++ b/meta/recipes-core/udev/eudev_3.2.12.bb
> @@ -18,7 +18,7 @@ SRC_URI[sha256sum] = "ccdd64ec3c381d3c3ed0e99d2e70d1f62988c7763de89ca7bdffafa5ea
>  
>  GITHUB_BASE_URI = "https://github.com/eudev-project/eudev/releases"
>  
> -inherit autotools update-rc.d qemu pkgconfig features_check manpages github-releases
> +inherit autotools update-rc.d qemu pkgconfig features_check manpages github-releases useradd
>  
>  CONFLICT_DISTRO_FEATURES = "systemd"
>  
> @@ -85,3 +85,6 @@ pkg_postinst:${PN}-hwdb () {
>  pkg_prerm:${PN}-hwdb () {
>  	rm -f $D${sysconfdir}/udev/hwdb.bin
>  }
> +
> +USERADD_PACKAGES = "${PN}"
> +GROUPADD_PARAM:${PN} = "-r sgx"
> -- 
> 2.39.0
> 

> 
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#184267): https://lists.openembedded.org/g/openembedded-core/message/184267
> Mute This Topic: https://lists.openembedded.org/mt/100141828/3617179
> Group Owner: openembedded-core+owner@lists.openembedded.org
> Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [alexandre.belloni@bootlin.com]
> -=-=-=-=-=-=-=-=-=-=-=-
> 


-- 
Alexandre Belloni, co-owner and COO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [OE-Core][PATCH] eudev: Add group sgx to eudev package
  2023-07-19 12:30 ` Alexandre Belloni
@ 2023-07-19 18:47   ` Alex Kiernan
       [not found]   ` <177359505CCD363D.12709@lists.openembedded.org>
  1 sibling, 0 replies; 5+ messages in thread
From: Alex Kiernan @ 2023-07-19 18:47 UTC (permalink / raw)
  To: Alexandre Belloni; +Cc: openembedded-core

On Wed, Jul 19, 2023 at 1:30 PM Alexandre Belloni
<alexandre.belloni@bootlin.com> wrote:
>
> Hello,
>
> I had a bit of trouble to find this but this causes the following
> oe-selftest failure:
>
> https://autobuilder.yoctoproject.org/typhoon/#/builders/56/builds/2274/steps/14/logs/stdio
>
> 2023-07-18 20:56:16,128 - oe-selftest - INFO - gdbserver.GdbServerTest.test_gdb_server (subunit.RemotedTestCase)
> 2023-07-18 20:56:16,129 - oe-selftest - INFO -  ... ERROR
> Stderr:
> 2023-07-18 20:32:39,581 - oe-selftest - INFO - Adding: "include selftest.inc" in /home/pokybuild/yocto-worker/oe-selftest/build/build-st-836115/conf/local.conf
> 2023-07-18 20:32:39,582 - oe-selftest - INFO - Adding: "include bblayers.inc" in bblayers.conf
> 2023-07-18 20:56:16,129 - oe-selftest - INFO - 14: 7/58 224/529 (269.86s) (0 failed) (gdbserver.GdbServerTest.test_gdb_server)
> 2023-07-18 20:56:16,129 - oe-selftest - INFO - testtools.testresult.real._StringException: Traceback (most recent call last):
>   File "/home/pokybuild/yocto-worker/oe-selftest/build/meta/lib/oeqa/selftest/cases/gdbserver.py", line 43, in test_gdb_server
>     shutil.unpack_archive(filename, debugfs)
>   File "/usr/lib64/python3.11/shutil.py", line 1323, in unpack_archive
>     func(filename, extract_dir, **kwargs)
>   File "/usr/lib64/python3.11/shutil.py", line 1244, in _unpack_tarfile
>     tarobj.extractall(extract_dir, filter=filter)
>   File "/usr/lib64/python3.11/tarfile.py", line 2257, in extractall
>     self._extract_one(tarinfo, path, set_attrs=not tarinfo.isdir(),
>   File "/usr/lib64/python3.11/tarfile.py", line 2324, in _extract_one
>     self._handle_fatal_error(e)
>   File "/usr/lib64/python3.11/tarfile.py", line 2320, in _extract_one
>     self._extract_member(tarinfo, os.path.join(path, tarinfo.name),
>   File "/usr/lib64/python3.11/tarfile.py", line 2403, in _extract_member
>     self.makefile(tarinfo, targetpath)
>   File "/usr/lib64/python3.11/tarfile.py", line 2448, in makefile
>     with bltn_open(targetpath, "wb") as target:
>          ^^^^^^^^^^^^^^^^^^^^^^^^^^^
> PermissionError: [Errno 13] Permission denied: '/tmp/debugfs-j_xgxhkm/./etc/gshadow'
>

That's interesting... really not sure why /etc/shadow doesn't trigger
this failure too (which is in the same set of tarballs). That said
I've no idea what the fix is, since this looks like collateral damage
rather than something which this change obviously broke. Will stare at
it a bit more.

> On 14/07/2023 15:09:55+0100, Alex Kiernan wrote:
> > Fix startup warning:
> >
> >   udevd[171]: specified group 'sgx' unknown
> >
> > This mirrors the change in bab455cd9b1b ("systemd: add group sgx to udev
> > package") for systemd-udev.
> >
> > Signed-off-by: Alex Kiernan <alex.kiernan@gmail.com>
> > ---
> >
> >  meta/recipes-core/udev/eudev_3.2.12.bb | 5 ++++-
> >  1 file changed, 4 insertions(+), 1 deletion(-)
> >
> > diff --git a/meta/recipes-core/udev/eudev_3.2.12.bb b/meta/recipes-core/udev/eudev_3.2.12.bb
> > index 572ccecafd0c..4268bcc2c5de 100644
> > --- a/meta/recipes-core/udev/eudev_3.2.12.bb
> > +++ b/meta/recipes-core/udev/eudev_3.2.12.bb
> > @@ -18,7 +18,7 @@ SRC_URI[sha256sum] = "ccdd64ec3c381d3c3ed0e99d2e70d1f62988c7763de89ca7bdffafa5ea
> >
> >  GITHUB_BASE_URI = "https://github.com/eudev-project/eudev/releases"
> >
> > -inherit autotools update-rc.d qemu pkgconfig features_check manpages github-releases
> > +inherit autotools update-rc.d qemu pkgconfig features_check manpages github-releases useradd
> >
> >  CONFLICT_DISTRO_FEATURES = "systemd"
> >
> > @@ -85,3 +85,6 @@ pkg_postinst:${PN}-hwdb () {
> >  pkg_prerm:${PN}-hwdb () {
> >       rm -f $D${sysconfdir}/udev/hwdb.bin
> >  }
> > +
> > +USERADD_PACKAGES = "${PN}"
> > +GROUPADD_PARAM:${PN} = "-r sgx"
> > --
> > 2.39.0
> >
>
> >
> > -=-=-=-=-=-=-=-=-=-=-=-
> > Links: You receive all messages sent to this group.
> > View/Reply Online (#184267): https://lists.openembedded.org/g/openembedded-core/message/184267
> > Mute This Topic: https://lists.openembedded.org/mt/100141828/3617179
> > Group Owner: openembedded-core+owner@lists.openembedded.org
> > Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [alexandre.belloni@bootlin.com]
> > -=-=-=-=-=-=-=-=-=-=-=-
> >
>
>
> --
> Alexandre Belloni, co-owner and COO, Bootlin
> Embedded Linux and Kernel engineering
> https://bootlin.com



--
Alex Kiernan


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [OE-Core][PATCH] eudev: Add group sgx to eudev package
       [not found]   ` <177359505CCD363D.12709@lists.openembedded.org>
@ 2023-07-19 19:58     ` Alex Kiernan
       [not found]     ` <17735D2C3E01ECA8.30279@lists.openembedded.org>
  1 sibling, 0 replies; 5+ messages in thread
From: Alex Kiernan @ 2023-07-19 19:58 UTC (permalink / raw)
  To: alex.kiernan; +Cc: Alexandre Belloni, openembedded-core

On Wed, Jul 19, 2023 at 7:48 PM Alex Kiernan via
lists.openembedded.org <alex.kiernan=gmail.com@lists.openembedded.org>
wrote:
>
> On Wed, Jul 19, 2023 at 1:30 PM Alexandre Belloni
> <alexandre.belloni@bootlin.com> wrote:
> >
> > Hello,
> >
> > I had a bit of trouble to find this but this causes the following
> > oe-selftest failure:
> >
> > https://autobuilder.yoctoproject.org/typhoon/#/builders/56/builds/2274/steps/14/logs/stdio
> >
> > 2023-07-18 20:56:16,128 - oe-selftest - INFO - gdbserver.GdbServerTest.test_gdb_server (subunit.RemotedTestCase)
> > 2023-07-18 20:56:16,129 - oe-selftest - INFO -  ... ERROR
> > Stderr:
> > 2023-07-18 20:32:39,581 - oe-selftest - INFO - Adding: "include selftest.inc" in /home/pokybuild/yocto-worker/oe-selftest/build/build-st-836115/conf/local.conf
> > 2023-07-18 20:32:39,582 - oe-selftest - INFO - Adding: "include bblayers.inc" in bblayers.conf
> > 2023-07-18 20:56:16,129 - oe-selftest - INFO - 14: 7/58 224/529 (269.86s) (0 failed) (gdbserver.GdbServerTest.test_gdb_server)
> > 2023-07-18 20:56:16,129 - oe-selftest - INFO - testtools.testresult.real._StringException: Traceback (most recent call last):
> >   File "/home/pokybuild/yocto-worker/oe-selftest/build/meta/lib/oeqa/selftest/cases/gdbserver.py", line 43, in test_gdb_server
> >     shutil.unpack_archive(filename, debugfs)
> >   File "/usr/lib64/python3.11/shutil.py", line 1323, in unpack_archive
> >     func(filename, extract_dir, **kwargs)
> >   File "/usr/lib64/python3.11/shutil.py", line 1244, in _unpack_tarfile
> >     tarobj.extractall(extract_dir, filter=filter)
> >   File "/usr/lib64/python3.11/tarfile.py", line 2257, in extractall
> >     self._extract_one(tarinfo, path, set_attrs=not tarinfo.isdir(),
> >   File "/usr/lib64/python3.11/tarfile.py", line 2324, in _extract_one
> >     self._handle_fatal_error(e)
> >   File "/usr/lib64/python3.11/tarfile.py", line 2320, in _extract_one
> >     self._extract_member(tarinfo, os.path.join(path, tarinfo.name),
> >   File "/usr/lib64/python3.11/tarfile.py", line 2403, in _extract_member
> >     self.makefile(tarinfo, targetpath)
> >   File "/usr/lib64/python3.11/tarfile.py", line 2448, in makefile
> >     with bltn_open(targetpath, "wb") as target:
> >          ^^^^^^^^^^^^^^^^^^^^^^^^^^^
> > PermissionError: [Errno 13] Permission denied: '/tmp/debugfs-j_xgxhkm/./etc/gshadow'
> >
>
> That's interesting... really not sure why /etc/shadow doesn't trigger
> this failure too (which is in the same set of tarballs). That said
> I've no idea what the fix is, since this looks like collateral damage
> rather than something which this change obviously broke. Will stare at
> it a bit more.
>

If I read the pieces right, we do:

        self._setup_dbg_rootfs(['/etc', '/var/lib/rpm',
'/var/cache/dnf', '/var/lib/dnf'])

I assume in order to prime the RPM config, which then copies gshadow
(and the rest of /etc - ignoring that its not using ${sysconfdir}),
which then causes this failure...

Something like:

diff --git a/meta/lib/oe/rootfs.py b/meta/lib/oe/rootfs.py
index 890ba5f03984..a2e81afb4f09 100644
--- a/meta/lib/oe/rootfs.py
+++ b/meta/lib/oe/rootfs.py
@@ -162,6 +162,9 @@ class Rootfs(object, metaclass=ABCMeta):
             bb.note("  Install extra debug packages...")
             self.pm.install(extra_debug_pkgs.split(), True)

+        sysconfdir = self.image_rootfs + self.d.getVar('sysconfdir')
+        shutil.rmtree(sysconfdir)
+
         bb.note("  Rename debug rootfs...")
         try:
             shutil.rmtree(self.image_rootfs + '-dbg')

Would appear sensible, but it feels rather "nuclear" and is way out of
my comfort zone of a proposed change!

> > On 14/07/2023 15:09:55+0100, Alex Kiernan wrote:
> > > Fix startup warning:
> > >
> > >   udevd[171]: specified group 'sgx' unknown
> > >
> > > This mirrors the change in bab455cd9b1b ("systemd: add group sgx to udev
> > > package") for systemd-udev.
> > >
> > > Signed-off-by: Alex Kiernan <alex.kiernan@gmail.com>
> > > ---
> > >
> > >  meta/recipes-core/udev/eudev_3.2.12.bb | 5 ++++-
> > >  1 file changed, 4 insertions(+), 1 deletion(-)
> > >
> > > diff --git a/meta/recipes-core/udev/eudev_3.2.12.bb b/meta/recipes-core/udev/eudev_3.2.12.bb
> > > index 572ccecafd0c..4268bcc2c5de 100644
> > > --- a/meta/recipes-core/udev/eudev_3.2.12.bb
> > > +++ b/meta/recipes-core/udev/eudev_3.2.12.bb
> > > @@ -18,7 +18,7 @@ SRC_URI[sha256sum] = "ccdd64ec3c381d3c3ed0e99d2e70d1f62988c7763de89ca7bdffafa5ea
> > >
> > >  GITHUB_BASE_URI = "https://github.com/eudev-project/eudev/releases"
> > >
> > > -inherit autotools update-rc.d qemu pkgconfig features_check manpages github-releases
> > > +inherit autotools update-rc.d qemu pkgconfig features_check manpages github-releases useradd
> > >
> > >  CONFLICT_DISTRO_FEATURES = "systemd"
> > >
> > > @@ -85,3 +85,6 @@ pkg_postinst:${PN}-hwdb () {
> > >  pkg_prerm:${PN}-hwdb () {
> > >       rm -f $D${sysconfdir}/udev/hwdb.bin
> > >  }
> > > +
> > > +USERADD_PACKAGES = "${PN}"
> > > +GROUPADD_PARAM:${PN} = "-r sgx"
> > > --
> > > 2.39.0
> > >
> >
> > >
> > >
> > >
> >
> >
> > --
> > Alexandre Belloni, co-owner and COO, Bootlin
> > Embedded Linux and Kernel engineering
> > https://bootlin.com
>
>
>
> --
> Alex Kiernan
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#184593): https://lists.openembedded.org/g/openembedded-core/message/184593
> Mute This Topic: https://lists.openembedded.org/mt/100141828/3618097
> Group Owner: openembedded-core+owner@lists.openembedded.org
> Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [alex.kiernan@gmail.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>


-- 
Alex Kiernan


^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [OE-Core][PATCH] eudev: Add group sgx to eudev package
       [not found]     ` <17735D2C3E01ECA8.30279@lists.openembedded.org>
@ 2023-07-20 10:21       ` Alex Kiernan
  0 siblings, 0 replies; 5+ messages in thread
From: Alex Kiernan @ 2023-07-20 10:21 UTC (permalink / raw)
  To: alex.kiernan; +Cc: Alexandre Belloni, openembedded-core

On Wed, Jul 19, 2023 at 8:58 PM Alex Kiernan via
lists.openembedded.org <alex.kiernan=gmail.com@lists.openembedded.org>
wrote:
>
> On Wed, Jul 19, 2023 at 7:48 PM Alex Kiernan via
> lists.openembedded.org <alex.kiernan=gmail.com@lists.openembedded.org>
> wrote:
> >
> > On Wed, Jul 19, 2023 at 1:30 PM Alexandre Belloni
> > <alexandre.belloni@bootlin.com> wrote:
> > >
> > > Hello,
> > >
> > > I had a bit of trouble to find this but this causes the following
> > > oe-selftest failure:
> > >
> > > https://autobuilder.yoctoproject.org/typhoon/#/builders/56/builds/2274/steps/14/logs/stdio
> > >
> > > 2023-07-18 20:56:16,128 - oe-selftest - INFO - gdbserver.GdbServerTest.test_gdb_server (subunit.RemotedTestCase)
> > > 2023-07-18 20:56:16,129 - oe-selftest - INFO -  ... ERROR
> > > Stderr:
> > > 2023-07-18 20:32:39,581 - oe-selftest - INFO - Adding: "include selftest.inc" in /home/pokybuild/yocto-worker/oe-selftest/build/build-st-836115/conf/local.conf
> > > 2023-07-18 20:32:39,582 - oe-selftest - INFO - Adding: "include bblayers.inc" in bblayers.conf
> > > 2023-07-18 20:56:16,129 - oe-selftest - INFO - 14: 7/58 224/529 (269.86s) (0 failed) (gdbserver.GdbServerTest.test_gdb_server)
> > > 2023-07-18 20:56:16,129 - oe-selftest - INFO - testtools.testresult.real._StringException: Traceback (most recent call last):
> > >   File "/home/pokybuild/yocto-worker/oe-selftest/build/meta/lib/oeqa/selftest/cases/gdbserver.py", line 43, in test_gdb_server
> > >     shutil.unpack_archive(filename, debugfs)
> > >   File "/usr/lib64/python3.11/shutil.py", line 1323, in unpack_archive
> > >     func(filename, extract_dir, **kwargs)
> > >   File "/usr/lib64/python3.11/shutil.py", line 1244, in _unpack_tarfile
> > >     tarobj.extractall(extract_dir, filter=filter)
> > >   File "/usr/lib64/python3.11/tarfile.py", line 2257, in extractall
> > >     self._extract_one(tarinfo, path, set_attrs=not tarinfo.isdir(),
> > >   File "/usr/lib64/python3.11/tarfile.py", line 2324, in _extract_one
> > >     self._handle_fatal_error(e)
> > >   File "/usr/lib64/python3.11/tarfile.py", line 2320, in _extract_one
> > >     self._extract_member(tarinfo, os.path.join(path, tarinfo.name),
> > >   File "/usr/lib64/python3.11/tarfile.py", line 2403, in _extract_member
> > >     self.makefile(tarinfo, targetpath)
> > >   File "/usr/lib64/python3.11/tarfile.py", line 2448, in makefile
> > >     with bltn_open(targetpath, "wb") as target:
> > >          ^^^^^^^^^^^^^^^^^^^^^^^^^^^
> > > PermissionError: [Errno 13] Permission denied: '/tmp/debugfs-j_xgxhkm/./etc/gshadow'
> > >
> >
> > That's interesting... really not sure why /etc/shadow doesn't trigger
> > this failure too (which is in the same set of tarballs). That said
> > I've no idea what the fix is, since this looks like collateral damage
> > rather than something which this change obviously broke. Will stare at
> > it a bit more.
> >
>
> If I read the pieces right, we do:
>
>         self._setup_dbg_rootfs(['/etc', '/var/lib/rpm',
> '/var/cache/dnf', '/var/lib/dnf'])
>
> I assume in order to prime the RPM config, which then copies gshadow
> (and the rest of /etc - ignoring that its not using ${sysconfdir}),
> which then causes this failure...
>
> Something like:
>
> diff --git a/meta/lib/oe/rootfs.py b/meta/lib/oe/rootfs.py
> index 890ba5f03984..a2e81afb4f09 100644
> --- a/meta/lib/oe/rootfs.py
> +++ b/meta/lib/oe/rootfs.py
> @@ -162,6 +162,9 @@ class Rootfs(object, metaclass=ABCMeta):
>              bb.note("  Install extra debug packages...")
>              self.pm.install(extra_debug_pkgs.split(), True)
>
> +        sysconfdir = self.image_rootfs + self.d.getVar('sysconfdir')
> +        shutil.rmtree(sysconfdir)
> +
>          bb.note("  Rename debug rootfs...")
>          try:
>              shutil.rmtree(self.image_rootfs + '-dbg')
>
> Would appear sensible, but it feels rather "nuclear" and is way out of
> my comfort zone of a proposed change!
>

Have sent a change to rework the way rpm is handled when building the
debugfs, which I think is the root cause of this failure.

> > > On 14/07/2023 15:09:55+0100, Alex Kiernan wrote:
> > > > Fix startup warning:
> > > >
> > > >   udevd[171]: specified group 'sgx' unknown
> > > >
> > > > This mirrors the change in bab455cd9b1b ("systemd: add group sgx to udev
> > > > package") for systemd-udev.
> > > >
> > > > Signed-off-by: Alex Kiernan <alex.kiernan@gmail.com>
> > > > ---
> > > >
> > > >  meta/recipes-core/udev/eudev_3.2.12.bb | 5 ++++-
> > > >  1 file changed, 4 insertions(+), 1 deletion(-)
> > > >
> > > > diff --git a/meta/recipes-core/udev/eudev_3.2.12.bb b/meta/recipes-core/udev/eudev_3.2.12.bb
> > > > index 572ccecafd0c..4268bcc2c5de 100644
> > > > --- a/meta/recipes-core/udev/eudev_3.2.12.bb
> > > > +++ b/meta/recipes-core/udev/eudev_3.2.12.bb
> > > > @@ -18,7 +18,7 @@ SRC_URI[sha256sum] = "ccdd64ec3c381d3c3ed0e99d2e70d1f62988c7763de89ca7bdffafa5ea
> > > >
> > > >  GITHUB_BASE_URI = "https://github.com/eudev-project/eudev/releases"
> > > >
> > > > -inherit autotools update-rc.d qemu pkgconfig features_check manpages github-releases
> > > > +inherit autotools update-rc.d qemu pkgconfig features_check manpages github-releases useradd
> > > >
> > > >  CONFLICT_DISTRO_FEATURES = "systemd"
> > > >
> > > > @@ -85,3 +85,6 @@ pkg_postinst:${PN}-hwdb () {
> > > >  pkg_prerm:${PN}-hwdb () {
> > > >       rm -f $D${sysconfdir}/udev/hwdb.bin
> > > >  }
> > > > +
> > > > +USERADD_PACKAGES = "${PN}"
> > > > +GROUPADD_PARAM:${PN} = "-r sgx"
> > > > --
> > > > 2.39.0
> > > >
> > >
> > > >
> > > >
> > > >
> > >
> > >
> > > --
> > > Alexandre Belloni, co-owner and COO, Bootlin
> > > Embedded Linux and Kernel engineering
> > > https://bootlin.com
> >
> >
> >
> > --
> > Alex Kiernan
> >
> >
> >
>
>
> --
> Alex Kiernan
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#184600): https://lists.openembedded.org/g/openembedded-core/message/184600
> Mute This Topic: https://lists.openembedded.org/mt/100141828/3618097
> Group Owner: openembedded-core+owner@lists.openembedded.org
> Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [alex.kiernan@gmail.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>


-- 
Alex Kiernan


^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2023-07-20 10:21 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-07-14 14:09 [OE-Core][PATCH] eudev: Add group sgx to eudev package Alex Kiernan
2023-07-19 12:30 ` Alexandre Belloni
2023-07-19 18:47   ` Alex Kiernan
     [not found]   ` <177359505CCD363D.12709@lists.openembedded.org>
2023-07-19 19:58     ` Alex Kiernan
     [not found]     ` <17735D2C3E01ECA8.30279@lists.openembedded.org>
2023-07-20 10:21       ` Alex Kiernan

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.