On Mon, Mar 06, 2023 at 01:49:50PM +0100, Martin Kletzander wrote:
On Mon, Mar 06, 2023 at 04:41:28AM -0800, Andrea Bolognani wrote:
A test called "controller-order" is definitely unrelated to SPICE TLS. For "q35-virt-manager-basic", I guess you could argue either way? It feels more unrelated than related to me.
No, of course, that's not the point, it is not trying to test spice with tls. But because the spice-tls tests are not next to each other (and would probably make a bigger mess in the order anyway) it *feels* arbitrary when the spiceTLS is set and unset, it's fine the way it is.
Gotcha :)
Or were you referring to the various VIR_FREE(*TLSx509secretUUID) calls that I'm adding? Because that was indeed intended to be a separate commit, which I seem to have accidentally squashed in O:-)
Posted that change as a separate patch, as it was originally intended, here: https://listman.redhat.com/archives/libvir-list/2023-March/238448.html Please let me know if I can use that instead of the combined version from v1 that you ACKed. -- Andrea Bolognani / Red Hat / Virtualization