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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux