<?xml version="1.0" encoding="UTF-8" ?>

<bugzilla version="5.2"
          urlbase="https://bugzilla.altlinux.org/"
          
          maintainer="jenya@basealt.ru"
>

    <bug>
          <bug_id>28082</bug_id>
          
          <creation_ts>2012-11-21 16:11:49 +0400</creation_ts>
          <short_desc>initrd. Брать название luks устройств из /etc/crypttab</short_desc>
          <delta_ts>2018-04-23 19:08:30 +0300</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>4</classification_id>
          <classification>Development</classification>
          <product>Sisyphus</product>
          <component>make-initrd-luks</component>
          <version>unstable</version>
          <rep_platform>all</rep_platform>
          <op_sys>Linux</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>LATER</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P3</priority>
          <bug_severity>enhancement</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="timonbl4@altlinux.org">timonbl4</reporter>
          <assigned_to name="Alexey Gladkov">legion</assigned_to>
          <cc>aen</cc>
    
    <cc>antohami</cc>
    
    <cc>boyarsh</cc>
    
    <cc>glebfm</cc>
    
    <cc>ldv</cc>
    
    <cc>legion</cc>
    
    <cc>mike</cc>
    
    <cc>placeholder</cc>
    
    <cc>vt</cc>
          
          <qa_contact>qa-sisyphus</qa_contact>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>135185</commentid>
    <comment_count>0</comment_count>
    <who name="timonbl4@altlinux.org">timonbl4</who>
    <bug_when>2012-11-21 16:11:49 +0400</bug_when>
    <thetext>В системе есть /etc/crypttab:
luks1    UUID=&lt;uuid1&gt;    none    luks

init скрипты смотрят в этот файл и проверяют, если luks1 не существует, то активировать UUID=&lt;uuid1&gt;

При загрузке получаем: в initrd активируем sda1, называет sda1-luks, init скрипты не находят luks1 и пытается повторно активировать sda1</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>135191</commentid>
    <comment_count>1</comment_count>
      <attachid>5643</attachid>
    <who name="timonbl4@altlinux.org">timonbl4</who>
    <bug_when>2012-11-21 16:17:23 +0400</bug_when>
    <thetext>Created attachment 5643
Патч на make-initrd 0.7.9-alt1</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>135211</commentid>
    <comment_count>2</comment_count>
    <who name="timonbl4@altlinux.org">timonbl4</who>
    <bug_when>2012-11-21 16:24:46 +0400</bug_when>
    <thetext>Патчи на ветку legion/master:
http://git.altlinux.org/people/timonbl4/packages/?p=make-initrd.git;a=shortlog;h=aab942ee02debefa157d0b694767adf723751cf9</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>135221</commentid>
    <comment_count>3</comment_count>
    <who name="Alexey Gladkov">legion</who>
    <bug_when>2012-11-21 16:54:50 +0400</bug_when>
    <thetext>+ LUKS_ROOT_UUID=$(blkid -o value -s UUID &quot;$LUKS_ROOT&quot;)

Видимо вы не поняли мой ответ по почте. Эти значения можно получить из udev (он их получает через libblkid) в хэндлере. Не нужно лишний раз дёргать одни и те же механизмы для получения одного и того же.

+ if [ x&quot;$crypttab_block&quot; = x&quot;$LUKS_ROOT&quot; ] || [ x&quot;$crypttab_block&quot; = x&quot;UUID=$LUKS_ROOT_UUID&quot; ]; then

Не могли бы вы придерживаться стиля принятого в остальном коде ?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>135228</commentid>
    <comment_count>4</comment_count>
    <who name="timonbl4@altlinux.org">timonbl4</who>
    <bug_when>2012-11-21 17:30:25 +0400</bug_when>
    <thetext>(В ответ на комментарий №3)
&gt; + LUKS_ROOT_UUID=$(blkid -o value -s UUID &quot;$LUKS_ROOT&quot;)
&gt; 
&gt; Видимо вы не поняли мой ответ по почте. Эти значения можно получить из udev (он
&gt; их получает через libblkid) в хэндлере.

$ID_FS_UUID_ENC ?

&gt; + if [ x&quot;$crypttab_block&quot; = x&quot;$LUKS_ROOT&quot; ] || [ x&quot;$crypttab_block&quot; =
&gt; x&quot;UUID=$LUKS_ROOT_UUID&quot; ]; then
&gt; 
&gt; Не могли бы вы придерживаться стиля принятого в остальном коде ?

