curtin-dev team mailing list archive

[Chris Peterson <mp+462092@xxxxxxxxxxxxxxxxxx>]

Diff comments:

> diff --git a/curtin/commands/curthooks.py b/curtin/commands/curthooks.py
> index 63a24fe..98cb34b 100644
> --- a/curtin/commands/curthooks.py
> +++ b/curtin/commands/curthooks.py
> @@ -920,6 +920,29 @@ def copy_crypttab(crypttab, target):
>      shutil.copy(crypttab, os.path.sep.join([target, 'etc/crypttab']))
>  
>  
> +def copy_cdrom(cdrom: str, target: str) -> None:
> +    """Copy installation media metadata"""
> +
> +    base: str = f"{cdrom}/.disk"
> +    ubuntu_dist_channel: str = f"{base}/ubuntu_dist_channel"
> +    media_info: str = f"{base}/info"
> +
> +    tdir: str = os.path.sep.join([target, "/var/log/installer"])
> +    util.ensure_dir(tdir)
> +
> +    if os.path.exists(media_info):
> +        LOG.info("copying media-info into target")
> +        shutil.copy(media_info, f"{tdir}/media-info")

typo log -> lib? Or do we want a new dir `/target/var/lib/installer`?

> +    else:
> +        LOG.warn(f"{media_info} not found, skipping")
> +
> +    if os.path.exists(ubuntu_dist_channel):
> +        LOG.info("copying ubuntu_dist_channel into target")
> +        shutil.copy(ubuntu_dist_channel, f"{tdir}/ubuntu_dist_channel")

good catch, thanks

> +    else:
> +        LOG.warn(f"{ubuntu_dist_channel} not found, skipping")
> +
> +
>  def copy_iscsi_conf(nodes_dir, target, target_nodes_dir='etc/iscsi/nodes'):
>      if not nodes_dir:
>          LOG.warn("nodes directory must be specified, not copying")


-- 
https://code.launchpad.net/~cpete/curtin/+git/curtin/+merge/462092
Your team curtin developers is subscribed to branch curtin:master.