From: Fabian Grünbichler <f.gruenbichler@proxmox.com>
Date: Tue, 5 Sep 2017 12:59:30 +0000 (+0200)
Subject: rbd: make monhost option optional
X-Git-Url: https://git.proxmox.com/?a=commitdiff_plain;h=0b9ef02ebc13e00bebe86836c3964cf123e4778d;hp=de8eff4d317774ce69f238858bdf3843c64c3e15;p=pve-storage.git

rbd: make monhost option optional

to allow differentiating between user-created external RBD storage
entries (WITH monhost), and those created and managed by pveceph
(without).

making monhost non-fixed allows easily opting into the managed behaviour via
'pvesm set STORAGE -delete monhost', but is also helpful for external clusters
(i.e., after adding or removing a monitor you need to update the monhost
parameter..)

adapt description accordingly.

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>
---

diff --git a/PVE/Storage/RBDPlugin.pm b/PVE/Storage/RBDPlugin.pm
index 250ee7c..4c2967c 100644
--- a/PVE/Storage/RBDPlugin.pm
+++ b/PVE/Storage/RBDPlugin.pm
@@ -258,7 +258,7 @@ sub plugindata {
 sub properties {
     return {
 	monhost => {
-	    description => "Monitors daemon ips.",
+	    description => "IP addresses of monitors (for external clusters).",
 	    type => 'string', format => 'pve-storage-portal-dns-list',
 	},
 	pool => {
@@ -284,7 +284,7 @@ sub options {
     return {
 	nodes => { optional => 1 },
 	disable => { optional => 1 },
-	monhost => { fixed => 1 },
+	monhost => { optional => 1},
 	pool => { optional => 1 },
 	username => { optional => 1 },
 	content => { optional => 1 },