[libvirt PATCH v3 00/11] Add ability to create mediated devices in libvirt

Michal Privoznik mprivozn at redhat.com
Tue Jun 16 18:41:38 UTC 2020


On 6/16/20 7:23 PM, Erik Skultety wrote:
> On Tue, Jun 16, 2020 at 09:27:48AM -0500, Jonathon Jongsma wrote:
>> This is the first portion of an effort to support persistent mediated devices
>> with libvirt. This first series simply enables creating and destroying
>> non-persistent mediated devices via the virNodeDeviceCreateXML() and
>> virNodeDeviceDestroy() functions. The 'mdevctl' utility[1] provides the backend
>> implementation.
>>
>> Changes in v3:
>>   - Includes all of the changes mentioned by Michal
>>   - makes iommuGroup element optional for the nodedev schema (per Erik)
>>   - substitutes a placeholder for the path to the mdevctl binary before
>>     comparing to the expected output
> 
> So, looks like that the gitlab pipeline doesn't like your changes
> https://gitlab.com/eskultety/libvirt/-/pipelines/156830100
> 
> However, I can run the same thing locally with
> "make -C ci ci-check at fedora-31" successfully as well as on bare metal without
> mdevctl installed, so I'll have to look into that more closely what's going on,
> this is merely an FYI if you don't run your builds in gitlab by any chance. If
> you do, great, that means that I forgot how to push to a git branch [1], anyways,
> I'll look into that failure in the morning.

Yeah, this is very weird because I cannot reproduce the issue.

The patches look good though.

Reviewed-by: Michal Privoznik <mprivozn at redhat.com>

However, what's missing is news.rst addition. But that can be a follow 
up patch.

Michal




More information about the libvir-list mailing list