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