]> git.proxmox.com Git - pve-storage.git/commit - PVE/Storage/Plugin.pm
add check_connection in storage_check_enabled.
authorAlexandre Derumier <aderumier@odiso.com>
Wed, 18 Jul 2012 11:35:00 +0000 (13:35 +0200)
committerDietmar Maurer <dietmar@proxmox.com>
Thu, 19 Jul 2012 08:58:19 +0000 (10:58 +0200)
commitc9eeac0131f3b4c09edd37146776633aa2c347df
tree8fbeb72cd643f2681f94cf94999a4cc32a1068c9
parent3c199da32cb2992a26f00e406794eb6fe2fa1f76
add check_connection in storage_check_enabled.

Currently, if a storage have an infinite timeout (nfs hard option or iscsi with high timeout for failover),
and if the storage is down or is doing ha failover with dual controller

- pvestatd hang on storage_info. (So pve-manager see the host as down, and hang on differents parts, gui,rrds display)

- pvedaemon hang on differents place (storage disk list,vm_status, hardware info,...)

- I also see a lot of qemu-img info process hanging in uninterruptible sleep state with iscsi storage.

This patch add a check to see if the storage is accessible.

For nfs and iscsi : ping with a timeout of 2 second.  (Maybe this can be polish with some kind of filesystem/storage read test with timeout...)

Default for other storage plugins is to check nothing. (not a problem with sheepdog,ceph,libiscsi... as commands have timeout)

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>
PVE/Storage.pm
PVE/Storage/Plugin.pm