On Thu, Feb 01, 2018 at 09:58:17AM -0500, John Ferlan wrote:
On 02/01/2018 09:04 AM, Michal Privoznik wrote:
In 2074ef6cd4a2 and c56cdf259 (and friends) we've added two attributes to virtio NICs: rx_queue_size and tx_queue_size. However, sysadmins might want to set these on per-host basis but don't necessarily have an access to domain XML (e.g. because they are generated by some other app). So let's expose them under
Define "sysadmins" here. If there's a management app running that covers some hosts, then the app is the sysadmin. Who will be setting these in the conf file? Some person? Manually? I don't think so. My bet is that it will be the same mgmt app that generates the XML. Or the same person who deploys the app, but still manually. If that app is unable to have a default, then why should libvirt be?
qemu.conf (the settings from domain XML still take precedence as they are more specific ones).
Signed-off-by: Michal Privoznik <mprivozn@redhat.com> ---
diff to v1: - Reworded docs and config file comment - Simplified logic in qemuBuildNicDevStr a bit - Make the values require qemu with corresponding capabilities
docs/formatdomain.html.in | 14 ++++++++-- src/qemu/libvirtd_qemu.aug | 4 +++ src/qemu/qemu.conf | 6 +++++ src/qemu/qemu_command.c | 55 +++++++++++++++++++++++++++----------- src/qemu/qemu_command.h | 3 ++- src/qemu/qemu_conf.c | 4 +++ src/qemu/qemu_conf.h | 3 +++ src/qemu/qemu_hotplug.c | 2 +- src/qemu/test_libvirtd_qemu.aug.in | 2 ++ 9 files changed, 74 insertions(+), 19 deletions(-)
Reviewed-by: John Ferlan <jferlan@redhat.com>
Sorry I missed this earlier, I'm going through my e-mail backlog kinda too late. But I don't think this should be released. Or anything similar. If we make this change, then where do we draw the line for setting defaults in qemu.conf? Will we eventually have defaults for first interface, memory size or even whole XML that get's parsed before the user-supplied one? I know it's too late to NACK this, but please consider reverting it before next release. Also feel free to discuss if you have arguments. Thank you.
John
-- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list