Code review comment for lp:~fred-yang/nova/TrustedComputingPools

Revision history for this message
fred yang (fred-yang) wrote :

Combining sting & value into a dict is good suggestion

> I don't know enough about the scheduler to comment on the functionality of the
> patch, but I do have a style comment. It doesn't seem necessary to split
> tlvl_string and tlvl_value as separate lists. Especially since the code
> combines them into a dict at runtime. It's probably cleaner to create a dict
> literal instead.

« Back to merge proposal