[Nova][Ironic] Reset Configurations in Baremetals Post Provisioning

Mark Goddard mark at stackhpc.com
Sun Jun 9 09:53:15 UTC 2019

On Fri, 7 Jun 2019, 18:02 Jay Pipes, <jaypipes at gmail.com> wrote:

> On 6/7/19 11:23 AM, Eric Fried wrote:
> >> Better still, add a standardized trait to os-traits for hyperthreading
> >> support, which is what I'd recommended in the original
> >> cpu-resource-tracking spec.
> >
> > HW_CPU_HYPERTHREADING was added via [1] and has been in os-traits since
> > 0.8.0.

I think we need a tri-state here. There are three options:

1. Give me a node with hyperthreading enabled
2. Give me a node with hyperthreading disabled
3. I don't care

For me, the lack of a trait is 3 - I wouldn't want existing flavours
without this trait to cause hyperthreading to be disabled.

The ironic deploy templates feature wasn't designed to support forbidden
traits - I don't think they were implemented at the time. The example use
cases so far have involved encoding values into a trait name, e.g.
CUSTOM_HYPERTHREADING_ON. Forbidden traits could be made to work in this
case, but it doesn't really extend to non Boolean things such as RAID

I'm not trying to shoot down new ideas, just explaining how we got here.

> Excellent, I had a faint recollection of that...
> -jay
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstack.org/pipermail/openstack-discuss/attachments/20190609/df4bf290/attachment.html>

More information about the openstack-discuss mailing list