Fix create partition job to work with LVM devices.

src/modules/partition/gui/CreatePartitionDialog.cpp

@@ -43,6 +43,8 @@
 #include <QDir>
 #include <QListWidgetItem>
 #include <QPushButton>
+#include <QRegularExpression>
+#include <QRegularExpressionValidator>
 #include <QSet>
 
 static QSet< FileSystem::Type > s_unmountableFS(
@@ -66,6 +68,19 @@ CreatePartitionDialog::CreatePartitionDialog( Device* device, PartitionNode* par
     m_ui->encryptWidget->setText( tr( "En&crypt" ) );
     m_ui->encryptWidget->hide();
 
+    if (m_device->type() == Device::Disk_Device) {
+        m_ui->lvNameLabel->hide();
+        m_ui->lvNameLineEdit->hide();
+    }
+    if (m_device->type() == Device::LVM_Device) {
+        /* LVM logical volume name can consist of: letters numbers _ . - +
+         * It cannot start with underscore _ and must not be equal to . or .. or any entry in /dev/
+         * QLineEdit accepts QValidator::Intermediate, so we just disable . at the beginning */
+        QRegularExpression re(QStringLiteral(R"(^(?!_|\.)[\w\-.+]+)"));
+        QRegularExpressionValidator *validator = new QRegularExpressionValidator(re, this);
+        m_ui->lvNameLineEdit->setValidator(validator);
+    }
+
     QStringList mountPoints = { "/", "/boot", "/home", "/opt", "/usr", "/var" };
     if ( PartUtils::isEfiSystem() )
         mountPoints << Calamares::JobQueue::instance()->globalStorage()->value( "efiSystemPartition" ).toString();
@@ -227,6 +242,10 @@ CreatePartitionDialog::createPartition()
         );
     }
 
+    if (m_device->type() == Device::LVM_Device) {
+        partition->setPartitionPath(m_device->deviceNode() + QStringLiteral("/") + m_ui->lvNameLineEdit->text().trimmed());
+    }
+
     PartitionInfo::setMountPoint( partition, m_ui->mountPointComboBox->currentText() );
     PartitionInfo::setFormat( partition, true );
 

src/modules/partition/gui/CreatePartitionDialog.ui

@@ -146,6 +146,16 @@
       </spacer>
      </item>
      <item row="6" column="0">
+      <widget class="QLabel" name="lvNameLabel">
+       <property name="text">
+        <string>LVM LV name</string>
+       </property>
+      </widget>
+     </item>
+     <item row="6" column="1">
+      <widget class="QLineEdit" name="lvNameLineEdit"/>
+     </item>
+     <item row="7" column="0">
       <widget class="QLabel" name="mountPointLabel">
        <property name="text">
         <string>&amp;Mount Point:</string>
@@ -155,7 +165,7 @@
        </property>
       </widget>
      </item>
-     <item row="6" column="1">
+     <item row="7" column="1">
       <widget class="QComboBox" name="mountPointComboBox">
        <property name="editable">
         <bool>true</bool>
@@ -165,21 +175,21 @@
        </property>
       </widget>
      </item>
-     <item row="7" column="1">
+     <item row="8" column="1">
       <widget class="QLabel" name="labelMountPoint">
        <property name="text">
         <string/>
        </property>
       </widget>
      </item>
-     <item row="8" column="0">
+     <item row="9" column="0">
       <widget class="QLabel" name="label_3">
        <property name="text">
         <string>Flags:</string>
        </property>
       </widget>
      </item>
-     <item row="8" column="1">
+     <item row="9" column="1">
       <widget class="QListWidget" name="m_listFlags">
        <property name="alternatingRowColors">
         <bool>true</bool>
@@ -192,7 +202,7 @@
        </property>
       </widget>
      </item>
-     <item row="9" column="0">
+     <item row="10" column="0">
       <spacer name="verticalSpacer">
        <property name="orientation">
         <enum>Qt::Vertical</enum>

src/modules/partition/jobs/CreatePartitionJob.cpp

@@ -30,6 +30,7 @@
 #include <kpmcore/backend/corebackendpartition.h>
 #include <kpmcore/backend/corebackendpartitiontable.h>
 #include <kpmcore/core/device.h>
+#include <kpmcore/core/lvmdevice.h>
 #include <kpmcore/core/partition.h>
 #include <kpmcore/core/partitiontable.h>
 #include <kpmcore/fs/filesystem.h>
@@ -78,10 +79,13 @@ CreatePartitionJob::prettyStatusMessage() const
 Calamares::JobResult
 CreatePartitionJob::exec()
 {
+    QString partitionPath;
+    FileSystem *fs;
+    Report report( nullptr );
+
+    if (m_device->type() == Device::Disk_Device) {
         int step = 0;
         const qreal stepCount = 4;
-
-    Report report( nullptr );
         QString message = tr( "The installer failed to create partition on disk '%1'." ).arg( m_device->name() );
 
         progress( step++ / stepCount );
@@ -106,7 +110,7 @@ CreatePartitionJob::exec()
         }
 
         progress( step++ / stepCount );
-    QString partitionPath = backendPartitionTable->createPartition( report, *m_partition );
+        partitionPath = backendPartitionTable->createPartition( report, *m_partition );
         if ( partitionPath.isEmpty() )
         {
             return Calamares::JobResult::error(
@@ -118,18 +122,10 @@ CreatePartitionJob::exec()
         backendPartitionTable->commit();
 
         progress( step++ / stepCount );
-    FileSystem& fs = m_partition->fileSystem();
+        fs = &m_partition->fileSystem();
-    if ( fs.type() == FileSystem::Unformatted || fs.type() == FileSystem::Extended )
+        if ( fs->type() == FileSystem::Unformatted || fs->type() == FileSystem::Extended )
             return Calamares::JobResult::ok();
 
-    if ( !fs.create( report, partitionPath ) )
-    {
-        return Calamares::JobResult::error(
-                   tr( "The installer failed to create file system on partition %1." ).arg( partitionPath ),
-                   report.toText()
-               );
-    }
-
         if ( !backendPartitionTable->setPartitionSystemType( report, *m_partition ) )
         {
             return Calamares::JobResult::error(
@@ -139,6 +135,32 @@ CreatePartitionJob::exec()
         }
 
         backendPartitionTable->commit();
+    }
+    else
+    {
+        LvmDevice *dev = dynamic_cast<LvmDevice*>(m_device);
+        m_partition->setState(Partition::StateNone);
+
+        partitionPath = m_partition->partitionPath();
+        QString lvname  = partitionPath.right(partitionPath.length() - partitionPath.lastIndexOf(QStringLiteral("/")) - 1);
+        if ( !LvmDevice::createLV(report, *dev, *m_partition, lvname))
+        {
+            return Calamares::JobResult::error(
+                       tr( "The installer failed to create LVM logical volume %1." ).arg( lvname ),
+                       report.toText()
+                   );
+        }
+        fs = &m_partition->fileSystem();
+    }
+
+    if ( !fs->create( report, partitionPath ) )
+    {
+        return Calamares::JobResult::error(
+                   tr( "The installer failed to create file system on partition %1." ).arg( partitionPath ),
+                   report.toText()
+               );
+    }
+
     return Calamares::JobResult::ok();
 }