[pve-devel] [v2 storage 2/4] rbd: make monhost option optional

Fabian Grünbichler f.gruenbichler at proxmox.com
Tue Aug 29 13:04:24 CEST 2017


to allow storages managed by pveceph to rely only on
/etc/pve/ceph.conf

Signed-off-by: Fabian Grünbichler <f.gruenbichler at proxmox.com>
---
new in v2

note: it might make sense to make this non-fixed, since if the storage is not
managed by pveceph, sooner or later the user will need to update the storage
definition when a new monitor is added and/or an old one is removed..

 PVE/Storage/RBDPlugin.pm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/PVE/Storage/RBDPlugin.pm b/PVE/Storage/RBDPlugin.pm
index c5590c4..70d1fda 100644
--- a/PVE/Storage/RBDPlugin.pm
+++ b/PVE/Storage/RBDPlugin.pm
@@ -288,7 +288,7 @@ sub options {
     return {
 	nodes => { optional => 1 },
 	disable => { optional => 1 },
-	monhost => { fixed => 1 },
+	monhost => { optional => 1, fixed => 1 },
 	pool => { optional => 1 },
 	username => { optional => 1 },
 	content => { optional => 1 },
-- 
2.11.0





More information about the pve-devel mailing list