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

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

    <bug>
          <bug_id>37569</bug_id>
          
          <creation_ts>2019-12-03 17:35:31 +0300</creation_ts>
          <short_desc>вернуть простую авторазбивку</short_desc>
          <delta_ts>2021-09-23 09:07:28 +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>alterator-vm</component>
          <version>unstable</version>
          <rep_platform>all</rep_platform>
          <op_sys>Linux</op_sys>
          <bug_status>CLOSED</bug_status>
          <resolution>WONTFIX</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>regression</keywords>
          <priority>P3</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Michael Shigorin">mike</reporter>
          <assigned_to name="Олег Соловьев">mcpain</assigned_to>
          <cc>aen</cc>
    
    <cc>cas</cc>
    
    <cc>ldv</cc>
    
    <cc>mcpain</cc>
    
    <cc>rider</cc>
          
          <qa_contact>qa-sisyphus</qa_contact>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>185973</commentid>
    <comment_count>0</comment_count>
    <who name="Michael Shigorin">mike</who>
    <bug_when>2019-12-03 17:35:31 +0300</bug_when>
    <thetext>Коммитом 2893f2c735f1678e49b9586bcce5aac22eada4a1 в alterator-vm был изменён метод разбивки по умолчанию с plain на lvm; это предъявляет дополнительные требования к содержанию дистрибутива и не было никак обосновано.  Предлагаю откатить ровно этот коммит -- кому нужны дополнительные возможности, тот пусть обеспечит наличие всего требуемого в образе.

Баг повешен по результатам сегодняшнего совещания, на котором было упомянуто как регрессия.

Могу подготовить NMU.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>185974</commentid>
    <comment_count>1</comment_count>
    <who name="Anton Farygin">rider</who>
    <bug_when>2019-12-03 17:41:32 +0300</bug_when>
    <thetext>Миша, это тестовый пример профиля, ни к чему не обязывающий.

Если ты сделаешь другой профиль _в своём дистрибутиве_, то там будет другая конфигурация по умолчанию.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>185976</commentid>
    <comment_count>2</comment_count>
    <who name="Sergey Bolshakov">sbolshakov</who>
    <bug_when>2019-12-03 17:48:34 +0300</bug_when>
    <thetext>это же пример профиля, он используется, когда нет дистроспецифичного --
то есть, примерно никогда.
я могу наверное переименовать его в sample или вообще убрать,
поскольку именно как sample он и предполагался с самого начала.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>185978</commentid>
    <comment_count>3</comment_count>
    <who name="Anton Farygin">rider</who>
    <bug_when>2019-12-03 17:52:09 +0300</bug_when>
    <thetext>нет, только пожалуйста, не надо убирать. Переименовать отличная идея, заодно выясним кто и зачем его использует.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>185979</commentid>
    <comment_count>4</comment_count>
    <who name="Michael Shigorin">mike</who>
    <bug_when>2019-12-03 18:05:28 +0300</bug_when>
    <thetext>Напоролись в LiveCD образовательного дистрибутива -- там vm-profile.scm оказался в installer-distro.

Что пример -- понимаю, но лучше пусть он всё-таки имеет больше шансов ещё и рабочим оказаться.

Если переименовать -- насколько помню, &quot;из коробки&quot; alterator-vm в инсталерах станет неприменимым для авторазбивки в тех случаях, когда сейчас годится.

То есть это будет другая регрессия.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>185980</commentid>
    <comment_count>5</comment_count>
    <who name="Anton Farygin">rider</who>
    <bug_when>2019-12-03 18:10:47 +0300</bug_when>
    <thetext>в livecd нужно включать другой профиль.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>185981</commentid>
    <comment_count>6</comment_count>
    <who name="Anton Farygin">rider</who>
    <bug_when>2019-12-03 18:12:26 +0300</bug_when>
    <thetext>А про какие дополнительные требования к содержанию дистрибутивов ты пишешь ? У нас есть дистрибутивы без lvm ?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>185983</commentid>
    <comment_count>7</comment_count>
    <who name="Michael Shigorin">mike</who>
    <bug_when>2019-12-03 18:29:47 +0300</bug_when>
    <thetext>(В ответ на комментарий №5)
&gt; в livecd нужно включать другой профиль.
Чем он должен отличаться?

