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

John Ferlan posted 19 patches 6 years, 10 months ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/20170509153026.19767-1-jferlan@redhat.com
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(-)
[libvirt] [PATCH 00/19] Start altering storage code to privatize of the object
Posted by John Ferlan 6 years, 10 months ago
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
Re: [libvirt] [PATCH 00/19] Start altering storage code to privatize of the object
Posted by John Ferlan 6 years, 10 months ago
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