ALT Linux Distributions development
 help / color / mirror / Atom feed
* [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