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>
This commit is contained in:
committed by
Wolfgang Bumiller
parent
de8eff4d31
commit
0b9ef02ebc
@ -258,7 +258,7 @@ sub plugindata {
|
|||||||
sub properties {
|
sub properties {
|
||||||
return {
|
return {
|
||||||
monhost => {
|
monhost => {
|
||||||
description => "Monitors daemon ips.",
|
description => "IP addresses of monitors (for external clusters).",
|
||||||
type => 'string', format => 'pve-storage-portal-dns-list',
|
type => 'string', format => 'pve-storage-portal-dns-list',
|
||||||
},
|
},
|
||||||
pool => {
|
pool => {
|
||||||
@ -284,7 +284,7 @@ sub options {
|
|||||||
return {
|
return {
|
||||||
nodes => { optional => 1 },
|
nodes => { optional => 1 },
|
||||||
disable => { optional => 1 },
|
disable => { optional => 1 },
|
||||||
monhost => { fixed => 1 },
|
monhost => { optional => 1},
|
||||||
pool => { optional => 1 },
|
pool => { optional => 1 },
|
||||||
username => { optional => 1 },
|
username => { optional => 1 },
|
||||||
content => { optional => 1 },
|
content => { optional => 1 },
|
||||||
|
|||||||
Reference in New Issue
Block a user