On 26/08/2021 12:03, Aaron Lauterer wrote:
> This new parameter allows to set the RBD keyring or CephFS secret for an
> external Ceph Cluster right when creating the storage.
> Up until now, they had to be manually placed in
> /etc/pve/priv/ceph/$file.
> 
> In order to reuse as much code as possible, I had to adjust the
> PVE::CephConfig::ceph_create_keyfile method to be able to pass the
> secret in as parameter.
> 
> changes since v2: rebased and hopefully omit adding unintentional
> characters ;)
> changes since v1: fixed checks to avoid accidentially deleting the
> keyring/secret file
> 
> Aaron Lauterer (2):
>   CephConfig: add optional $secret parameter
>   Ceph: add keyring parameter for external clusters
> 
>  PVE/API2/Storage/Config.pm  |  2 +-
>  PVE/CLI/pvesm.pm            | 12 ++++++++++--
>  PVE/CephConfig.pm           | 15 +++++++++------
>  PVE/Storage/CephFSPlugin.pm | 22 ++++++++++++++++------
>  PVE/Storage/RBDPlugin.pm    | 26 ++++++++++++++++++++------
>  5 files changed, 56 insertions(+), 21 deletions(-)
> 



applied both patches, thanks!


_______________________________________________
pve-devel mailing list
[email protected]
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

Reply via email to