(В ответ на комментарий №6)
&gt; А про какие дополнительные требования к содержанию дистрибутивов ты пишешь ?
Очевидно, наличие пакета make-initrd-lvm (и вообще-то lvm2 тоже).

&gt; У нас есть дистрибутивы без lvm ?
У нас бывают образы без lvm и ломать универсальные компоненты вижу странным.
Ну, нарушение принципа наименьшего удивления, работоспособности из коробки,
всё такое.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>185985</commentid>
    <comment_count>8</comment_count>
    <who name="Anton Farygin">rider</who>
    <bug_when>2019-12-03 18:37:54 +0300</bug_when>
    <thetext>Это изменение было сделано год назад, похоже что потребность в его откате надумана.

Ну и да, я уверен в том, что отсутствие в дистрибутивах поддержки lvm является серьёзнейшей ошибкой автора такого дистрибутива.

В любом случае - пример конфига мне не очень интересен, но хотелось бы что бы он было максимально функциональный, в том числе и с поддержкой lvm.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>185987</commentid>
    <comment_count>9</comment_count>
    <who name="Michael Shigorin">mike</who>
    <bug_when>2019-12-03 18:51:35 +0300</bug_when>
    <thetext>Антон, я собирался эту просьбу повесить тогда же, как только заметил -- но тогда уже заполненную багу решил не отправлять, сказав добавить конфиг в образ.

Примеры конфигов в самых разных вариантах положить в документацию alterator-vm было бы неплохо (готов приложить руку), но усложнение без сколь-нибудь внятной причины ситуации по умолчанию всё-таки считаю регрессией.

Против твоей уверенности в полезности lvm есть моя уверенность в ненужности усложнений на ровном месте, а также бухтёх gremlin@ о том, что в dm известны арифметические ошибки (и он порой видит их последствия у других) -- в любом случае вкусы хороши уж тем, что бывают разные, но логику пока не отменяли.
Могу расписать дерево ситуаций, но надо ли?..

PS: помнишь, как для школьного терминального сервера была сделана авторазбивка с raid (там было осмысленно так)?  По твоей логике следует и это воткнуть в штатный конфиг, потому что и функция же ж хорошая, и иллюстрирует; по моей -- простое стоит делать простым, сложное -- возможным.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>185988</commentid>
    <comment_count>10</comment_count>
    <who name="Sergey Bolshakov">sbolshakov</who>
    <bug_when>2019-12-03 18:53:59 +0300</bug_when>
    <thetext>давайте я ещё раз попробую:
обсуждаемое изменение было сделано в примере, таковым это файл считался
всегда, поскольку очевидно, что некоей разбивки по умолчанию, пригодной
для всех случаев, просто не существует, поэтому:
я готов выслушать предложения по изменениям в этом файле/с этим файлом,
чтобы более ясно донести ровно эту мысль -- как-то: переименовать, убрать,
написать БОЛЬШИМИ БУКВАМИ DO NOT USE IN PRODUCTION -- что-то ещё ?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>185989</commentid>
    <comment_count>11</comment_count>
    <who name="Anton Farygin">rider</who>
    <bug_when>2019-12-03 18:56:57 +0300</bug_when>
    <thetext>Миша, почему наличие lvm в дистрибутиве нужно (если он устанавливается через alterator-vm), так это то, что с помощью alterator-vm в ручном режиме lvm делается без проблем и ты получишь незагружаемую систему (evms таскает с собой реализацию lvm и ему для этого никаких тулзов не надо).

