Bug 1289872

Summary: It's better to remove "sheepdog" from the type list when creating a storage pool
Product: [Community] Virtualization Tools Reporter: Xiaodai Wang <xiaodwan>
Component: virt-managerAssignee: Cole Robinson <crobinso>
Status: CLOSED DEFERRED QA Contact:
Severity: medium Docs Contact:
Priority: medium    
Version: unspecifiedCC: berrange, crobinso, gscrivan, juzhou, mxie, mzhan, rbalakri, tzheng
Target Milestone: ---   
Target Release: ---   
Hardware: x86_64   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2017-03-05 18:20:29 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Xiaodai Wang 2015-12-09 09:28:24 UTC
Description of problem:
It's better to remove "sheepdog" from the type list when creating a storage pool.

Version-Release number of selected component (if applicable):
virt-manager-1.2.1-8.el7.noarch

How reproducible:
100%

Steps to Reproduce:
1. Launch virt-manager.
2. Click "Edit->Connection Details", switch to Storage tab, Click '+' button and Create storage pool dialog pops up.
3. Click the type list button.

Actual results:
sheepdog displays in the list.

Expected results:
sheepdog is not supported by rhev, so it's better to remove it.

Additional info:

Comment 1 Pavel Hrdina 2016-05-26 07:05:03 UTC
Moving to RHEL-7.4, this is not a minor issue.

Comment 2 Pavel Hrdina 2016-05-26 07:08:51 UTC
Correction: this is a minor issue.

Comment 3 Pavel Hrdina 2017-02-08 17:11:59 UTC
Moving to upstream, this would require support from Libvirt to provide list of supported storage pools.

Comment 4 Cole Robinson 2017-03-05 18:20:29 UTC
This isn't really worth tracking against virt-manager IMO. If libvirt grows some kind of capability reporting for storage drivers then we will use it, but otherwise it's better just to advertise everything the app might support