Можете пояснить?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>135229</commentid>
    <comment_count>5</comment_count>
    <who name="Michael Shigorin">mike</who>
    <bug_when>2012-11-21 17:55:20 +0400</bug_when>
    <thetext>(In reply to comment #4)
&gt; &gt; + if [ x&quot;$crypttab_block&quot; = x&quot;$LUKS_ROOT&quot; ] || [ x&quot;$crypttab_block&quot; =
&gt; &gt; x&quot;UUID=$LUKS_ROOT_UUID&quot; ]; then
&gt; &gt; Не могли бы вы придерживаться стиля принятого в остальном коде ?
&gt; Можете пояснить?
Видимо, [ x$a = x... ] -- дикая архаика, которая в некоторых местах ещё встречается (не считая случаев вроде configure, когда работать должно и на совсем странном хламе).

Возможно, лучше так:
&gt; if [ &quot;$crypttab_block&quot; = &quot;$LUKS_ROOT&quot; -o &quot;$crypttab_block&quot; = &quot;UUID=$LUKS_ROOT_UUID&quot; ]; then

Хотя я бы м.б. оформил как

&gt; case &quot;$crypttab_block&quot; in
&gt;         &quot;$LUKS_ROOT&quot;|&quot;UUID=$LUKS_ROOT_UUID&quot;) ...;;
&gt; esac</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>135230</commentid>
    <comment_count>6</comment_count>
    <who name="Alexey Gladkov">legion</who>
    <bug_when>2012-11-21 18:04:16 +0400</bug_when>
    <thetext>(В ответ на комментарий №4)
&gt; $ID_FS_UUID_ENC ?

Угу ... ID_FS_UUID*. Их там насколько я помню несколько под разные цели.
Передать её можно точно также как LUKS_ROOT.
 

&gt; &gt; + if [ x&quot;$crypttab_block&quot; = x&quot;$LUKS_ROOT&quot; ] || [ x&quot;$crypttab_block&quot; =
&gt; &gt; x&quot;UUID=$LUKS_ROOT_UUID&quot; ]; then
&gt; &gt; 
&gt; &gt; Не могли бы вы придерживаться стиля принятого в остальном коде ?
&gt; 
&gt; Можете пояснить?

выражения в if/while не разделяются.
x&quot;something&quot; это бесполезный код -&gt; &quot;something&quot;.

