Bug 46724

Summary: Обновление инструкций quickstart
Product: Sisyphus Reporter: Toxblh <toxblh>
Component: mkimage-profilesAssignee: Антон Мидюков <antohami>
Status: CLOSED FIXED QA Contact: qa-sisyphus
Severity: enhancement    
Priority: P5 CC: antohami, mike, toxblh
Version: unstable   
Hardware: x86_64   
OS: Linux   
Attachments:
Description Flags
Патч для ревью
none
Исправленный патч none

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.