<div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, 7 Jun 2019, 18:02 Jay Pipes, <<a href="mailto:jaypipes@gmail.com" target="_blank" rel="noreferrer">jaypipes@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 6/7/19 11:23 AM, Eric Fried wrote:<br>
>> Better still, add a standardized trait to os-traits for hyperthreading<br>
>> support, which is what I'd recommended in the original<br>
>> cpu-resource-tracking spec.<br>
> <br>
> HW_CPU_HYPERTHREADING was added via [1] and has been in os-traits since<br>
> 0.8.0.<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">I think we need a tri-state here. There are three options:</div><div dir="auto"><br></div><div dir="auto">1. Give me a node with hyperthreading enabled</div><div dir="auto">2. Give me a node with hyperthreading disabled</div><div dir="auto">3. I don't care</div><div dir="auto"><br></div><div dir="auto">For me, the lack of a trait is 3 - I wouldn't want existing flavours without this trait to cause hyperthreading to be disabled.</div><div dir="auto"><br></div><div dir="auto">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 levels.</div><div dir="auto"><br></div><div dir="auto">I'm not trying to shoot down new ideas, just explaining how we got here.</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Excellent, I had a faint recollection of that...<br>
<br>
-jay<br>
<br>
</blockquote></div></div></div>