Bug 43793 - Пустая переменная окружения CDROMDEV вызывает некорректную работа скрипта 90-pkg.sh из installer-feature-repo-add
Summary: Пустая переменная окружения CDROMDEV вызывает некорректную работа скрипта 90-...
Status: CLOSED FIXED
Alias: None
Product: Branch p10
Classification: Unclassified
Component: installer (show other bugs)
Version: не указана
Hardware: x86_64 Linux
: P5 normal
Assignee: Антон Мидюков
QA Contact: qa-p10@altlinux.org
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2022-09-14 19:37 MSK by jqt4
Modified: 2022-10-02 17:58 MSK (History)
1 user (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description jqt4 2022-09-14 19:37:27 MSK 
Устанавливаю тестовый ISO образ alt-server на Intel Nuc c SSD, подключенного по USB.
В установщике раздел SSD с ISO образом выглядит как /dev/sda1
На этапе postinstall получаю сбой в скрипте 90-pkg.sh и некорректную запись в fstab вида:

UUID=2d85209b-275a-4456-b0f4-98ac1acb160b
a9ffa983-b014-47fb-bd8d-45958e512162
2022-09-14-10-56-46-00
993F-0FCE       /media/ALTLinux udf,iso9660 ro,noauto,user,utf8,nofail,comment=x-gvfs-show  0 0

Эта запись формируется в скрипте 90-pkg.sh
https://git.altlinux.org/gears/i/installer-feature-repo-add.git?p=installer-feature-repo-add.git;a=blob;f=installer-feature-repo-add/postinstall.d/90-pkg.sh;h=6eae9ec38312aa1348132993987aa317280a1bb8;hb=5202b54c845337f4fd89ac796e2f9bc4f07b9f2e#l36
https://git.altlinux.org/gears/i/installer-feature-repo-add.git?p=installer-feature-repo-add.git;a=blob;f=installer-feature-repo-add/postinstall.d/90-pkg.sh;h=6eae9ec38312aa1348132993987aa317280a1bb8;hb=5202b54c845337f4fd89ac796e2f9bc4f07b9f2e#l42
с использованием переменных окружения $DEVICE$CDROMDEV

Обе эти переменные содержат пустые строки.

Предполагаю, что строка $CDROMDEV делается пустой в результате исправления в скрипте installer/scripts/install2 из пакета installer
https://git.altlinux.org/gears/i/installer.git?p=installer.git;a=commitdiff;h=8f25c934e344748878d42d4c9781b98e066ab79f
Comment 1 Антон Мидюков 2022-09-15 00:13:52 MSK 
Такая же проблема будет при выборе method=disk, но немного другая. Обе переменные назначены:
CDROMDEV="UUID=$(blkid -o value -s UUID $DEVICE$CDROMDEV)"
при загрузке с флэшки:
CDROMDEV="UUID=$(blkid -o value -s UUID sdb1/dev/sdb1)"

Но это не регресс. Раньше такая же дребедень была.
А значит и раньше в legacy с syslinux + gfxboot при загрузке с флешки была проблема с неправильной записью в /etc/fstab.

Обе проблемы должны быть исправлены заданием:
#306814 FAILED #1 [test-only] p10 installer.git=1.12.4-alt1
Comment 2 Антон Мидюков 2022-09-30 12:24:29 MSK 
[#306814] p10 DONE (try 3) installer.git=1.12.4-alt1

Thu Sep 15 2022 Anton Midyukov <antohami at altlinux> 1.12.4-alt1
 - install2: add set_device for define $DEVICE variable (Closes: 43793)
 - install2: assign $CDROMDEV variable for /dev/cdrom*|/dev/dvd* also