curtin-dev team mailing list archive

Thread
Date

Re: [Merge] ~ogayot/curtin:nvme-o-tcp-storageconfig into curtin:master

To: mp+458446@xxxxxxxxxxxxxxxxxx
From: Dan Bungert <mp+458446@xxxxxxxxxxxxxxxxxx>
Date: Thu, 11 Jan 2024 16:19:06 -0000
In-reply-to: <170498395202.922238.11027153605991001149.launchpad@juju-98d295-prod-launchpad-16>
Reply-to: mp+458446@xxxxxxxxxxxxxxxxxx
Sender: noreply@xxxxxxxxxxxxx

Review: Needs Fixing

As we are adjusting the config format, we want to update doc/topics/storage.rst as well with the changes to add the new storage command.

As we're early days in terms of nvme-stas support, I suggest you mark that documentation experimental similar to how ZFS things are marked.

Please either include that doc change in this MP, or land it first, so that we don't forget to do the doc update.

More of a review later.
-- 
https://code.launchpad.net/~ogayot/curtin/+git/curtin/+merge/458446
Your team curtin developers is subscribed to branch curtin:master.