Skip to content

Conversation

@Jkintalix
Copy link
Contributor

@Jkintalix Jkintalix commented Nov 18, 2025

Graphical installer (Calamares) now uses a unified iso9660/vfat detection function based on blkid to identify ISO and FAT-based bootable USB media. These devices are excluded from the list of installation targets.

Terminal installer (diskutils.go) has been updated to extend read-only media detection using /proc/mounts, covering iso9660 and FAT (vfat/fat32) filesystems, including Rufus-created USB drives.

This ensures consistent filtering across both installers and prevents the UEFI bootable USB drive from appearing as a valid installation destination.

Merge Checklist

All boxes should be checked before merging the PR

  • The changes in the PR have been built and tested
  • [] cgmanifest file has been updated if required
  • Ready to merge

Description

ITEP-81482: During EMT ISO installation using a bootable USB drive, the installer was incorrectly listing the UEFI USB device (containing the ISO image) as a valid installation target in both Terminal and Graphical installer modes, which could lead to data loss or incomplete installation if the user selected the USB drive as the destination. This patch introduces unified filtering logic to prevent such scenarios by detecting and excluding all installation media types, including iso9660 filesystems from DD-flashed ISOs or CD/DVDs, FAT/VFAT/FAT32 USB devices created using tools like Rufus, and read-only USB partitions that contain “CDROM” signatures. With these improvements, both Calamares and the terminal installer consistently filter out bootable USB/ISO media, ensuring they no longer appear in the list of valid installation targets.

Any Newly Introduced Dependencies

NO

How Has This Been Tested?

Manually tested

@Jkintalix Jkintalix changed the title partition: extend device filtering across graphical &terminal installations Partition extend device filtering across graphical &terminal installations Nov 18, 2025
@Jkintalix Jkintalix marked this pull request as ready for review November 18, 2025 06:40
@Jkintalix Jkintalix requested a review from a team as a code owner November 18, 2025 06:40
@Jkintalix Jkintalix force-pushed the fix/filter-usb-iso-vfat-media branch 2 times, most recently from 31e4226 to 10c46a4 Compare November 18, 2025 06:49
@Jkintalix Jkintalix force-pushed the fix/filter-usb-iso-vfat-media branch from 10c46a4 to c893df0 Compare November 18, 2025 10:16
@Jkintalix Jkintalix self-assigned this Nov 18, 2025
…allers

Graphical installer (Calamares) now uses a unified iso9660/vfat detection
function based on blkid to identify ISO and FAT-based bootable USB media.
These devices are excluded from the list of installation targets.

Terminal installer (diskutils.go) has been updated to extend read-only
media detection using /proc/mounts, covering iso9660 and FAT (vfat/fat32)
filesystems, including Rufus-created USB drives.

This ensures consistent filtering across both installers and prevents the
UEFI bootable USB drive from appearing as a valid installation destination.

Signed-off-by: kinatli jayanth <jayanthx.kintali@intel.com>
@Jkintalix Jkintalix force-pushed the fix/filter-usb-iso-vfat-media branch from c893df0 to 2fe1242 Compare November 20, 2025 04:39
@cheeyanglee cheeyanglee merged commit 33dcbec into open-edge-platform:3.0-dev Nov 20, 2025
16 of 19 checks passed
liulis-sg pushed a commit that referenced this pull request Nov 25, 2025
…allers (#586)

Graphical installer (Calamares) now uses a unified iso9660/vfat detection
function based on blkid to identify ISO and FAT-based bootable USB media.
These devices are excluded from the list of installation targets.

Terminal installer (diskutils.go) has been updated to extend read-only
media detection using /proc/mounts, covering iso9660 and FAT (vfat/fat32)
filesystems, including Rufus-created USB drives.

This ensures consistent filtering across both installers and prevents the
UEFI bootable USB drive from appearing as a valid installation destination.

Signed-off-by: kinatli jayanth <jayanthx.kintali@intel.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants