Bug 46724 - Обновление инструкций quickstart
Summary: Обновление инструкций quickstart
Status: CLOSED FIXED
Alias: None
Product: Sisyphus
Classification: Development
Component: mkimage-profiles (show other bugs)
Version: unstable
Hardware: x86_64 Linux
: P5 enhancement
Assignee: Антон Мидюков
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2023-07-01 20:59 MSK by Toxblh
Modified: 2023-07-05 05:44 MSK (History)
3 users (show)

See Also:


Attachments
Патч для ревью (1.30 KB, patch)
2023-07-01 20:59 MSK, Toxblh
no flags Details | Diff
Исправленный патч (867 bytes, patch)
2023-07-02 19:59 MSK, Toxblh
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Toxblh 2023-07-01 20:59:26 MSK
Created attachment 13659 [details]
Патч для ревью

Доброго времени суток. 
У меня тут patch на ревью для mkimage-profiles.

Я тут попробовал собрать свою сборку. Всё хорошо - но quickstart, можно улучшить :) чтобы реально был Quick. 
А то заставляет читать мануал к hasher, как настроить доступ к /proc, да и обновить группы можно через повторный вход, но можно его ускорить через `su USER`

Протестировал ещё с новым hasher - теперь надо руками стартовать службу ещё, wiki упоминается некая версия 2.0, хотя с 1.7 уже надо делать ровно так, даже "баг" есть https://bugzilla.altlinux.org/44218
Comment 1 Антон Мидюков 2023-07-02 15:25:11 MSK
1. Есть команда builder-useradd, который одной командой создаёт пользователя с настроенным hasher и mkimage-profiles.

2. Документация есть собранная в пакете и онлайн:
http://nightly.altlinux.org/docs/mkimage-profiles.html

3. Ну нет. Не думаю, что такое ускорение нужно в quickstart
+$ su USER

4. В hasher-priv сейчас добавляют автостарт службы, кстати, так что скоро будет не актуально уже.

Итого, патч отклоняю, но нужно:
- написать про builder-useradd
- добавить ссылку на онлайн-документацию

Спасибо за баг-репорт.
Comment 2 Антон Мидюков 2023-07-02 15:28:39 MSK
В актуальном Сизифе уже hasher-useradd сам включает hasher-privd.
Comment 3 Toxblh 2023-07-02 19:22:00 MSK
builder-useradd - прям полезно. 
Спасибо, не знал :) 

Про `su USER` скорее, как подсказка, как получить новые группы без logout/login цикла "(после повторного входа)". Я на виртуалке собирал всё на чистых системах, потому оказалось полезным такое, поставить всё, стянуть репу, su USER и пошёл собирать сразу.

Тут на ваше усмотрение.


Спасибо за ревью.
Comment 4 Toxblh 2023-07-02 19:59:44 MSK
Created attachment 13667 [details]
Исправленный патч
Comment 5 Toxblh 2023-07-02 20:03:16 MSK
> В актуальном Сизифе уже hasher-useradd сам включает hasher-privd.

Ага проверил, на чистой сборке - и правда, при добавлении пользователя сам включается.
Всё работает. Тогда всё даже упрощается. 


> $ git config --global user.email "my@email"
> $ git config --global user.name "My Name"
> $ mkdir -p ~/out ~/tmp

Если честно можно даже это убрать, так как builder-useradd добавляет заглушки от текущего пользователя. А ~/out будет добавлена автоматически при первом создании образа. Но не стоит.
Comment 6 Антон Мидюков 2023-07-05 05:44:39 MSK
Добавил ссылку на онлайн-документацию и примечание про builder-useradd.