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

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

 





On Thu, May 21, 2020 at 2:44 PM Peter Krempa <pkrempa@xxxxxxxxxx> 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@xxxxxxxxxx
Phone: +861065339333

[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