On 06/03/2015 07:44 AM, John Ferlan wrote: > https://bugzilla.redhat.com/show_bug.cgi?id=1181087 > > Patch 1 was just found by inspection > Patch 2 sets up for the the eventual fix > Patch 3 Adjusts the error message to make it clearer what the problem is > Patch 4 Adds a check in the checkPool path for valid FS storage pool config > before checking for IsMounted and declaring "isActive = true" > > John Ferlan (4): > storage: Remove extraneous @conn from function comments > storage: Refactor storage pool type checks > storage: FS backend adjust error message on error path > storage: Add check for valid FS types in checkPool callback > > src/storage/storage_backend_fs.c | 97 ++++++++++++++++++++-------------------- > 1 file changed, 48 insertions(+), 49 deletions(-) > Made suggested changes, adjusted commit messages, and pushed - Thanks - John -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list