[devel-distro] remount и luks | Re: [cyber] I: Sisyphus-20200417 packages: +8! -6 +60 (17569)

[devel-distro] remount и luks | Re: [cyber] I: Sisyphus-20200417 packages: +8! -6 +60 (17569)

[Michael Shigorin]

On Fri, Apr 17, 2020 at 04:42:38AM +0000, QA Team Robot wrote:
> installer-scripts-remount-stage2 - Shared installer scripts: remount
> * Wed Apr 15 2020 Oleg Solovyov <mcpain@altlinux> 0.5.17-alt1
> - start_luks: Remount only empty password containers

А вот в этом не уверен -- помнится, там выставлялся временный
пароль (один на все LUKS-контейнеры), который _впоследствии_
заменялся на указанный пользователем.

Проверь, пожалуйста, что ты не сломал то, что работало.
Возможно, сравни с http://nightly.altlinux.org/p8/release/

> * Mon Feb 03 2020 Oleg Solovyov <mcpain@altlinux> 0.5.16-alt1

-- 
 ---- WBR, Michael Shigorin / http://altlinux.org
  ------ http://opennet.ru / http://anna-news.info

Re: [devel-distro] remount и luks | Re: [cyber] I: Sisyphus-20200417 packages: +8! -6 +60 (17569)

[mcpain]

[-- Attachment #1: Type: text/plain, Size: 767 bytes --]

В письме от пятница, 17 апреля 2020 г. 12:21:17 MSK пользователь Michael 
Shigorin написал:
> On Fri, Apr 17, 2020 at 04:42:38AM +0000, QA Team Robot wrote:
> > installer-scripts-remount-stage2 - Shared installer scripts: remount
> > * Wed Apr 15 2020 Oleg Solovyov <mcpain@altlinux> 0.5.17-alt1
> > - start_luks: Remount only empty password containers
> 
> А вот в этом не уверен -- помнится, там выставлялся временный
> пароль (один на все LUKS-контейнеры), который _впоследствии_
> заменялся на указанный пользователем.

Разве?
Где? Не увидел ничего такого в коде.

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

Re: [devel-distro] remount и luks | Re: [cyber] I: Sisyphus-20200417 packages: +8! -6 +60 (17569)

[Michael Shigorin]

On Fri, Apr 17, 2020 at 12:29:22PM +0300, mcpain wrote:
> > > installer-scripts-remount-stage2 - Shared installer scripts: remount
> > > * Wed Apr 15 2020 Oleg Solovyov <mcpain@altlinux> 0.5.17-alt1
> > > - start_luks: Remount only empty password containers
> > А вот в этом не уверен -- помнится, там выставлялся временный
> > пароль (один на все LUKS-контейнеры), который _впоследствии_
> > заменялся на указанный пользователем.
> Разве? Где? Не увидел ничего такого в коде.

Мне так припоминается (хотя сейчас смотрю и пытаюсь вспомнить,
при чём тут $CRYPTSETUP_KEY и не была ли это как раз попытка
работать с ещё пустым на этой стадии паролем).  Просто уж очень
давно с этими кусками ковырялся (как бы не во времена p7),
а сам luks не применяю.

Кандидаты -- installer, alterator-vm, guile-evms;
ну или grep -ri luks /usr в запущенном инсталере
и что найдётся, то уже по contents_index сверять.
Сам installer-scripts-remount-stage2 начинался
как ещё один скрипт в пакете installer.

В общем, коммит 4161ab1fed65fbd33d7f8159466e6b1657ca2e8d
явно стоило прокомментировать тщательней -- *зачем* это
делается, *какая* проблема решается.  Не стоит полагаться
на свою память, через год код может стать незнакомым...
да и другим лучше пояснить сразу, чем потом.

-- 
 ---- WBR, Michael Shigorin / http://altlinux.org
  ------ http://opennet.ru / http://anna-news.info

Re: [devel-distro] remount и luks | Re: [cyber] I: Sisyphus-20200417 packages: +8! -6 +60 (17569)

[Oleg Solovyov]

В письме от пятница, 17 апреля 2020 г. 13:05:19 MSK пользователь Michael 
Shigorin написал:
> On Fri, Apr 17, 2020 at 12:29:22PM +0300, mcpain wrote:
> > > > installer-scripts-remount-stage2 - Shared installer scripts: remount
> > > > * Wed Apr 15 2020 Oleg Solovyov <mcpain@altlinux> 0.5.17-alt1
> > > > - start_luks: Remount only empty password containers
> > > 
> > > А вот в этом не уверен -- помнится, там выставлялся временный
> > > пароль (один на все LUKS-контейнеры), который _впоследствии_
> > > заменялся на указанный пользователем.
> > 
> > Разве? Где? Не увидел ничего такого в коде.
> 
> Мне так припоминается (хотя сейчас смотрю и пытаюсь вспомнить,
> при чём тут $CRYPTSETUP_KEY и не была ли это как раз попытка
> работать с ещё пустым на этой стадии паролем).  Просто уж очень
> давно с этими кусками ковырялся (как бы не во времена p7),
> а сам luks не применяю.
> 
> Кандидаты -- installer, alterator-vm, guile-evms;
> ну или grep -ri luks /usr в запущенном инсталере
> и что найдётся, то уже по contents_index сверять.
> Сам installer-scripts-remount-stage2 начинался
> как ещё один скрипт в пакете installer.
> 
> В общем, коммит 4161ab1fed65fbd33d7f8159466e6b1657ca2e8d
> явно стоило прокомментировать тщательней -- *зачем* это
> делается, *какая* проблема решается.  Не стоит полагаться
> на свою память, через год код может стать незнакомым...
> да и другим лучше пояснить сразу, чем потом.

Раз сразу не получилось, то поясню сейчас:
Раньше при установке системы возникала ошибка, если после разметки 
дисков оставались LUKS-разделы от предыдущих установок: скрипт пытается 
открыть все контейнеры, а когда натыкается на контейнер с паролем, то 
валится с ошибкой.

Сейчас скрипт пытается открыть контейнер с пустым паролем и если у него 
не получается, то переходит к следующему.

Re: [devel-distro] remount и luks | Re: [cyber] I: Sisyphus-20200417 packages: +8! -6 +60 (17569)

[Michael Shigorin]

On Fri, Apr 17, 2020 at 01:43:03PM +0300, Oleg Solovyov wrote:
> > В общем, коммит 4161ab1fed65fbd33d7f8159466e6b1657ca2e8d
> > явно стоило прокомментировать тщательней -- *зачем* это
> > делается, *какая* проблема решается.  Не стоит полагаться
> > на свою память, через год код может стать незнакомым...
> > да и другим лучше пояснить сразу, чем потом.
> Раз сразу не получилось, то поясню сейчас:
> Раньше при установке системы возникала ошибка, если после
> разметки дисков оставались LUKS-разделы от предыдущих
> установок: скрипт пытается открыть все контейнеры, а когда
> натыкается на контейнер с паролем, то валится с ошибкой.
> Сейчас скрипт пытается открыть контейнер с пустым паролем
> и если у него не получается, то переходит к следующему.

Отлично, спасибо!  Разумеется, такой случай действительно
не был мной предусмотрен и не тестировался.

Только ещё раз призываю: коллеги, пишите это в коммитах.
Если кому-то неудобно -- пинайте, постараюсь перевести.
Но хоть тушкой, хоть чучелком -- пишите, пока в голове.

-- 
 ---- WBR, Michael Shigorin / http://altlinux.org
  ------ http://opennet.ru / http://anna-news.info