Regression #14102
closedConsole menu incorrectly shows option ``99`` on some ARMv7/ARM64 installations
100%
Description
The console menu is intended to only show menu option 99 'Install to device' if pfSense is not running from eMMC or SATA.
However it's possible to incorrectly display it if the code creates an invalid device that then returns no results.
For example this 21.05.1 install:
[21.05.1-RELEASE][root@pfSense.home.arpa]/root: glabel status Name Status Components flash/uboot N/A flash/spi0s.uboot flash/ubootenv N/A flash/spi0s.ubootenv diskid/DISK-B1D52868 N/A mmcsd0 msdosfs/EFISYS N/A diskid/DISK-B1D52868s1
The code here runs that:
https://gitlab.netgate.com/pfSense/factory/-/blob/plus-master/src/etc/rc.initial#L125
And creates an invalid device to test against:
+ /sbin/geom label status + /usr/bin/grep diskid/DISK-B1D52868 + /usr/bin/awk '{ printf $3 }' + DEV=mmcsd0diskid/DISK-B1D52868s1
Hence it fails to find the eMMC device:
+ /sbin/geom disk list mmcsd0diskid/DISK-B1D52868s1 + /usr/bin/grep MMCHC + MMCDEV=''
Updated by Reid Linnemann almost 2 years ago
I've simplified and improved the EMMC/SATA rootdev check for aarch64 devices. The modified script is more specific about matching the root dev label in the label list only in the first output field.
Updated by Jim Pingle over 1 year ago
- Status changed from Feedback to New
Hate to be the bearer of bad news but I still see option 99 offered on my 1100. It's running ZFS, not UFS, so there is no diskid, and mount -p /
returns an error.
Updated by Jim Pingle over 1 year ago
- Status changed from New to Resolved
- Assignee changed from Reid Linnemann to Luiz Souza
- % Done changed from 0 to 100
After the latest Plus commit for this issue from Luiz (d470b17c454abb59f48846d361a260191655f1fe) it appears to be OK now. I no longer see option 99 on my 1100.
Updated by Jim Pingle over 1 year ago
- Subject changed from Console menu incorrectly shows option 99 to Console menu incorrectly shows option ``99`` on some ARMv7/ARM64 installations
Updating subject for release notes.