[PATCH 0/3] Rmove support of tftp storage protocol

[Han Han]

On Thu, May 21, 2020 at 2:44 PM Peter Krempa <pkrempa at redhat.com> wrote:

> On Thu, May 21, 2020 at 14:08:17 +0800, Han Han wrote:
> > Let's drop tftp support since it is ususabled before or after QEMU v2.8
> > [1].
> >
> > [1]: https://bugzilla.redhat.com/show_bug.cgi?id=1372143
> >
> >
> > Han Han (3):
> >   docs: Remove tftp protocol support from docs
> >   src: Remove tftp network storage support
> >   news: Remove support for tftp storage protocol
>
> NACK series:
>
> 1) qemu 2.8 is still supported by libvirt
>
However, the tftp is actually unusable for all versions of qemu, while the
formatdomain doc
states that it is optional protocol

> 2) we can't remove XML parser support for previously supported XMLs as
> the existing defined VMs would vanish. That's exactly why the check in
> my patch was added into the validator which doesn't validate existing
> defined configs.
>
I understand the consideration of the compatibility. How about only
removing it from docs?


-- 
Best regards,
-----------------------------------
Han Han
Quality Engineer
Redhat.

Email: hhan at redhat.com
Phone: +861065339333
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20200521/ba622862/attachment-0001.htm>