public inbox for isar-users@googlegroups.com
 help / color / mirror / Atom feed
From: "'cedric.hombourger@siemens.com' via isar-users" <isar-users@googlegroups.com>
To: "quirin.gylstorff@siemens.com" <quirin.gylstorff@siemens.com>,
	"isar-users@googlegroups.com" <isar-users@googlegroups.com>,
	"Kiszka, Jan" <jan.kiszka@siemens.com>,
	"MOESSBAUER, Felix" <felix.moessbauer@siemens.com>
Subject: Re: [PATCH v5 04/12] rootfs: Copy the newly created initrd.img to the work directory
Date: Sat, 1 Nov 2025 08:26:32 +0000	[thread overview]
Message-ID: <cd2c4a2649734f2feb9b06993c509939d0ae3a25.camel@siemens.com> (raw)
In-Reply-To: <20251030094451.1303871-5-Quirin.Gylstorff@siemens.com>

On Thu, 2025-10-30 at 10:44 +0100, Quirin Gylstorff wrote:
> From: Quirin Gylstorff <quirin.gylstorff@siemens.com>
> 
> This allows to change the permissions of the initrd to 0644. And
> all followup steps do not need sudo.
> 
> This is necessary as dracut creates the initrd with 0600 permissions.
> 
> Signed-off-by: Quirin Gylstorff <quirin.gylstorff@siemens.com>
> ---
>  meta/classes/rootfs.bbclass | 14 ++++----------
>  1 file changed, 4 insertions(+), 10 deletions(-)
> 
> diff --git a/meta/classes/rootfs.bbclass
> b/meta/classes/rootfs.bbclass
> index 1f2ad80b..6b30744b 100644
> --- a/meta/classes/rootfs.bbclass
> +++ b/meta/classes/rootfs.bbclass
> @@ -594,17 +594,11 @@ rootfs_generate_initramfs() {
>              echo "Total number of modules: $mods_total"
>              echo "Generating initrd for kernel version:
> $kernel_version"
>              sudo -E chroot "${ROOTFSDIR}" sh -c ' \

you may want to add "set -e" here

I have created a negative test where I forcefully made the dracut
command fail and the build succeeded (while
ROOTFS_INITRAMFS_GENERATOR_CMDLINE had failed, the find command
succeeded as find does not complain if there are no matches)


> -                ${ROOTFS_INITRAMFS_GENERATOR_CMDLINE};'
> +                ${ROOTFS_INITRAMFS_GENERATOR_CMDLINE}; \
> +                find /boot -name "initrd.img-$kernel_version*" -exec
> install --mode 0644 {} /isar-work/initrd.img \; \
> +                '
>          done
> -        if [ -n "${INITRD_DEPLOY_FILE}" ]; then
> -            if [ -f "${ROOTFSDIR}/initrd.img" ]; then
> -                # debian (mkinitramfs)
> -                cp ${ROOTFSDIR}/initrd.img
> ${DEPLOYDIR}/${INITRD_DEPLOY_FILE}
> -            else
> -                # ubuntu (dracut)
> -                cp ${ROOTFSDIR}/boot/initrd.img
> ${DEPLOYDIR}/${INITRD_DEPLOY_FILE}
> -            fi
> -        fi
> +        install --owner $(id -u) --group $(id -g)
> ${WORKDIR}/initrd.img ${DEPLOYDIR}/${INITRD_DEPLOY_FILE}
>      else
>          echo "no kernel in this rootfs, do not generate initrd"
>      fi

-- 
Cedric Hombourger
Siemens AG
www.siemens.com

-- 
You received this message because you are subscribed to the Google Groups "isar-users" group.
To unsubscribe from this group and stop receiving emails from it, send an email to isar-users+unsubscribe@googlegroups.com.
To view this discussion visit https://groups.google.com/d/msgid/isar-users/cd2c4a2649734f2feb9b06993c509939d0ae3a25.camel%40siemens.com.

  parent reply	other threads:[~2025-11-01  8:26 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-10-30  9:44 [PATCH v5 00/12] Add support for dracut 'Quirin Gylstorff' via isar-users
2025-10-30  9:44 ` [PATCH v5 01/12] Add dracut to custom kernel builds 'Quirin Gylstorff' via isar-users
2025-10-30  9:44 ` [PATCH v5 02/12] rootfs: Allow to overwrite the initramfs generation cmds 'Quirin Gylstorff' via isar-users
2025-10-31 15:03   ` 'cedric.hombourger@siemens.com' via isar-users
2025-11-01  6:02     ` 'cedric.hombourger@siemens.com' via isar-users
2025-10-30  9:44 ` [PATCH v5 03/12] rootfs: Add isar-work directory to rootfs mounts 'Quirin Gylstorff' via isar-users
2025-10-30  9:44 ` [PATCH v5 04/12] rootfs: Copy the newly created initrd.img to the work directory 'Quirin Gylstorff' via isar-users
2025-10-30 12:04   ` 'Quirin Gylstorff' via isar-users
2025-11-01  8:26   ` 'cedric.hombourger@siemens.com' via isar-users [this message]
2025-10-30  9:44 ` [PATCH v5 05/12] rootfs: Add dracut to initramfs generator 'Quirin Gylstorff' via isar-users
2025-10-30  9:44 ` [PATCH v5 06/12] initramfs: allow to set the generator command 'Quirin Gylstorff' via isar-users
2025-10-30  9:44 ` [PATCH v5 07/12] Add class to generate custom dracut initramfs 'Quirin Gylstorff' via isar-users
2025-10-30  9:44 ` [PATCH v5 08/12] rootfs: add flag to use dracut if it is not part of the package list 'Quirin Gylstorff' via isar-users
2025-10-30  9:44 ` [PATCH v5 09/12] Add example dracut initramfs 'Quirin Gylstorff' via isar-users
2025-10-30  9:44 ` [PATCH v5 10/12] Add dracut module helper 'Quirin Gylstorff' via isar-users
2025-10-30 11:26   ` 'Jan Kiszka' via isar-users
2025-10-30 11:53     ` 'Quirin Gylstorff' via isar-users
2025-10-30  9:44 ` [PATCH v5 11/12] Use lighttpd as a example how to add a dracut module 'Quirin Gylstorff' via isar-users
2025-10-30  9:44 ` [PATCH v5 12/12] user_manual: Add dracut for initramfs generation 'Quirin Gylstorff' via isar-users

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=cd2c4a2649734f2feb9b06993c509939d0ae3a25.camel@siemens.com \
    --to=isar-users@googlegroups.com \
    --cc=cedric.hombourger@siemens.com \
    --cc=felix.moessbauer@siemens.com \
    --cc=jan.kiszka@siemens.com \
    --cc=quirin.gylstorff@siemens.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox