Re: [PATCH v4] virt-aa-helper: Set the supported features

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Feb 06, 2018 at 14:48:23 +0100, Christian Ehrhardt wrote:
> On Tue, Feb 6, 2018 at 2:42 PM, Peter Krempa <pkrempa@xxxxxxxxxx> wrote:
> > On Tue, Feb 06, 2018 at 08:08:14 -0500, Shivaprasad G Bhat wrote:
> >> The virt-aa-helper fails to parse the xmls with the memory/cpu
> >> hotplug features or user assigned aliases. Set the features in
> >> xmlopt->config for the parsing to succeed.
> >>
> >> Signed-off-by: Shivaprasad G Bhat <sbhat@xxxxxxxxxxxxxxxxxx>
> >> ---
> >>  src/conf/domain_conf.h        |    2 ++
> >>  src/security/virt-aa-helper.c |    8 +++++++-
> >>  2 files changed, 9 insertions(+), 1 deletion(-)
> >
> > ACK from my side, but I'll wait for a while for the debian/ubuntu guys
> > chiming in.
> 
> I had a review and test on v2.
> This essentially differes by a comment that I also requested - so still ack.
> 
> Tested-by: Christian Ehrhardt <christian.ehrhardt@xxxxxxxxxxxxx>
> Reviewed-by: Christian Ehrhardt <christian.ehrhardt@xxxxxxxxxxxxx>

I've added the tags and pushed the patch.

> 
> P.S. you could say I'm an Ubuntu guy :-)

Definitely :)

Sorry for not addressing you directly but I was too lazy to look up the
previous conversation.

Attachment: signature.asc
Description: PGP signature

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux