Bug 39855

Summary: [FR] Прошу добавить возможность задания UUID для ISO образа
Product: Sisyphus Reporter: jqt4
Component: mkimageAssignee: Alexey Gladkov <legion>
Status: CLOSED FIXED QA Contact: qa-sisyphus
Severity: normal    
Priority: P5 CC: antohami, glebfm, jqt4, legion, mike
Version: unstable   
Hardware: all   
OS: Linux   

Description jqt4 2021-03-30 20:13:26 MSK 
При загрузке ISO образа Альт, использующего propagator, могут быть обнаружены остатки ранее перезаписанных образов на носителях пользователя.
Если propagator ошибочно выберет для загрузки такой образ то произойдёт сбой.
Для обхода проблемы можно передать propagator идентификатор, позволяющий ему не ошибиться с выбором образа.
В качестве такого идентификатора предлагаю использовать UUID.
Подготовил патч для mkimage, реализующий эту возможность:

http://git.altlinux.org/people/jqt4/packages/?p=mkimage.git;a=commitdiff;h=4f142f1be4b9f982729bdff5f0419bd616e33e6f

Прошу рассмотреть.
Comment 1 Michael Shigorin 2021-03-30 21:08:40 MSK 
Посмотрите mkimage-profiles по слову rescue_hash (a2fcc601c98440c70603c6e2f6a93bd9260a8b88).
Comment 2 Антон Мидюков 2021-03-31 05:10:54 MSK 
(Ответ для Michael Shigorin на комментарий #1)
> Посмотрите mkimage-profiles по слову rescue_hash
> (a2fcc601c98440c70603c6e2f6a93bd9260a8b88).

Это вот вообще не то. Нам надо, чтобы образ был собран с заданным uuid, так как нельзя подправить конфигурационные файлы на уже собранном iso. А заранее угадать uuid мы, естественно, не можем. Только собрать с заданным uuid.
Comment 3 jqt4 2021-03-31 12:48:15 MSK 
Вот подписанный коммит:

http://git.altlinux.org/people/jqt4/packages/?p=mkimage.git;a=commitdiff;h=7217f92fbb1d33b7129c06914636333da69db89a

если это нужно.
Comment 4 Alexey Gladkov 2021-03-31 13:09:55 MSK 
+       uuid_opts="\
+       -volume_date uuid $UUID_ISO_SHRT \
+"

Результат будет uuid_opts="-volume_date uuid $UUID_ISO_SHRT "
Зачем вы делаете эти переносы строк ?

К чему вообще этот if если вы не делаете с этим параметром никаких манипуляций ?

можно же написать:

xorriso -follow param \
         -read_mkisofsrc \
          ${UUID_ISO_SHRT:+-volume_date uuid "$UUID_ISO_SHRT"} \
         -as mkisofs \
...
Comment 5 jqt4 2021-03-31 20:12:55 MSK 
Внёс исправления.
Вот исправленный коммит:
http://git.altlinux.org/people/jqt4/packages/?p=mkimage.git;a=commitdiff;h=1bee52a9c4f724d996fd28d9364fc56983f0c123
Comment 6 Repository Robot 2021-04-03 15:58:58 MSK 
mkimage-0.2.41-alt1 -> sisyphus:

 Sat Apr 03 2021 Alexey Gladkov <legion@altlinux.ru> 0.2.41-alt1
 - tools/mki-pack-efionly-isoboot, tools/mki-pack-grubpc-isoboot,
   tools/mki-pack-isoboot:
   + Set the UUID of the ISO image explicitly (ALT#39855)
Comment 7 Антон Мидюков 2021-04-03 16:49:25 MSK 
Спасибо!