Bug 39855 - [FR] Прошу добавить возможность задания UUID для ISO образа
Summary: [FR] Прошу добавить возможность задания UUID для ISO образа
Status: CLOSED FIXED
Alias: None
Product: Sisyphus
Classification: Development
Component: mkimage (show other bugs)
Version: unstable
Hardware: all Linux
: P5 normal
Assignee: Alexey Gladkov
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-03-30 20:13 MSK by jqt4
Modified: 2021-04-03 16:49 MSK (History)
5 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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
Спасибо!