<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Feb 7, 2022 at 3:30 PM Julia Kreger <<a href="mailto:juliaashleykreger@gmail.com">juliaashleykreger@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">On Mon, Feb 7, 2022 at 12:21 PM Ruby Loo <<a href="mailto:opensrloo@gmail.com" target="_blank">opensrloo@gmail.com</a>> wrote:<br>
><br>
><br>
><br>
> On Mon, Feb 7, 2022 at 2:20 PM Carol Bouchard <<a href="mailto:cbouchar@redhat.com" target="_blank">cbouchar@redhat.com</a>> wrote:<br>
>><br>
[trim]<br>
>> Is this root_gb a new requirement?  More background: At the moment, I don't have baremetal devices and I'm starting my work by<br>
>> using VMs bifrost created for me.  Also I'm using the xena branch of bifrost/ironic.<br>
>><br>
>> Carol<br>
>><br>
><br>
> root_gb is old. I suspect bifrost doesn't use/need that information. If all the info you have for using the anaconda deploy interface is in the OS image or via configs, you'll be ok. Although when that patch merges, it'll break your use case if the VM-ironic-node doesn't have instance_info['root_gb'] specified. That code is getting triggered via:<br>
><br>
>    PXEAnacondaDeploy.prepare(): node.instance_info = deploy_utils.build_instance_info_for_deploy(<br>
>                 task).<br>
>    which causes this code to be invoked [1]<br>
><br>
> To get around it, you could set the ironic node's instance_info to have 'root_gb' = <some value>. Eg: "openstack baremetal node set <node> --instance-info root_gb=10"<br>
<br>
Yeah, we really need to nuke the requirement, but I could see it still<br>
being needed if someone tries to perform a partition image deployment.<br>
That being said, I don't think we should be requiring it on the<br>
anaconda deployment interface.<br>
<br>
Bifrost largely was designed around just deploying whole disk images,<br>
which is why it is not present on the node.<br>
<br><snip></blockquote><div><br></div><div>Ok, I updated the PR [1] to skip the root_gb check. The updated PR won't automatically backport nicely (because of a recent change to master) but it isn't too difficult to manually fix. Hopefully there won't be other issues ;)</div><div><br></div><div>[1] <a href="https://review.opendev.org/c/openstack/ironic/+/827924/3">https://review.opendev.org/c/openstack/ironic/+/827924/3</a> </div></div></div>