Bug 36605 - libvirt-check.sh: libvirt/check/timeout=5s too low
libvirt-check.sh: libvirt/check/timeout=5s too low
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: Virtualization - UVMM
UCS 3.2
amd64 Linux
: P5 normal (vote)
: UCS 4.0-0-errata
Assigned To: Philipp Hahn
Erik Damrose
:
Depends on: 33966
Blocks:
  Show dependency treegraph
 
Reported: 2014-11-12 16:08 CET by Philipp Hahn
Modified: 2015-01-29 11:41 CET (History)
6 users (show)

See Also:
What kind of report is it?: ---
What type of bug is this?: ---
Who will be affected by this bug?: ---
How will those affected feel about the bug?: ---
User Pain:
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional):
Max CVSS v3 score:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Philipp Hahn univentionstaff 2014-11-12 16:08:34 CET
+++ This bug was initially created as a clone of Bug #33966 +++
The UCRV libvirt/check/timeout=5 is much too low as an "virsh list" on a Xen host with 12 VMs took 5s. This leads to libvirtd being killed needlessly, which breaks the connection to UVMM. As that happened each time, UVMM takes minutes to recover a working connection to the host.

As "virsh list" depends on the number of VMs running on the cost, please also consider switching to another command which is not O(n) but O(1), like "virsh nodeinfo" (see "virsh help host" for a list of possible commands).

But on the affected host this also takes ~5s:
# time virsh nodeinfo >/dev/null
real    0m4.980s
user    0m0.212s
sys     0m0.060s

# time virsh capabilities >/dev/null 
real    0m6.885s
user    0m0.196s
sys     0m0.080s
Comment 1 Stefan Gohmann univentionstaff 2015-01-05 10:17:39 CET
Please also check a backport to UCS 3.2.
Comment 2 Philipp Hahn univentionstaff 2015-01-08 14:32:42 CET
r57187 | Bug #36605 libvirt: Copyright 2015
r57188 | Bug #36605 libvirt: Increase timeout in check script

Package: univention-virtual-machine-manager-node
Version: 3.0.4-1.87.201501081418
Branch: ucs_4.0-0
Scope: errata4.0-0

2014-12-08-univention-virtual-machine-manager-node.yaml
 Increase timeout for libvirtd check to 30 seconds.

Still using "list" as this is the most portable one.
Comment 3 Erik Damrose univentionstaff 2015-01-22 16:42:57 CET
OK: New timeout 30s
OK: On update with unchanged old default: set to timeout to 30s
OK: YAML
Comment 4 Janek Walkenhorst univentionstaff 2015-01-29 11:41:03 CET
<http://errata.univention.de/ucs/4.0/54.html>