[pve-devel] [PATCH pve-manager master v1 1/2] fix #6652: ceph: osd: enable autoactivation for OSD LVs on creation

[Max R. Carrara]

On Wed Aug 13, 2025 at 9:48 AM CEST, Fabian Grünbichler wrote:
> On August 12, 2025 6:46 pm, Max R. Carrara wrote:
> > ... by directly calling lvcreate instead of using the LVM storage
> > plugin's helper sub.
> > 
> > Autoactivation is required for LVs used by Ceph OSDs, as Ceph
> > otherwise doesn't activate them by itself.
> > 
> > This is a regression from f296ffc4e4d in pve-storage [0].
> > 
> > [0]: https://git.proxmox.com/?p=pve-storage.git;a=commitdiff;h=f296ffc4e4d64b574c3001dc7cc6af3da1406441
> > 
> > Fixes: #6652
> > Signed-off-by: Max R. Carrara <m.carrara at proxmox.com>
> > ---
> >  PVE/API2/Ceph/OSD.pm | 20 +++++++++++++++++++-
> >  1 file changed, 19 insertions(+), 1 deletion(-)
> > 
> > diff --git a/PVE/API2/Ceph/OSD.pm b/PVE/API2/Ceph/OSD.pm
> > index 23e187ce..4801f8dd 100644
> > --- a/PVE/API2/Ceph/OSD.pm
> > +++ b/PVE/API2/Ceph/OSD.pm
> > @@ -443,7 +443,25 @@ __PACKAGE__->register_method({
> >                  my $lv = $type . "-" . UUID::uuid();
> >  
> >                  PVE::Storage::LVMPlugin::lvm_create_volume_group($dev->{devpath}, $vg);
> > -                PVE::Storage::LVMPlugin::lvcreate($vg, $lv, "${size}k");
> > +
> > +                my $cmd = [
> > +                    '/sbin/lvcreate',
> > +                    '-aly',
> > +                    '-Wy',
> > +                    '--yes',
> > +                    '--size',
> > +                    # size in kilobytes
> > +                    $size . "k",
> > +                    '--name',
> > +                    $lv,
> > +                    # explicitly enable autoactivation,
> > +                    # otherwise ceph-volume cannot bring the LV online
> > +                    '--setautoactivation',
> > +                    'y',
> > +                    $vg,
> > +                ];
> > +
> > +                run_command($cmd, errmsg => "lvcreate '$vg/$lv' error");
>
> there's a second call to PVE::Storage::LVMPlugin::lvcreate a few lines
> below, is that not also problematic?

Mea culpa; missed that one. Will send a v2.

>
> >  
> >                  if (PVE::Diskmanage::is_partition($dev->{devpath})) {
> >                      eval { PVE::Diskmanage::change_parttype($dev->{devpath}, '8E00'); };
> > -- 
> > 2.47.2
> > 
> > 
> > 
> > _______________________________________________
> > pve-devel mailing list
> > pve-devel at lists.proxmox.com
> > https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
> > 
> > 
> > 
>
>
> _______________________________________________
> pve-devel mailing list
> pve-devel at lists.proxmox.com
> https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel