← Back to team overview

curtin-dev team mailing list archive

Re: [Merge] ~mwhudson/curtin:FR-2657-record-block-device-path into curtin:master

 


Diff comments:

> diff --git a/curtin/commands/block_meta.py b/curtin/commands/block_meta.py
> index 5614883..45bec55 100644
> --- a/curtin/commands/block_meta.py
> +++ b/curtin/commands/block_meta.py
> @@ -2054,12 +2071,17 @@ def meta_custom(args):
>                  description="configuring %s: %s" % (command['type'],
>                                                      command['id'])):
>              try:
> -                handler(command, storage_config_dict, command_handlers)
> +                handler(command, storage_config_dict, context)
>              except Exception as error:
>                  LOG.error("An error occured handling '%s': %s - %s" %
>                            (item_id, type(error).__name__, error))
>                  raise
>  
> +    device_map_path = cfg['storage'].get('device_map_path')

Cool. I added some (basic!) words to the documentation.

> +    if device_map_path is not None:
> +        with open(device_map_path, 'w') as fp:
> +            json.dump(context.id_to_device, fp)
> +
>      if args.testmode:
>          util.subp(['losetup', '--detach'] + list(DEVS))
>  


-- 
https://code.launchpad.net/~mwhudson/curtin/+git/curtin/+merge/429404
Your team curtin developers is requested to review the proposed merge of ~mwhudson/curtin:FR-2657-record-block-device-path into curtin:master.



References