* [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