curtin-dev team mailing list archive

[Chris Peterson <mp+473058@xxxxxxxxxxxxxxxxxx>]

Diff comments:

> diff --git a/curtin/commands/curthooks.py b/curtin/commands/curthooks.py
> index 0539c83..6a09277 100644
> --- a/curtin/commands/curthooks.py
> +++ b/curtin/commands/curthooks.py
> @@ -1953,7 +1972,13 @@ def builtin_curthooks(cfg, target, state):
>          if osfamily == DISTROS.debian:
>              # re-enable update_initramfs
>              enable_update_initramfs(cfg, target, machine)
> -            update_initramfs(target, all_kernels=True)
> +            if not (pathlib.Path(target) / 'usr/bin/dracut').exists():
> +                update_initramfs(target, all_kernels=True)

I think that call just gets bounced since we haven't re-enabled updates to the initramfs yet right? I agree we should put the check inside update_initramfs() so it always catches though.

> +            else:
> +                # For the NVMe/TCP POC, we don't have initramfs-tools anymore
> +                # at this point and dracut was not diverted, so no need to
> +                # re-run it.
> +                pass
>          elif osfamily == DISTROS.redhat:
>              redhat_update_initramfs(target, cfg)
>  


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