Re: [libvirt] [PATCH 00/19] Start altering storage code to privatize of the object

2017-05-19 Thread John Ferlan 

ping?

Tks -

John

On 05/09/2017 11:30 AM, John Ferlan wrote:
> Begin the process of making adjustments to the storage pool and volume code
> in order to privatize the virStoragePoolObj and virStoragePoolObjList
> 
> Didn't want to post 40+ patches at one time...  This is the first storage
> pile including some test driver cleanups w/r/t storage code.
> 
> John Ferlan (19):
>   test: Fix up formatting in storage test API's
>   test: Use consistent variable names for storage test driver APIs
>   test: Cleanup exit/failure paths of some storage pool APIs
>   test: Add helpers to fetch active/inactive storage pool by name
>   test: Add testStorageVolDefFindByName for storage volume tests
>   storage: Fix return value checks for virAsprintf
>   storage: Use consistent variable names in virstorageobj
>   storage: Use consistent variable names for driver
>   storage: Alter volume num, name, and export API's to just take obj
>   storage: Create accessor API's for virStoragePoolObj
>   storage: Introduce virStoragePoolObjNew
>   storage: Introduce virStoragePoolObj{Get|Set}Autostart
>   storage: Move autostartLink deletion to virstorageobj
>   storage: Introduce storage volume add, delete, count APIs
>   storage: Introduce virStoragePoolObjForEachVolume
>   storage: Use virStoragePoolObj accessors for driver
>   storage: Use virStoragePoolObj accessors for storage test API's
>   storage: Use virStoragePoolObj accessors for storage_util
>   storage: Change storage_util to use obj instead of pool
> 
>  src/conf/virstorageobj.c   |  568 +-
>  src/conf/virstorageobj.h   |   84 +-
>  src/libvirt_private.syms   |   16 +
>  src/storage/storage_backend_disk.c |   31 +-
>  src/storage/storage_backend_gluster.c  |3 +-
>  src/storage/storage_backend_logical.c  |5 +-
>  src/storage/storage_backend_mpath.c|3 +-
>  src/storage/storage_backend_rbd.c  |   12 +-
>  src/storage/storage_backend_sheepdog.c |8 +-
>  src/storage/storage_backend_zfs.c  |9 +-
>  src/storage/storage_driver.c   | 1308 
> 
>  src/storage/storage_driver.h   |4 +-
>  src/storage/storage_util.c |  200 ++---
>  src/storage/storage_util.h |   30 +-
>  src/test/test_driver.c |  780 +--
>  15 files changed, 1614 insertions(+), 1447 deletions(-)
> 

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

[libvirt] [PATCH 00/19] Start altering storage code to privatize of the object

2017-05-09 Thread John Ferlan 
Begin the process of making adjustments to the storage pool and volume code
in order to privatize the virStoragePoolObj and virStoragePoolObjList

Didn't want to post 40+ patches at one time...  This is the first storage
pile including some test driver cleanups w/r/t storage code.

John Ferlan (19):
  test: Fix up formatting in storage test API's
  test: Use consistent variable names for storage test driver APIs
  test: Cleanup exit/failure paths of some storage pool APIs
  test: Add helpers to fetch active/inactive storage pool by name
  test: Add testStorageVolDefFindByName for storage volume tests
  storage: Fix return value checks for virAsprintf
  storage: Use consistent variable names in virstorageobj
  storage: Use consistent variable names for driver
  storage: Alter volume num, name, and export API's to just take obj
  storage: Create accessor API's for virStoragePoolObj
  storage: Introduce virStoragePoolObjNew
  storage: Introduce virStoragePoolObj{Get|Set}Autostart
  storage: Move autostartLink deletion to virstorageobj
  storage: Introduce storage volume add, delete, count APIs
  storage: Introduce virStoragePoolObjForEachVolume
  storage: Use virStoragePoolObj accessors for driver
  storage: Use virStoragePoolObj accessors for storage test API's
  storage: Use virStoragePoolObj accessors for storage_util
  storage: Change storage_util to use obj instead of pool

 src/conf/virstorageobj.c   |  568 +-
 src/conf/virstorageobj.h   |   84 +-
 src/libvirt_private.syms   |   16 +
 src/storage/storage_backend_disk.c |   31 +-
 src/storage/storage_backend_gluster.c  |3 +-
 src/storage/storage_backend_logical.c  |5 +-
 src/storage/storage_backend_mpath.c|3 +-
 src/storage/storage_backend_rbd.c  |   12 +-
 src/storage/storage_backend_sheepdog.c |8 +-
 src/storage/storage_backend_zfs.c  |9 +-
 src/storage/storage_driver.c   | 1308 
 src/storage/storage_driver.h   |4 +-
 src/storage/storage_util.c |  200 ++---
 src/storage/storage_util.h |   30 +-
 src/test/test_driver.c |  780 +--
 15 files changed, 1614 insertions(+), 1447 deletions(-)

-- 
2.9.3

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