On 7/15/19 10:20 AM, Dominik Csapak wrote:
> do not warn in case there exist nautilus osds
> since the upgrade has to be done by the time a user should add new
> osds, it does not make sense to warn in that case
> 
> Signed-off-by: Dominik Csapak <d.csa...@proxmox.com>
> ---
>  PVE/CLI/pve5to6.pm | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/PVE/CLI/pve5to6.pm b/PVE/CLI/pve5to6.pm
> index 2724e33d..d3c8e2db 100644
> --- a/PVE/CLI/pve5to6.pm
> +++ b/PVE/CLI/pve5to6.pm
> @@ -561,8 +561,9 @@ sub check_ceph {
>      my $local_ceph_ver = PVE::Ceph::Tools::get_local_version(1);
>      if (defined($local_ceph_ver)) {
>       if ($local_ceph_ver == 14) {
> +         my $ceph_volume_osds = PVE::Ceph::Tools::ceph_volume_list();
>           my $scanned_osds = PVE::Tools::dir_glob_regex('/etc/ceph/osd', 
> '^.*\.json$');
> -         if (-e '/var/lib/ceph/osd/' && !defined($scanned_osds)) {
> +         if (-e '/var/lib/ceph/osd/' && !defined($scanned_osds) && !(keys 
> %$ceph_volume_osds)) {
>               log_warn("local Ceph version is Nautilus, local OSDs detected, 
> but no conversion from ceph-disk to ceph-volume done (yet).");
>           }
>       }
> 

applied, thanks!

_______________________________________________
pve-devel mailing list
pve-devel@pve.proxmox.com
https://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

Reply via email to