В общем по мне так ошибки тут нет. Дальнейшую дискуссию продолжать смысла не вижу.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>203124</commentid>
    <comment_count>12</comment_count>
    <who name="Michael Shigorin">mike</who>
    <bug_when>2021-09-23 08:33:16 +0300</bug_when>
    <thetext>(In reply to Anton Farygin from comment #8)
&gt; Это изменение было сделано год назад, похоже что потребность в его откате
&gt; надумана.

Когда заметил, тогда и повесил -- я сейчас x86 редко занимаюсь.

&gt; Ну и да, я уверен в том, что отсутствие в дистрибутивах поддержки lvm
&gt; является серьёзнейшей ошибкой автора такого дистрибутива.

Это твоя позиция.  Моя диаметрально противоположна.  При этом на моей стороне тот тривиальный довод, что предлагать вместо опирающегося на заведомо присутствующее в дистрибутиве опирающееся на то, что надо положить (притом не являющееся технически необходимым) -- значит на ровном месте увеличивать шансы на неработоспособность суммы.


(In reply to Anton Farygin from comment #11)
&gt; Миша, почему наличие lvm в дистрибутиве нужно (если он устанавливается через
&gt; alterator-vm), так это то, что с помощью alterator-vm в ручном режиме lvm
&gt; делается без проблем и ты получишь незагружаемую систему (evms таскает с
&gt; собой реализацию lvm и ему для этого никаких тулзов не надо).

Это не довод, а сообщение об _ошибке_ в alterator-vm, который умеет делать медвежью услугу -- сравни с поведением по mkfs.* (не предлагать создать те ФС, утилиты для которых не найдены).

&gt; В общем по мне так ошибки тут нет. Дальнейшую дискуссию продолжать смысла
&gt; не вижу.

Верю тебе, но поскольку alterator-vm на сейчас не только часть KWorkstation,
а и часть любого дистрибутива альта -- придётся попросить мнение руководства,
раз не удалось самостоятельно тебя убедить в деструктивности таких изменений.


(In reply to Sergey Bolshakov from comment #10)
&gt; обсуждаемое изменение было сделано в примере, таковым это файл считался
&gt; всегда, поскольку очевидно, что некоей разбивки по умолчанию, пригодной
&gt; для всех случаев, просто не существует, поэтому:
&gt; я готов выслушать предложения по изменениям в этом файле/с этим файлом,
&gt; чтобы более ясно донести ровно эту мысль -- как-то: переименовать, убрать,
&gt; написать БОЛЬШИМИ БУКВАМИ DO NOT USE IN PRODUCTION -- что-то ещё ?

Боюсь, в случаях, когда она по факту применяется -- её по факту уже не заметили.  Ну и опять же с точки зрения хоть какой-то работоспособности по умолчанию хотя бы корень сделать, если есть где -- вполне логично (плюс, возможно, ESP в случае EFI или что ещё особенного в случае других фирмварей -- это уже за рамками данной баги и местами уже висит).

И всё равно не понял: зачем тогда захламлять неиспользуемый пример ещё и lvm?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>203126</commentid>
    <comment_count>13</comment_count>
    <who name="Anton Farygin">rider</who>
    <bug_when>2021-09-23 08:44:53 +0300</bug_when>
    <thetext>Нет, lvm по умолчанию это обязательно и не регрессия. Все дистрибутивы готовы к этом изменению, а кто не готов - сделает соответствующий профиль.

То, что ты не понимаешь как работает lvm не является поводом откатывать данное изменение.

Считай это мнением руководства с моей стороны.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>203127</commentid>
    <comment_count>14</comment_count>
    <who name="Anton Farygin">rider</who>
    <bug_when>2021-09-23 08:57:00 +0300</bug_when>
    <thetext>Добавлю, что если за два года это никому не помешало, то твои _требования_ неразумны и надуманны.

Как перечисляли выше - данный файл является примером, показывающим возможности alterator-vm, и с lvm в том числе.

Не хотите его использовать - не надо. Так и происходит в 100% профилей дистрибутивов.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>203128</commentid>
    <comment_count>15</comment_count>
    <who name="AEN">aen</who>
    <bug_when>2021-09-23 09:05:38 +0300</bug_when>
    <thetext>Можно добавить другой пример, полагаю. 

Мне кажется, что это вопрос не &quot;начальства&quot;, а релиз-менеджеров. А если хочется устроить обсуждение, то нужно посмотреть и описание т Ъ как это сделано &quot;у них&quot;. Не потому, что там правильно, а чтобы расширить число участник ков обсуждения. :)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>203129</commentid>
    <comment_count>16</comment_count>
    <who name="Anton Farygin">rider</who>
    <bug_when>2021-09-23 09:07:28 +0300</bug_when>
    <thetext>Это вопрос разработчиков alterator-vm. Какой им захотелось пример вставить в пакет, такой и вставили. Если нужен другой пример - можно добавить и его, но ошибка должна быть именно об этом, а не об откате lvm в example разбивки.

А тратить время на бесполезное с точки зрения продуктивности обсуждение не нужно.</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>