[libvirt] [PATCH 7/7] storage: Add storage backend pool/vol API's to capability output

[Michal Privoznik]

On 1/16/19 2:15 AM, John Ferlan wrote:
> https://bugzilla.redhat.com/show_bug.cgi?id=1581670
> 
> Add the Storage Pool and Volume API's defined for each generated
> capability output, such as:
> 
>   <pool>
>      <type>dir</pool>
>      <poolapis>
>        <buildPool/>
>        <refreshPool/>
>        <deletePool/>
>      </poolapis>
>      <volapis>
>        <buildVol/>
>        <buildVolFrom/>
>        <createVol/>
>        <refreshVol/>
>        <deleteVol/>
>        <resizeVol/>
>        <uploadVol/>
>        <downloadVol/>
>        <wipeVol/>
>      </volapis>
>    </pool>
> 
> ...
> 
>    <pool>
>      <type>iscsi</pool>
>      <poolapis>
>        <findPoolSources/>
>        <startPool/>
>        <refreshPool/>
>        <stopPool/>
>      </poolapis>
>      <volapis>
>        <uploadVol/>
>        <downloadVol/>
>        <wipeVol/>
>      </volapis>
>    </pool>

Frankly, I don't like this. Firstly, our users don't see 'uploadVol' 
they see 'virStorageVolUpload'. Secondly, I am not sure if we want to be 
introspectable this much. I mean, our users can just call the API and 
deal with VIR_ERR_NO_SUPPORT. This looks like an overkill to me.

Michal