[pve-devel] applied: [PATCH manager/storage v4 00/19] pveceph storage.cfg management

[Wolfgang Bumiller]

applied whole series

On Tue, Sep 05, 2017 at 02:59:29PM +0200, Fabian Grünbichler wrote:
> this patch series implements storage.cfg management for pveceph-managed ceph
> clusters.
> 
> the following is implemented:
> 
> - allow rbd storages without a hard-coded monitor list, using
>   /etc/pve/ceph.conf instead for pveceph-managed clusters and their storages
> - optionally adding/removing storages when creating/destroying a pool + GUI
> - creating/destroying pools happens in a worker now
> - adding/removing a pveceph-managed RBD storage handles its keyring as well
> 
> the following is still missing:
> - adapt RBD add/edit dialogue in GUI (follow up by Dominik)
> 
> high-level changes since v3 based on Dominik's feedback and further discussion:
> - drop explicit pveceph flag in favor of simply making monhost optional
> 
> high-level changes since v2 based on Thomas' feedback:
> - moved keyring handling to pve-storage (PVE::API2::Storage::Config)
> - dropped add/list/removestorages API paths in PVE::API2::Ceph
> 
> high-level changes since v1:
> - pveceph flag is now doing its magic on the storage plugin side as well, no
>   more need to update the monhost string when adding/removing a monitor (thanks
>   for the suggestion Dietmar!)
> - createpool/destroypool are now using a worker
> - treat more failures as fatal in API paths
> - {name} API placeholder for pool names is now {pool}
> - includes and extends Dominik's GUI patches to support the new functionality
> 
> changed in v4:
> 1, 3-5, 9
> 
> dropped in v4:
> 2
> 
> rest is just rebased..
> 
> pve-manager:
> 
> Dominik Csapak (3):
>   add a params object to the safedestroy window
>   add add storages checkbox to ceph pool creation
>   add remove_storages parameter to the pool destruction
> 
> Fabian Grünbichler (12):
>   ceph: add add_storage helper
>   ceph: add get_storages helper
>   ceph: create/destroypool: shorten variable name
>   ceph/createpool: optionally add storages
>   ceph/destroypool: optionally remove storages
>   ceph/destroypool: refactor to use get_storages
>   ceph: make create/destroypool API paths async
>   ceph/destroypool: move image check into worker
>   add task description for cephcreatepool
>   add showProgress to SafeDestroy
>   enable showProgress for create/destroy pool
>   rename 'Remove' ceph pool button to 'Destroy'
> 
>  PVE/API2/Ceph.pm                   | 215 ++++++++++++++++++++++++++-----------
>  www/manager6/Utils.js              |   1 +
>  www/manager6/ceph/Pool.js          |  17 ++-
>  www/manager6/window/SafeDestroy.js |  39 ++++++-
>  4 files changed, 205 insertions(+), 67 deletions(-)
> 
> 
> base-commit: 27769b1f5dd2ddd3efc16fb112b6fa221ea186f8
> 
> pve-storage:
> 
> Fabian Grünbichler (4):
>   rbd: make monhost option optional
>   rbd: refactor cmdline helpers
>   rbd: implement handling of unset monhost
>   rbd: manage keyring for pveceph storages
> 
>  PVE/API2/Storage/Config.pm | 24 ++++++++++++--
>  PVE/Storage/RBDPlugin.pm   | 80 ++++++++++++++++++++++++----------------------
>  2 files changed, 63 insertions(+), 41 deletions(-)
> 
> 
> base-commit: 5a2eba91dcaec0ba68052579d5e0e4ebcb499ff5
> -- 
> 2.11.0