* [devel-distro] remount и luks | Re: [cyber] I: Sisyphus-20200417 packages: +8! -6 +60 (17569)
@ 2020-04-17 9:21 ` Michael Shigorin
2020-04-17 9:29 ` mcpain
0 siblings, 1 reply; 5+ messages in thread
From: Michael Shigorin @ 2020-04-17 9:21 UTC (permalink / raw)
To: Oleg Solovyov; +Cc: devel-distro
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
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [devel-distro] remount и luks | Re: [cyber] I: Sisyphus-20200417 packages: +8! -6 +60 (17569)
2020-04-17 9:21 ` [devel-distro] remount и luks | Re: [cyber] I: Sisyphus-20200417 packages: +8! -6 +60 (17569) Michael Shigorin
@ 2020-04-17 9:29 ` mcpain
2020-04-17 10:05 ` Michael Shigorin
0 siblings, 1 reply; 5+ messages in thread
From: mcpain @ 2020-04-17 9:29 UTC (permalink / raw)
To: Oleg Solovyov, Michael Shigorin; +Cc: devel-distro
[-- 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 --]
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [devel-distro] remount и luks | Re: [cyber] I: Sisyphus-20200417 packages: +8! -6 +60 (17569)
2020-04-17 9:29 ` mcpain
@ 2020-04-17 10:05 ` Michael Shigorin
2020-04-17 10:43 ` Oleg Solovyov
0 siblings, 1 reply; 5+ messages in thread
From: Michael Shigorin @ 2020-04-17 10:05 UTC (permalink / raw)
To: Oleg Solovyov, devel-distro
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
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [devel-distro] remount и luks | Re: [cyber] I: Sisyphus-20200417 packages: +8! -6 +60 (17569)
2020-04-17 10:05 ` Michael Shigorin
@ 2020-04-17 10:43 ` Oleg Solovyov
2020-04-17 11:13 ` Michael Shigorin
0 siblings, 1 reply; 5+ messages in thread
From: Oleg Solovyov @ 2020-04-17 10:43 UTC (permalink / raw)
To: devel-distro, Michael Shigorin
В письме от пятница, 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-разделы от предыдущих установок: скрипт пытается
открыть все контейнеры, а когда натыкается на контейнер с паролем, то
валится с ошибкой.
Сейчас скрипт пытается открыть контейнер с пустым паролем и если у него
не получается, то переходит к следующему.
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [devel-distro] remount и luks | Re: [cyber] I: Sisyphus-20200417 packages: +8! -6 +60 (17569)
2020-04-17 10:43 ` Oleg Solovyov
@ 2020-04-17 11:13 ` Michael Shigorin
0 siblings, 0 replies; 5+ messages in thread
From: Michael Shigorin @ 2020-04-17 11:13 UTC (permalink / raw)
To: devel-distro
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
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2020-04-17 11:13 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-04-17 9:21 ` [devel-distro] remount и luks | Re: [cyber] I: Sisyphus-20200417 packages: +8! -6 +60 (17569) Michael Shigorin
2020-04-17 9:29 ` mcpain
2020-04-17 10:05 ` Michael Shigorin
2020-04-17 10:43 ` Oleg Solovyov
2020-04-17 11:13 ` Michael Shigorin
ALT Linux Distributions development
This inbox may be cloned and mirrored by anyone:
git clone --mirror http://lore.altlinux.org/devel-distro/0 devel-distro/git/0.git
# If you have public-inbox 1.1+ installed, you may
# initialize and index your mirror using the following commands:
public-inbox-init -V2 devel-distro devel-distro/ http://lore.altlinux.org/devel-distro \
devel-distro@lists.altlinux.org devel-distro@lists.altlinux.ru devel-distro@lists.altlinux.com
public-inbox-index devel-distro
Example config snippet for mirrors.
Newsgroup available over NNTP:
nntp://lore.altlinux.org/org.altlinux.lists.devel-distro
AGPL code for this site: git clone https://public-inbox.org/public-inbox.git