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

Revision history for this message
Johannes Erdfelt (johannes.erdfelt) wrote :

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