meta-isar: Fixing isar-image-installer to bundle correct target disk image.

Message ID 20240619095351.192990-1-alexander.heinisch@siemens.com
State Accepted, archived
Headers show
Series meta-isar: Fixing isar-image-installer to bundle correct target disk image. | expand

Commit Message

alexander.heinisch June 19, 2024, 9:53 a.m. UTC
From: Alexander Heinisch <alexander.heinisch@siemens.com>

When using multiconfig setups with differing machine configs the deploy dir of the target to be installed differs from the deploy dir of the installer.

Signed-off-by: Alexander Heinisch <alexander.heinisch@siemens.com>
---
 meta-isar/classes/installer-add-rootfs.bbclass | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

Comments

Uladzimir Bely June 26, 2024, 6:30 a.m. UTC | #1
On Wed, 2024-06-19 at 11:53 +0200, alexander.heinisch via isar-users
wrote:
> From: Alexander Heinisch <alexander.heinisch@siemens.com>
> 
> When using multiconfig setups with differing machine configs the
> deploy dir of the target to be installed differs from the deploy dir
> of the installer.
> 
> Signed-off-by: Alexander Heinisch <alexander.heinisch@siemens.com>
> ---
>  meta-isar/classes/installer-add-rootfs.bbclass | 5 +++--
>  1 file changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/meta-isar/classes/installer-add-rootfs.bbclass b/meta-
> isar/classes/installer-add-rootfs.bbclass
> index 7767d96e..c738f690 100644
> --- a/meta-isar/classes/installer-add-rootfs.bbclass
> +++ b/meta-isar/classes/installer-add-rootfs.bbclass
> @@ -12,6 +12,7 @@ INSTALLER_TARGET_IMAGE ??= "isar-image-base"
>  INSTALLER_TARGET_MC ??= "installer-target"
>  INSTALLER_TARGET_DISTRO ??= "${DISTRO}"
>  INSTALLER_TARGET_MACHINE ??= "${MACHINE}"
> +INSTALLER_TARGET_DEPLOY_DIR_IMAGE ??=
> "${DEPLOY_DIR}/images/${INSTALLER_TARGET_MACHINE}"
>  
>  IMAGE_DATA_FILE ??= "${INSTALLER_TARGET_IMAGE}-
> ${INSTALLER_TARGET_DISTRO}-${INSTALLER_TARGET_MACHINE}"
>  IMAGE_DATA_POSTFIX ??= "wic.zst"
> @@ -24,9 +25,9 @@ def get_installer_source(d, suffix):
>      installer_target_image = d.getVar('INSTALLER_TARGET_IMAGE') or
> ""
>      if not installer_target_image:
>          return ""
> -    deploy_dir = d.getVar('DEPLOY_DIR_IMAGE')
> +    target_deploy_dir =
> d.getVar('INSTALLER_TARGET_DEPLOY_DIR_IMAGE')
>      image_data = d.getVar('IMAGE_DATA_FILE')
> -    return f"{deploy_dir}/{image_data}.{suffix}"
> +    return f"{target_deploy_dir}/{image_data}.{suffix}"
>  
>  def get_installer_destination(d, suffix):
>      installer_target_image = d.getVar('INSTALLER_TARGET_IMAGE') or
> ""
> -- 
> 2.43.0
> 

Applied to next, thanks.

Patch

diff --git a/meta-isar/classes/installer-add-rootfs.bbclass b/meta-isar/classes/installer-add-rootfs.bbclass
index 7767d96e..c738f690 100644
--- a/meta-isar/classes/installer-add-rootfs.bbclass
+++ b/meta-isar/classes/installer-add-rootfs.bbclass
@@ -12,6 +12,7 @@  INSTALLER_TARGET_IMAGE ??= "isar-image-base"
 INSTALLER_TARGET_MC ??= "installer-target"
 INSTALLER_TARGET_DISTRO ??= "${DISTRO}"
 INSTALLER_TARGET_MACHINE ??= "${MACHINE}"
+INSTALLER_TARGET_DEPLOY_DIR_IMAGE ??= "${DEPLOY_DIR}/images/${INSTALLER_TARGET_MACHINE}"
 
 IMAGE_DATA_FILE ??= "${INSTALLER_TARGET_IMAGE}-${INSTALLER_TARGET_DISTRO}-${INSTALLER_TARGET_MACHINE}"
 IMAGE_DATA_POSTFIX ??= "wic.zst"
@@ -24,9 +25,9 @@  def get_installer_source(d, suffix):
     installer_target_image = d.getVar('INSTALLER_TARGET_IMAGE') or ""
     if not installer_target_image:
         return ""
-    deploy_dir = d.getVar('DEPLOY_DIR_IMAGE')
+    target_deploy_dir = d.getVar('INSTALLER_TARGET_DEPLOY_DIR_IMAGE')
     image_data = d.getVar('IMAGE_DATA_FILE')
-    return f"{deploy_dir}/{image_data}.{suffix}"
+    return f"{target_deploy_dir}/{image_data}.{suffix}"
 
 def get_installer_destination(d, suffix):
     installer_target_image = d.getVar('INSTALLER_TARGET_IMAGE') or ""