diff --git a/doc/anaconda.md b/doc/anaconda.md new file mode 100644 index 00000000000..eea9cc67d32 --- /dev/null +++ b/doc/anaconda.md @@ -0,0 +1,84 @@ +Cockpit Storage in Anaconda Mode +================================ + +Anaconda (the OS Installer) can open the Cockpit "storaged" page for +advanced setup of the target storage devices. When this is done, +storaged is in a special "Anaconda mode" and behaves significantly +different. + +In essence, the storaged page restricts itself to working with the +target environment. It will hide the real root filesystem (on the USB +stick that the Live environment was booted from, say), but let the +user create a "fake" root filesystem on some block device. + +Entering Anaconda mode +---------------------- + +The "storaged" page is put into Anaconda mode by storing a +"cockpit_anaconda" item in its `window.localStorage`. The value +should be a JSON encoded object, the details of which are explained +below. + +Since both Anaconda and the storaged page are served from the same +origin, Anaconda can just execute something like this: + +``` + window.localStorage.setItem("cockpit_anaconda", + JSON.stringify({ + "mount_point_prefix": "/sysroot", + "ignore_devices": [ "/dev/sr0", "/dev/loop0" ] + })); + window.open("/cockpit/@localhost/storage/index.html", "storage-tab"); +``` + +Ignoring storage devices +------------------------ + +Anaconda needs to tell Cockpit which devices can not be used to +install the OS on. This is done with the "ignore_devices" entry, which +is a array of strings. + +``` +{ + "ignore_devices": [ "/dev/sda" ] +} +``` + +Entries in that array can refer to block devices, LVM2 volume groups +(/dev/vgroup-name/), and Stratis pools (/dev/stratis/pool-name/). + +Mount point prefix +------------------ + +Cockpit can be put into a kind of "chroot" environment by giving it a +mount point prefix like so: + +``` +{ + "mount_point_prefix": "/sysroot" +} +``` + +This works at the UI level: filesystems that have mount points outside +of "/sysroot" are hidden from the user, and when letting the user work +with mount points below "/sysroot", the "/sysroot" prefix is +omitted. So when the user says to create a filesystem on "/var", they +are actually creating one on "/sysroot/var". + +However, Cockpit (via UDisks2) will still write the new mount point +configuration into the real /etc/fstab (_not_ /sysroot/etc/fstab). + +In addition to that, Cockpit will also store the mount points in the +`"cockpit_mount_points"` item in `window.localStorage`, as a JSON +encoded object, for the benefit of Anaconda. + +This is a simple map from mount point to block device, like + +``` +{ + "/boot": "/dev/vda1", + "/": "/dev/vda2" +} +``` + +The mount points do not include the mount point prefix. diff --git a/pkg/storaged/anaconda.jsx b/pkg/storaged/anaconda.jsx new file mode 100644 index 00000000000..12ee15ee47c --- /dev/null +++ b/pkg/storaged/anaconda.jsx @@ -0,0 +1,35 @@ +/* + * This file is part of Cockpit. + * + * Copyright (C) 2023 Red Hat, Inc. + * + * Cockpit is free software; you can redistribute it and/or modify it + * under the terms of the GNU Lesser General Public License as published by + * the Free Software Foundation; either version 2.1 of the License, or + * (at your option) any later version. + * + * Cockpit is distributed in the hope that it will be useful, but + * WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + * Lesser General Public License for more details. + * + * You should have received a copy of the GNU Lesser General Public License + * along with Cockpit; If not, see . + */ + +import cockpit from "cockpit"; +import React from "react"; +import { Alert, AlertActionLink } from "@patternfly/react-core/dist/esm/components/Alert/index.js"; +import { Page, PageSection } from "@patternfly/react-core/dist/esm/components/Page/index.js"; + +const _ = cockpit.gettext; + +export const AnacondaAdvice = ({ client }) => { + return ( + +

Anaconda will tell us here what is wrong with the current config.

+ + ); +}; diff --git a/pkg/storaged/client.js b/pkg/storaged/client.js index 6dd9f43ec9c..85a5e14c7db 100644 --- a/pkg/storaged/client.js +++ b/pkg/storaged/client.js @@ -591,6 +591,7 @@ client.update = (first_time) => { update_indices(); client.path_warnings = find_warnings(client); create_pages(); + client.export_mount_point_mapping(); client.dispatchEvent("changed"); } }; @@ -721,6 +722,15 @@ function init_model(callback) { ).then(() => info); } + try { + client.anaconda = JSON.parse(window.localStorage.getItem("cockpit_anaconda")); + } catch { + console.warn("Can't parse cockpit_anaconda configuration as JSON"); + client.anaconda = null; + } + + console.log("ANACONDA", client.anaconda); + pull_time().then(() => { read_os_release().then(os_release => { client.os_release = os_release; @@ -1464,4 +1474,56 @@ client.get_config = (name, def) => { } }; +client.in_anaconda_mode = () => !!client.anaconda; + +client.strip_mount_point_prefix = (dir) => { + const mpp = client.anaconda?.mount_point_prefix; + + if (dir && mpp) { + if (dir.indexOf(mpp) != 0) + return false; + + dir = dir.substr(mpp.length); + if (dir == "") + dir = "/"; + } + + return dir; +}; + +client.add_mount_point_prefix = (dir) => { + const mpp = client.anaconda?.mount_point_prefix; + if (mpp) { + if (dir == "/") + dir = mpp; + else + dir = mpp + dir; + } + return dir; +}; + +client.should_ignore_device = (devname) => { + return client.anaconda?.ignore_devices && client.anaconda.ignore_devices.indexOf(devname) != -1; +}; + +client.should_ignore_block = (block) => { + return client.should_ignore_device(utils.decode_filename(block.PreferredDevice)); +}; + +client.export_mount_point_mapping = () => { + const mpm = { }; + for (const p in client.blocks) { + const b = client.blocks[p]; + for (const c of b.Configuration) { + if (c[0] == "fstab") { + const dir = client.strip_mount_point_prefix(utils.decode_filename(c[1].dir.v)); + if (dir) + mpm[dir] = utils.decode_filename(b.PreferredDevice); + } + } + } + + window.localStorage.setItem("cockpit_mount_points", JSON.stringify(mpm)); +}; + export default client; diff --git a/pkg/storaged/content-views.jsx b/pkg/storaged/content-views.jsx index aea8e65f0ac..8c94a3f12dd 100644 --- a/pkg/storaged/content-views.jsx +++ b/pkg/storaged/content-views.jsx @@ -509,7 +509,7 @@ function block_description(client, block, options) { type = cockpit.format(C_("storage-id-desc", "$0 filesystem"), block.IdType); if (client.fsys_sizes.data[mount_point]) size = client.fsys_sizes.data[mount_point]; - used_for = mount_point; + used_for = client.strip_mount_point_prefix(mount_point); } else if (block.IdUsage == "raid") { if (block_pvol && client.vgroups[block_pvol.VolumeGroup]) { const vgroup = client.vgroups[block_pvol.VolumeGroup]; @@ -601,6 +601,18 @@ function append_row(client, rows, level, key, name, desc, tabs, job_object, opti if (info) info = <>{"\n"}{info}; + let location; + if (desc.used_for === false) { + // XXX - urks + location = _("(Not part of target)"); + menu = null; + tabs.actions = null; + tabs.renderers = []; + } else if (desc.link) + location = ; + else + location = desc.used_for; + const cols = [ { title: ( @@ -610,7 +622,7 @@ function append_row(client, rows, level, key, name, desc, tabs, job_object, opti ) }, { title: desc.type }, - { title: desc.link ? : desc.used_for }, + { title: location }, { title: desc.size.length ? diff --git a/pkg/storaged/details.jsx b/pkg/storaged/details.jsx index a9723443afd..fef07e61352 100644 --- a/pkg/storaged/details.jsx +++ b/pkg/storaged/details.jsx @@ -35,12 +35,21 @@ import { VDODetails } from "./vdo-details.jsx"; import { NFSDetails } from "./nfs-details.jsx"; import { StratisPoolDetails, StratisStoppedPoolDetails } from "./stratis-details.jsx"; import { JobsPanel } from "./jobs-panel.jsx"; +import { AnacondaAdvice } from "./anaconda.jsx"; const _ = cockpit.gettext; export const StdDetailsLayout = ({ client, alerts, header, content, sidebar }) => { const top = <> { (alerts || []).filter(a => !!a).map((a, i) => {a}) } + { client.in_anaconda_mode() + ? + + + + + : null + } { header } diff --git a/pkg/storaged/dialog.jsx b/pkg/storaged/dialog.jsx index ec97684d52b..5e5ffdeb579 100644 --- a/pkg/storaged/dialog.jsx +++ b/pkg/storaged/dialog.jsx @@ -1081,7 +1081,8 @@ export const BlockingMessage = (usage) => { pvol: _("physical volume of LVM2 volume group"), "mdraid-member": _("member of RAID device"), vdo: _("backing device for VDO device"), - "stratis-pool-member": _("member of Stratis pool") + "stratis-pool-member": _("member of Stratis pool"), + mounted: _("Filesystem outside the target"), }; console.log("U", usage); @@ -1173,9 +1174,15 @@ export const TeardownMessage = (usage, expect_single_unmount) => { const name = (fsys ? fsys.Devnode : block_name(client.blocks[use.block.CryptoBackingDevice] || use.block)); + let location = use.location; + if (use.usage == "mounted") { + location = client.strip_mount_point_prefix(location); + if (location === false) + location = _("(Not part of target)"); + } rows.push({ columns: [name, - use.location || "-", + location || "-", use.actions.length ? use.actions.join(", ") : "-", { title: , diff --git a/pkg/storaged/drives-panel.jsx b/pkg/storaged/drives-panel.jsx index d60a887a739..a529c211614 100644 --- a/pkg/storaged/drives-panel.jsx +++ b/pkg/storaged/drives-panel.jsx @@ -105,7 +105,8 @@ export class DrivesPanel extends React.Component { const drives = drive_rows(client); return ( - { - // "nofail" is the default for new filesystems with Cockpit so - // that a failure to mount one of them will not prevent - // Cockpit from starting. This allows people to debug and fix - // these failures with Cockpit itself. - // + // "nofail" is the default for new filesystems with Cockpit so + // that a failure to mount one of them will not prevent + // Cockpit from starting. This allows people to debug and fix + // these failures with Cockpit itself. + // + // In Anaconda mode however, we don't make "nofail" the + // default since people will be creating the core filesystems + // like "/", "/var", etc. + + if (!client.in_anaconda_mode()) options.nofail = true; + utils.get_parent_blocks(client, block.path).forEach(p => { if (utils.is_netdev(client, p)) { options._netdev = true; } @@ -137,10 +142,10 @@ export function format_dialog(client, path, start, size, enable_dos_extended) { return false; }) .then(version => { - format_dialog_internal(client, path, start, size, enable_dos_extended, version); + return format_dialog_internal(client, path, start, size, enable_dos_extended, version); }); } else { - format_dialog_internal(client, path, start, size, enable_dos_extended); + return format_dialog_internal(client, path, start, size, enable_dos_extended); } } @@ -237,6 +242,10 @@ function format_dialog_internal(client, path, start, size, enable_dos_extended, if (old_opts == undefined) old_opts = initial_mount_options(client, block); + old_dir = client.strip_mount_point_prefix(old_dir); + if (old_dir === false) + return Promise.reject(_("This device can not be used for the installation target.")); + const split_options = parse_options(old_opts); extract_option(split_options, "noauto"); const opt_ro = extract_option(split_options, "ro"); @@ -274,7 +283,7 @@ function format_dialog_internal(client, path, start, size, enable_dos_extended, visible: is_filesystem, value: old_dir || "", validate: (val, values, variant) => { - return is_valid_mount_point(client, block, val, variant == "nomount"); + return is_valid_mount_point(client, block, client.add_mount_point_prefix(val), variant == "nomount"); } }), SelectOne("type", _("Type"), @@ -468,6 +477,7 @@ function format_dialog_internal(client, path, start, size, enable_dos_extended, if (mount_point != "") { if (mount_point[0] != "/") mount_point = "/" + mount_point; + mount_point = client.add_mount_point_prefix(mount_point); config_items.push(["fstab", { dir: { t: 'ay', v: utils.encode_filename(mount_point) }, diff --git a/pkg/storaged/fsys-panel.jsx b/pkg/storaged/fsys-panel.jsx index fad062456c6..191ff902b69 100644 --- a/pkg/storaged/fsys-panel.jsx +++ b/pkg/storaged/fsys-panel.jsx @@ -49,6 +49,9 @@ export class FilesystemsPanel extends React.Component { function is_mount(path) { const block = client.blocks[path]; + if (client.should_ignore_block(block)) + return false; + // Stratis filesystems are handled separately if (client.blocks_stratis_fsys[path]) return false; @@ -72,7 +75,7 @@ export class FilesystemsPanel extends React.Component { function make_mount(path) { const block = client.blocks[path]; - const [, mount_point] = get_fstab_config(block, true); + let [, mount_point] = get_fstab_config(block, true); const fsys_size = client.fsys_sizes.data[mount_point]; const backing_block = client.blocks[block.CryptoBackingDevice] || block; const block_lvm2 = client.blocks_lvm2[backing_block.path]; @@ -80,6 +83,10 @@ export class FilesystemsPanel extends React.Component { const vgroup = lvol && client.vgroups[lvol.VolumeGroup]; let name = null; + mount_point = client.strip_mount_point_prefix(mount_point); + if (mount_point === false) + return null; + if (vgroup) name = vgroup.Name + "/" + lvol.Name; @@ -106,7 +113,8 @@ export class FilesystemsPanel extends React.Component { } const mounts = Object.keys(client.blocks).filter(is_mount) - .map(make_mount); + .map(make_mount) + .filter(m => m != null); function has_filesystems(path) { return client.stratis_pool_filesystems[path].length > 0; @@ -132,8 +140,11 @@ export class FilesystemsPanel extends React.Component { let mount = "-"; if (block) { const [, mp] = get_fstab_config(block, true); - if (mp) - mount = mp; + if (mp) { + mount = client.strip_mount_point_prefix(mp); + if (mount === false) + return null; + } } return { props: { path, client, key: fs.path }, @@ -152,7 +163,7 @@ export class FilesystemsPanel extends React.Component { } ] }; - }); + }).filter(m => m != null); } const pools = Object.keys(client.stratis_pools).filter(has_filesystems) @@ -177,6 +188,7 @@ export class FilesystemsPanel extends React.Component { sortBy={{ index: 0, direction: SortByDirection.asc }} aria-label={_("Filesystems")} onRowClick={onRowClick} + emptyCaption={_("No filesystems")} columns={[ { title: _("Source"), sortable: true }, { title: _("Type"), sortable: true }, diff --git a/pkg/storaged/fsys-tab.jsx b/pkg/storaged/fsys-tab.jsx index a6465500b48..05fbac8e482 100644 --- a/pkg/storaged/fsys-tab.jsx +++ b/pkg/storaged/fsys-tab.jsx @@ -180,6 +180,10 @@ export function mounting_dialog(client, block, mode, forced_options) { const [old_config, old_dir, old_opts, old_parents] = get_fstab_config(block, true); const options = old_config ? old_opts : initial_tab_options(client, block, true); + const old_dir_for_display = client.strip_mount_point_prefix(old_dir); + if (old_dir_for_display === false) + return Promise.reject(_("This device can not be used for the installation target.")); + const split_options = parse_options(options); extract_option(split_options, "noauto"); const opt_never_auto = extract_option(split_options, "x-cockpit-never-auto"); @@ -331,8 +335,8 @@ export function mounting_dialog(client, block, mode, forced_options) { fields = [ TextInput("mount_point", _("Mount point"), { - value: old_dir, - validate: val => is_valid_mount_point(client, block, val, mode == "update" && !is_filesystem_mounted, true) + value: old_dir_for_display, + validate: val => is_valid_mount_point(client, block, client.add_mount_point_prefix(val), mode == "update" && !is_filesystem_mounted, true) }), CheckBoxes("mount_options", _("Mount options"), { @@ -453,7 +457,8 @@ export function mounting_dialog(client, block, mode, forced_options) { opts = opts.concat(forced_options); if (vals.mount_options.extra !== false) opts = opts.concat(parse_options(vals.mount_options.extra)); - return (maybe_update_config(vals.mount_point, unparse_options(opts), + return (maybe_update_config(client.add_mount_point_prefix(vals.mount_point), + unparse_options(opts), vals.passphrase, passphrase_type) .then(() => maybe_set_crypto_options(vals.mount_options.ro, opts.indexOf("noauto") == -1, @@ -534,6 +539,7 @@ export class FilesystemTab extends React.Component { let mount_point_text = null; if (old_dir) { + mount_point_text = client.strip_mount_point_prefix(old_dir); let opt_texts = []; if (opt_ro) opt_texts.push(_("read only")); @@ -545,11 +551,13 @@ export class FilesystemTab extends React.Component { opt_texts.push(_("ignore failure")); else opt_texts.push(_("stop boot on failure")); + if (mount_point_text === false) { + mount_point_text = ""; + opt_texts.push(_("not part of target")); + } opt_texts = opt_texts.concat(split_options); if (opt_texts.length) { - mount_point_text = cockpit.format("$0 ($1)", old_dir, opt_texts.join(", ")); - } else { - mount_point_text = old_dir; + mount_point_text = cockpit.format("$0 ($1)", mount_point_text, opt_texts.join(", ")); } } diff --git a/pkg/storaged/pages.jsx b/pkg/storaged/pages.jsx index 4f4f2f8fbbb..5113aa4dd54 100644 --- a/pkg/storaged/pages.jsx +++ b/pkg/storaged/pages.jsx @@ -19,6 +19,7 @@ import cockpit from "cockpit"; import React from "react"; +import client from "./client.js"; import { CardBody } from "@patternfly/react-core/dist/esm/components/Card/index.js"; import { StackItem } from "@patternfly/react-core/dist/esm/layouts/Stack/index.js"; @@ -298,6 +299,8 @@ const PageTable = ({ emptyCaption, aria_label, pages, crossrefs }) => { function make_page_rows(pages, level) { for (const p of pages) { + if (client.should_ignore_device(p.name) || client.should_ignore_device(p.columns[1])) + continue; rows.push(make_row(p, null, level, rows.length)); make_page_rows(p.children, level + 1); } diff --git a/pkg/storaged/pages/filesystem.jsx b/pkg/storaged/pages/filesystem.jsx index 23db5733999..a6a39587063 100644 --- a/pkg/storaged/pages/filesystem.jsx +++ b/pkg/storaged/pages/filesystem.jsx @@ -128,10 +128,14 @@ export function make_filesystem_page(parent, backing_block, content_block, fstab const mounted = content_block && is_mounted(client, content_block); let mp_text; + const short_mp = client.strip_mount_point_prefix(mount_point); + if (!short_mp) + return; + if (mount_point && mounted) - mp_text = mount_point; + mp_text = short_mp; else if (mount_point && !mounted) - mp_text = mount_point + " " + _("(not mounted)"); + mp_text = short_mp + " " + _("(not mounted)"); else mp_text = _("(not mounted)"); @@ -185,7 +189,8 @@ export const MountPoint = ({ fstab_config, forced_options, backing_block, conten opt_texts.push(_("stop boot on failure")); opt_texts = opt_texts.concat(split_options); if (opt_texts.length) { - mount_point_text = cockpit.format("$0 ($1)", old_dir, opt_texts.join(", ")); + mount_point_text = cockpit.format("$0 ($1)", client.strip_mount_point_prefix(old_dir), + opt_texts.join(", ")); } else { mount_point_text = old_dir; } diff --git a/pkg/storaged/pages/overview.jsx b/pkg/storaged/pages/overview.jsx index 0ba6ab074eb..8b5d6399d9e 100644 --- a/pkg/storaged/pages/overview.jsx +++ b/pkg/storaged/pages/overview.jsx @@ -148,18 +148,22 @@ const OverviewPage = ({ page, plot_state }) => { return ( - - - - - - - + { !client.in_anaconda_mode() && + + + + + + + + } - - - + { !client.in_anaconda_mode() && + + + + } ); }; diff --git a/pkg/storaged/side-panel.jsx b/pkg/storaged/side-panel.jsx index 1bdb1684fc8..602ce1f731a 100644 --- a/pkg/storaged/side-panel.jsx +++ b/pkg/storaged/side-panel.jsx @@ -41,7 +41,8 @@ export class SidePanel extends React.Component { render() { let show_all_button = null; - let rows = this.props.rows.filter(row => !!row); + const client = this.props.client; + let rows = this.props.rows.filter(row => !!row && !(client && client.should_ignore_device(row.devname))); // Find new items for animations const current_keys = rows.map(row => row.key); @@ -83,7 +84,7 @@ export class SidePanel extends React.Component { feature={this.props.feature} not_installed_text={this.props.not_installed_text} install_title={this.props.install_title}> - { this.props.rows.length > 0 + { rows.length > 0 ? { children } diff --git a/pkg/storaged/storage-page.jsx b/pkg/storaged/storage-page.jsx index 79ad1709880..04aeec80fda 100644 --- a/pkg/storaged/storage-page.jsx +++ b/pkg/storaged/storage-page.jsx @@ -28,12 +28,11 @@ import { Page, PageBreadcrumb, PageSection } from "@patternfly/react-core/dist/e import { Breadcrumb, BreadcrumbItem } from "@patternfly/react-core/dist/esm/components/Breadcrumb/index.js"; import { MultipathAlert } from "./multipath.jsx"; +import { AnacondaAdvice } from "./anaconda.jsx"; export const StoragePage = ({ location, plot_state }) => { const page = get_page_from_location(location); - // XXX - global alerts here, Multipath, Anaconda - const parent_crumbs = []; let pp = page.parent; while (pp) { @@ -56,6 +55,7 @@ export const StoragePage = ({ location, plot_state }) => { + diff --git a/pkg/storaged/stratis-details.jsx b/pkg/storaged/stratis-details.jsx index 2fde5b647dc..bab30755c1a 100644 --- a/pkg/storaged/stratis-details.jsx +++ b/pkg/storaged/stratis-details.jsx @@ -123,6 +123,7 @@ function set_mount_options(client, path, vals, forced_options) { return Promise.resolve(); if (mount_point[0] != "/") mount_point = "/" + mount_point; + mount_point = client.add_mount_point_prefix(mount_point); const config = ["fstab", @@ -280,7 +281,7 @@ export function stratis_content_rows(client, pool, options) { }; } - const [, mount_point] = get_fstab_config(block); + let [, mount_point] = get_fstab_config(block); const fs_is_mounted = is_mounted(client, block); function mount() { @@ -347,7 +348,7 @@ export function stratis_content_rows(client, pool, options) { }), SelectOne("at_boot", _("At boot"), { - value: "nofail", + value: client.in_anaconda_mode() ? "local" : "nofail", explanation: mount_explanation.nofail, choices: [ { @@ -430,7 +431,7 @@ export function stratis_content_rows(client, pool, options) { if (info) info = <>{"\n"}{info}; - const tabs = [ + let tabs = [ { name, renderer: FilesystemTab, @@ -443,7 +444,7 @@ export function stratis_content_rows(client, pool, options) { } ]; - const actions = []; + let actions = []; const menuitems = []; if (!fs_is_mounted) { @@ -457,6 +458,13 @@ export function stratis_content_rows(client, pool, options) { menuitems.push({_("Snapshot")}); menuitems.push({_("Delete")}); + mount_point = client.strip_mount_point_prefix(mount_point); + if (mount_point === false) { + mount_point = _("(Not part of target)"); + actions = null; + tabs = null; + } + const cols = [ { title: ( @@ -486,7 +494,7 @@ export function stratis_content_rows(client, pool, options) { return { props: { key: fsys.Name }, columns: cols, - expandedContent: + expandedContent: tabs ? : null }; } @@ -516,7 +524,7 @@ function create_fs(client, pool) { TextInput("mount_point", _("Mount point"), { validate: (val, values, variant) => { - return is_valid_mount_point(client, null, val, variant == "nomount"); + return is_valid_mount_point(client, null, client.add_mount_point_prefix(val), variant == "nomount"); } }), CheckBoxes("mount_options", _("Mount options"), diff --git a/pkg/storaged/utils.js b/pkg/storaged/utils.js index b971ef31333..465ab601782 100644 --- a/pkg/storaged/utils.js +++ b/pkg/storaged/utils.js @@ -498,7 +498,9 @@ export function get_available_spaces(client) { !is_vdo_backing_dev() && !is_swap() && !block_ptable && - !(block_part && block_part.IsContainer)); + !(block_part && block_part.IsContainer) && + !is_snap(client, block) && + !client.should_ignore_device(decode_filename(block.PreferredDevice))); } function make(path) { @@ -570,7 +572,8 @@ export function get_other_devices(client) { block.Size > 0 && !client.legacy_vdo_overlay.find_by_block(block) && !client.blocks_stratis_fsys[block.path] && - !is_snap(client, block)); + !is_snap(client, block) && + !client.should_ignore_block(block)); }); } @@ -807,7 +810,7 @@ export function get_active_usage(client, path, top_action, child_action) { location, set_noauto: !is_top, actions: is_top ? get_actions(_("unmount")) : [_("unmount")], - blocking: false + blocking: client.strip_mount_point_prefix(location) === false, }); }