Кстати, в вашем &quot;while read&quot; нет проверки на пустые строки и комментарии. Если с первыми проблем больших нет если верить, что LUKS_ROOT_UUID всегда заполнен, но с вторым можно легко выдать не то.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>136249</commentid>
    <comment_count>7</comment_count>
    <who name="Michael Shigorin">mike</who>
    <bug_when>2012-12-21 18:12:52 +0400</bug_when>
    <thetext>(In reply to comment #0)
&gt; При загрузке получаем: в initrd активируем sda1, называет sda1-luks, init
&gt; скрипты не находят luks1 и пытается повторно активировать sda1
TWIMC: текущий сизифный installer не создаёт /etc/crypttab, вследствие чего получаем bug #28255.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>170560</commentid>
    <comment_count>8</comment_count>
    <who name="Alexey Gladkov">legion</who>
    <bug_when>2018-04-23 19:08:30 +0300</bug_when>
    <thetext>Reopen if needed</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>5643</attachid>
            <date>2012-11-21 16:17:23 +0400</date>
            <delta_ts>2012-11-21 16:17:23 +0400</delta_ts>
            <desc>Патч на make-initrd 0.7.9-alt1</desc>
            <filename>make-initrd-patch2.patch</filename>
            <type>text/plain</type>
            <size>1701</size>
            <attacher name="timonbl4@altlinux.org">timonbl4</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL2ZlYXR1cmVzL2x1a3MvY29uZmlnLm1rIGIvZmVhdHVyZXMvbHVrcy9jb25m
aWcubWsKaW5kZXggOTU2NGJjNi4uYTYyNWQzNyAxMDA2NDQKLS0tIGEvZmVhdHVyZXMvbHVrcy9j
b25maWcubWsKKysrIGIvZmVhdHVyZXMvbHVrcy9jb25maWcubWsKQEAgLTEsNiArMSw3IEBACiBD
UllQVFNFVFVQX0JJTgkJPz0gL3Vzci9zYmluL2NyeXB0c2V0dXAKK0NSWVBUVEFCX0ZJTEUJCT89
IC9ldGMvY3J5cHR0YWIKIExVS1NfREFUQURJUgkJPz0gJChGRUFUVVJFU0RJUikvbHVrcy9kYXRh
CiBMVUtTX0NJUEhFUlMJCT89IGFlcwogTFVLU19CTE9DS0NJUEhFUlMJPz0gY2JjCiBMVUtTX0hB
U0hFUwkJPz0gc2hhMjU2Ci1MVUtTX01PRFVMRVMJCSA9IGRtLWNyeXB0ICQoTFVLU19DSVBIRVJT
KSAkKExVS1NfQkxPQ0tDSVBIRVJTKSAkKExVS1NfSEFTSEVTKQpcIE5vIG5ld2xpbmUgYXQgZW5k
IG9mIGZpbGUKK0xVS1NfTU9EVUxFUwkJID0gZG0tY3J5cHQgJChMVUtTX0NJUEhFUlMpICQoTFVL
U19CTE9DS0NJUEhFUlMpICQoTFVLU19IQVNIRVMpCmRpZmYgLS1naXQgYS9mZWF0dXJlcy9sdWtz
L2RhdGEvbGliL2hhbmRsZXJzLzA1MC1sdWtzIGIvZmVhdHVyZXMvbHVrcy9kYXRhL2xpYi9oYW5k
bGVycy8wNTAtbHVrcwppbmRleCA3ZGVhYzhiLi45MWVkMDU2IDEwMDc1NQotLS0gYS9mZWF0dXJl
cy9sdWtzL2RhdGEvbGliL2hhbmRsZXJzLzA1MC1sdWtzCisrKyBiL2ZlYXR1cmVzL2x1a3MvZGF0
YS9saWIvaGFuZGxlcnMvMDUwLWx1a3MKQEAgLTU1LDYgKzU1LDE2IEBAIHJlYWRrZXkoKSB7CiBo
YW5kbGVyKCkgewogCW5hbWVsdWtzPSIke0xVS1NfUk9PVCMjKi99LWx1a3MiCiAKKwlpZiBbIC1y
IC9ldGMvY3J5cHR0YWIgXTsgdGhlbgorCQlMVUtTX1JPT1RfVVVJRD0kKGJsa2lkIC1vIHZhbHVl
IC1zIFVVSUQgIiRMVUtTX1JPT1QiKQorCQl3aGlsZSByZWFkIGNyeXB0dGFiX25hbWUgY3J5cHR0
YWJfYmxvY2sgY3J5cHR0YWJfc3BlY2lmaWVzIGNyeXB0dGFiX29wdGlvbnM7IGRvCisJCQlpZiBb
IHgiJGNyeXB0dGFiX2Jsb2NrIiA9IHgiJExVS1NfUk9PVCIgXSB8fCBbIHgiJGNyeXB0dGFiX2Js
b2NrIiA9IHgiVVVJRD0kTFVLU19ST09UX1VVSUQiIF07IHRoZW4KKwkJCQluYW1lbHVrcz0kY3J5
cHR0YWJfbmFtZQorCQkJCWJyZWFrCisJCQlmaQorCQlkb25lIDwgL2V0Yy9jcnlwdHRhYgorCWZp
CisKIAkjIHNraXAgaWYgJG5hbWVsdWtzIGhhcyBhbHJlYWR5IGV4aXN0CiAJISBkbXNldHVwIGlu
Zm8gIiRuYW1lbHVrcyIgPi9kZXYvbnVsbCAyPiYxIHx8CiAJCWV4aXQgMApkaWZmIC0tZ2l0IGEv
ZmVhdHVyZXMvbHVrcy9ydWxlcy5tayBiL2ZlYXR1cmVzL2x1a3MvcnVsZXMubWsKaW5kZXggMGRj
OWQxOC4uZjcwMTk0OSAxMDA2NDQKLS0tIGEvZmVhdHVyZXMvbHVrcy9ydWxlcy5taworKysgYi9m
ZWF0dXJlcy9sdWtzL3J1bGVzLm1rCkBAIC01LDYgKzUsNyBAQCAkKGNhbGwgcmVxdWlyZSxkZXZt
YXBwZXIsZGVwbW9kLWltYWdlKQogbHVrczogZGV2bWFwcGVyCiAJQGVjaG8gIkFkZGluZyBMVUtT
IHN1cHBvcnQgLi4uIgogCUBwdXQtZmlsZSAiJChST09URElSKSIgJChDUllQVFNFVFVQX0JJTikK
KwlAcHV0LWZpbGUgIiQoUk9PVERJUikiICQoQ1JZUFRUQUJfRklMRSkKIAlAcHV0LXRyZWUgIiQo
Uk9PVERJUikiICQoTFVLU19EQVRBRElSKQogCiBkZXBtb2QtaW1hZ2U6IGx1a3MK
</data>

          </attachment>
      

    </bug>

</bugzilla>