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

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

    <bug>
          <bug_id>49340</bug_id>
          
          <creation_ts>2024-02-09 13:22:32 +0300</creation_ts>
          <short_desc>update-kernel -n не скачивает пакеты</short_desc>
          <delta_ts>2024-02-27 05:56:01 +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>update-kernel</component>
          <version>unstable</version>
          <rep_platform>x86_64</rep_platform>
          <op_sys>Linux</op_sys>
          <bug_status>CLOSED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P5</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Сергей Сысоев">sysoevsa</reporter>
          <assigned_to name="Vitaly Chikunov">vt</assigned_to>
          <cc>boyarsh</cc>
    
    <cc>evg</cc>
    
    <cc>lav</cc>
    
    <cc>mike</cc>
    
    <cc>sysoevsa</cc>
    
    <cc>vt</cc>
          
          <qa_contact>qa-sisyphus</qa_contact>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>241332</commentid>
    <comment_count>0</comment_count>
    <who name="Сергей Сысоев">sysoevsa</who>
    <bug_when>2024-02-09 13:22:32 +0300</bug_when>
    <thetext>Добрый день.

update-kernel в мануале (и в справке) имеет параметр:
&gt;       -n, --dry-run
&gt;              скачать пакеты, но не устанавливать  (--download-only)

Но на самом деле *скачивание* при использовании данного ключа не происходит.
Происходит так сказать *dry run* (apt-get dist-upgrade ... --dry-run ..), который фактически ничего не делает, только показывает что будет делать. 
Т.е. не подходит для дальнейшей оффлайн-установки.

Прошу либо добавить отдельный ключ -d (такой же как у apt-get dist-upgrade -d) для скачивания, либо всё-таки сделать, чтобы этот параметр именно качала пакеты.

Спасибо.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>241401</commentid>
    <comment_count>1</comment_count>
    <who name="Repository Robot">repository-robot</who>
    <bug_when>2024-02-11 02:14:43 +0300</bug_when>
    <thetext>update-kernel-1.14-alt1 -&gt; sisyphus:

 Sat Feb 10 2024 Vitaly Chikunov &lt;vt@altlinux&gt; 1.14-alt1
 - Distinguish --dry-run and --download-only options (ALT#49340).</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>241430</commentid>
    <comment_count>2</comment_count>
    <who name="Сергей Сысоев">sysoevsa</who>
    <bug_when>2024-02-12 06:23:38 +0300</bug_when>
    <thetext>Как мне кажется, текущее поведение после изменений некорректно.

Теперь поведение зависит от порядка аргументов:
1) update-kernel -d -n -v
+ apt-get install ... --dry-run kernel-image-un-def...

2) /update-kernel -n -d -v
+ apt-get install ... --download-only kernel-image-un-def...

Хотя при ключе -n (--dry-run) точно не должно ничего происходить (и скачивание тоже).


И ещё пара мелких замечаний-предложений, если можно то в этот же баг, чтобы не заводить другие:
- ни в мане, ни в хелпе нет опции -v (--verbose) - было бы неплохо добавить. А то сейчас можно о ней догадаться только заглянув в код
- При любом варианте (-d/-n или без опций) выводится вопрос:
Try to install new kernel ... and update its modules [Y/n]?
Может было бы точнее писать download вместо install при ключе -d и что-нибудь типа dry-run/simulate install при -n ?

Спасибо.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>241431</commentid>
    <comment_count>3</comment_count>
    <who name="Vitaly Chikunov">vt</who>
    <bug_when>2024-02-12 06:56:40 +0300</bug_when>
    <thetext>Изменять текст это пусть в кроличью нору. Потому что кроме download, есть новое ядро, не новое ядро, реинсталл, не реинсталл, соответственно, там далее уже может быть не &quot;updates its modules&quot;, а что-другое, много модулей или 1 модуль. Так код вывода этой строки станет больше чем остальная логика, ведь он должен отражать все возможные ситуации в системе и выборы пользователя.

--verbose просто недокументированная опция не предназначенная пользователю.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>241432</commentid>
    <comment_count>4</comment_count>
    <who name="Сергей Сысоев">sysoevsa</who>
    <bug_when>2024-02-12 06:59:38 +0300</bug_when>
    <thetext>Ок, согласен. Но с первой частью комментария - про порядок аргументов - это всё же существенная проблема, как мне кажется</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>241435</commentid>
    <comment_count>5</comment_count>
    <who name="Vitaly Chikunov">vt</who>
    <bug_when>2024-02-12 08:25:03 +0300</bug_when>
    <thetext>Предварительно собрано в задании 340445 если захотите протестировать или у вас возникнут еще какие идеи. Заинсталлировать можно так:

  # apt-repo test 340445

Пожалуйста _после закрытия_ бага открывайте новый (если там не та же самая проблема), иначе постоянное закрытие одного и того же бага выглядит странно в changelog.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>241440</commentid>
    <comment_count>6</comment_count>
    <who name="Сергей Сысоев">sysoevsa</who>
    <bug_when>2024-02-12 09:04:01 +0300</bug_when>
    <thetext>(In reply to Vitaly Chikunov from comment #5)
&gt; Предварительно собрано в задании 340445 если захотите протестировать или у
&gt; вас возникнут еще какие идеи. Заинсталлировать можно так:
&gt; 
&gt;   # apt-repo test 340445

Посмотрел, вопросов нет, вроде бы всё как надо.
Теперь --dry-run вообще не задаёт вопросов, но так как ничего и не изменяет, то тоже нормально. Спасибо.

&gt; Пожалуйста _после закрытия_ бага открывайте новый (если там не та же самая
&gt; проблема), иначе постоянное закрытие одного и того же бага выглядит странно
&gt; в changelog.

Понял, спасибо, буду в следующий раз так и делать.

Можете подсказать, когда изменение в репозиторий P10 попадет?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>241442</commentid>
    <comment_count>7</comment_count>
    <who name="Vitaly Chikunov">vt</who>
    <bug_when>2024-02-12 09:27:10 +0300</bug_when>
    <thetext>Спасибо за тестирование.

Не планировал сдать в p10, но если вы хотите, то пошлю - и тогда пара недель.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>241456</commentid>
    <comment_count>8</comment_count>
    <who name="Сергей Сысоев">sysoevsa</who>
    <bug_when>2024-02-12 11:19:06 +0300</bug_when>
    <thetext>(In reply to Vitaly Chikunov from comment #7)
&gt; Не планировал сдать в p10, но если вы хотите, то пошлю - и тогда пара недель.

Да, хотел бы, спасибо.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>241480</commentid>
    <comment_count>9</comment_count>
    <who name="Repository Robot">repository-robot</who>
    <bug_when>2024-02-12 13:01:38 +0300</bug_when>
    <thetext>update-kernel-1.15-alt1 -&gt; sisyphus:

 Mon Feb 12 2024 Vitaly Chikunov &lt;vt@altlinux&gt; 1.15-alt1
 - Make --dry-run and --download-only non-exclusive (ALT#49340).
 - Improve informativeness of confirmation dialog.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>242136</commentid>
    <comment_count>10</comment_count>
    <who name="Vitaly Chikunov">vt</who>
    <bug_when>2024-02-26 22:24:27 +0300</bug_when>
    <thetext>Попало в p10.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>242165</commentid>
    <comment_count>11</comment_count>
    <who name="Сергей Сысоев">sysoevsa</who>
    <bug_when>2024-02-27 05:56:01 +0300</bug_when>
    <thetext>Виталий, спасибо!</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>