storage: Detect partition tables in unsupported places

These likely belong to virtual machines and we shouldn't touch them.

See https://bugzilla.redhat.com/show_bug.cgi?id=2364699

Author: mvollmer

pkg/storaged/block/create-pages.jsx

@@ -38,11 +38,12 @@ import { make_btrfs_filesystem_card } from "../btrfs/filesystem.jsx";
 import { make_btrfs_subvolume_pages } from "../btrfs/subvolume.jsx";
 
 import { new_page } from "../pages.jsx";
+import { register_available_block_space } from "../utils";
 
 /* CARD must have page_name, page_location, and page_size set.
  */
 
-export function make_block_page(parent, block, card) {
+export function make_block_page(parent, block, card, options) {
     let is_crypto = block.IdUsage == 'crypto';
     let content_block = is_crypto ? client.blocks_cleartext[block.path] : block;
     const fstab_config = get_fstab_config(content_block || block, true);
@@ -62,7 +63,7 @@ export function make_block_page(parent, block, card) {
     const single_device_volume = block_btrfs_blockdev && block_btrfs_blockdev.data.num_devices === 1;
 
     if (client.blocks_ptable[block.path]) {
-        make_partition_table_page(parent, block, card);
+        make_partition_table_page(parent, block, card, options && options.partitionable);
         return;
     }
 

pkg/storaged/block/other.jsx

@@ -49,7 +49,7 @@ export function make_other_page(parent, block) {
         actions: partitionable_block_actions(block),
     });
 
-    make_block_page(parent, block, other_card);
+    make_block_page(parent, block, other_card, { partitionable: true });
 }
 
 const OtherCard = ({ card, block }) => {

pkg/storaged/drive/drive.jsx

@@ -105,7 +105,7 @@ export function make_drive_page(parent, drive) {
     }
 
     if (block.Size > 0) {
-        make_block_page(parent, block, card);
+        make_block_page(parent, block, card, { partitionable: true });
     } else {
         new_page(parent, card);
     }

pkg/storaged/mdraid/mdraid.jsx

@@ -247,7 +247,7 @@ export function make_mdraid_page(parent, mdraid) {
     if (!block) {
         new_page(parent, mdraid_card);
     } else
-        make_block_page(parent, block, mdraid_card);
+        make_block_page(parent, block, mdraid_card, { partitionable: true });
 }
 
 const MDRaidCard = ({ card, mdraid, block }) => {

pkg/storaged/partitions/partition-table.jsx

@@ -21,6 +21,9 @@ import cockpit from "cockpit";
 import React from "react";
 import client from "../client";
 
+import { Alert } from "@patternfly/react-core/dist/esm/components/Alert/index.js";
+import { CardBody } from "@patternfly/react-core/dist/esm/components/Card/index.js";
+
 import { get_partitions, register_available_free_space } from "../utils.js";
 import { StorageCard, ChildrenTable, new_page, new_card } from "../pages.jsx";
 import { format_dialog } from "../block/format-dialog.jsx";
@@ -84,20 +87,21 @@ function make_partition_pages(parent, block) {
                        block_ptable.Type == 'dos');
 }
 
-export function make_partition_table_page(parent, block, next_card) {
+export function make_partition_table_page(parent, block, next_card, partitionable) {
     const block_ptable = client.blocks_ptable[block.path];
 
     const parts_card = new_card({
         title: (block_ptable.Type
             ? cockpit.format(_("$0 partitions"), block_ptable.Type.toLocaleUpperCase())
             : _("Partitions")),
         next: next_card,
-        component: PartitionsCard,
+        component: partitionable ? PartitionsCard : UnexpectedPartitionsCard,
         props: { },
     });
 
     const p = new_page(parent, parts_card, { sorted: false });
-    make_partition_pages(p, block);
+    if (partitionable)
+        make_partition_pages(p, block);
 }
 
 const PartitionsCard = ({ card }) => {
@@ -110,3 +114,15 @@ const PartitionsCard = ({ card }) => {
         </StorageCard>
     );
 };
+
+const UnexpectedPartitionsCard = ({ card }) => {
+    return (
+        <StorageCard card={card}>
+            <CardBody>
+                <Alert isInline isPlain variant="info" title={_("Unexpected partitions")}>
+                    <p>{_("Partitions are not supported on this block device. If it is used as a disk for a virtual machine, the partitions must be managed by the operating system inside the virtual machine.")}</p>
+                </Alert>
+            </CardBody>
+        </StorageCard>
+    );
+};

test/verify/check-storage-lvm2

@@ -847,6 +847,31 @@ class TestStorageLvm2(storagelib.StorageCase):
             b.wait_in_text(self.card_desc("LVM2 logical volume", "Physical volumes"), bn(disk))
             b.wait_in_text(self.card_desc("LVM2 logical volume", "Physical volumes"), bn(disk2))
 
+    def testUnsupportedPartitions(self):
+        b = self.browser
+        m = self.machine
+
+        self.login_and_go("/storage")
+
+        dev_1 = self.add_ram_disk(100)
+        m.execute(f"vgcreate vgroup0 {dev_1}")
+        self.addCleanupVG("vgroup0")
+        m.execute("lvcreate vgroup0 -n lvol0 -l100%FREE")
+        m.execute("parted /dev/vgroup0/lvol0 -s mktable gpt mkpart primary ext2 1M 50M")
+        self.addCleanup(m.execute, "parted /dev/vgroup0/lvol0 -s mktable gpt; wipefs -a /dev/vgroup0/lvol0")
+
+        b.wait_text(self.card_row_col("Storage", row_name="lvol0", col_index=3), "GPT partitions")
+        self.click_card_row("Storage", name="lvol0")
+        b.wait_in_text(self.card("GPT partitions"), "Partitions are not supported on this block device.")
+
+        # Neither the partition nor the free space should be offered
+        # for creating new things.
+
+        b.click(self.card_parent_link())
+        b.click(self.card_parent_link())
+        self.click_devices_dropdown("Create LVM2 volume group")
+        b.wait_in_text("#dialog", "No disks are available")
+
 
 class TestStorageLvm2Destructive(storagelib.StorageCase):