[pve-devel] applied: [PATCH manager 1/2] pveceph: Make service parameter optional

Thomas Lamprecht t.lamprecht at proxmox.com
Mon Feb 17 15:37:09 CET 2020


On 2/17/20 12:28 PM, Dominic Jäger wrote:
> The default value for "pveceph start" and "pveceph stop" is "ceph.target".
> However, omitting the parameter to use the default has been forbidden.
> 
> Signed-off-by: Dominic Jäger <d.jaeger at proxmox.com>
> ---
>  PVE/CLI/pveceph.pm | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/PVE/CLI/pveceph.pm b/PVE/CLI/pveceph.pm
> index 3e37a7cd..064ae545 100755
> --- a/PVE/CLI/pveceph.pm
> +++ b/PVE/CLI/pveceph.pm
> @@ -177,8 +177,8 @@ our $cmddef = {
>  	create => [ 'PVE::API2::Ceph::MDS', 'createmds', [], { node => $nodename }, $upid_exit],
>  	destroy => [ 'PVE::API2::Ceph::MDS', 'destroymds', ['name'], { node => $nodename }, $upid_exit],
>      },
> -    start => [ 'PVE::API2::Ceph', 'start', ['service'], { node => $nodename }, $upid_exit],
> -    stop => [ 'PVE::API2::Ceph', 'stop', ['service'], { node => $nodename }, $upid_exit],
> +    start => [ 'PVE::API2::Ceph', 'start', [], { node => $nodename }, $upid_exit],
> +    stop => [ 'PVE::API2::Ceph', 'stop', [], { node => $nodename }, $upid_exit],
>      install => [ __PACKAGE__, 'install', [] ],
>      purge => [  __PACKAGE__, 'purge', [] ],
>      status => [ 'PVE::API2::Ceph', 'status', [], { node => $nodename }, sub {
> 

applied, thanks!




More information about the pve-devel mailing list