Re: [libvirt] [PATCH] Don't check if the path exists for LVM

I'm not entirely sure why Libvirt is checking to see if the target file exists for LVM but it is a problem. Empty volume groups do not appear in /dev. When libvirt starts these pools then fail to autostart because their path does not exist. This is not useful, this is the patch we are running to resolve this issue. ---  src/storage/storage_backend_logical.c | 8 +++-----  1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/src/storage/storage_backend_logical.c b/src/storage/storage_backend_logical.c index 42dec05ba0..2c60719b8c 100644 --- a/src/storage/storage_backend_logical.c +++ b/src/storage/storage_backend_logical.c @@ -641,11 +641,9 @@ virStorageBackendLogicalCheckPool(virStoragePoolObjPtr pool,  {      virStoragePoolDefPtr def = virStoragePoolObjGetDef(pool); -    /* If we can find the target.path as well as ensure that the -     * pool's def source -     */ -    *isActive = virFileExists(def->target.path) && -                virStorageBackendLogicalMatchPoolSource(pool); +    /* Ensure that the pool's def source is correct */ +    *isActive = virStorageBackendLogicalMatchPoolSource(pool); +      return 0;  }