[libvirt] [PATCH 15/21] caps2xml resctrl-skx-twocaches

[Martin Kletzander]

On Tue, Nov 14, 2017 at 04:20:45PM -0500, John Ferlan wrote:
>
>Would be nice to have more information here regarding what this is....
>

Only after sending have I found out that I somehow missed few commits when
adding commit messages.  This is one of the examples, another one is "conf:
cachetune".  I can resend part of the series or just add it as a reply to the
mail if you want.

>It seems this is just some new test...  It works, but does it add to
>test bloat? Could it be merged into existing skx test? IOW: does the
>existing test "need" to have just one bank?  I'm OK with the separate
>test, but I guess it's just not clear why there needs to be a separate
>one (I could have missed something subtle too).  In any case -
>

For example the commit message that's missing here would say that this
particular test data actually make future test from patch 19 to have more
coverage.  There are test cases with comments on why particular cases should
fail and I wanted to cover all obvious failures with only a few system<->domain
combinations.  I would have to find the table and the matrix of all of them that
I wrote somewhere on a paper to tell you why exactly this one is needed, but it
is written somewhere, maybe it's just in trash =D

>Reviewed-by: John Ferlan <jferlan at redhat.com>
>
>John
>
>[apologies for the "timeliness" of reviews - it's been a distracting
>day... Sometimes it's more worthwhile to just turn off IRC ;-)]
>

No need to apologize, it's very fast review, I thought I will not get any for
couple of weeks =D
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: Digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20171115/f6cd2c0e/attachment-0001.sig>