Bug 49340 - update-kernel -n не скачивает пакеты
Summary: update-kernel -n не скачивает пакеты
Status: CLOSED FIXED
Alias: None
Product: Sisyphus
Classification: Development
Component: update-kernel (show other bugs)
Version: unstable
Hardware: x86_64 Linux
: P5 normal
Assignee: Vitaly Chikunov
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2024-02-09 13:22 MSK by Сергей Сысоев
Modified: 2024-02-27 05:56 MSK (History)
6 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Сергей Сысоев 2024-02-09 13:22:32 MSK
Добрый день.

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

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

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

Спасибо.
Comment 1 Repository Robot 2024-02-11 02:14:43 MSK
update-kernel-1.14-alt1 -> sisyphus:

 Sat Feb 10 2024 Vitaly Chikunov <vt@altlinux> 1.14-alt1
 - Distinguish --dry-run and --download-only options (ALT#49340).
Comment 2 Сергей Сысоев 2024-02-12 06:23:38 MSK
Как мне кажется, текущее поведение после изменений некорректно.

Теперь поведение зависит от порядка аргументов:
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 ?

Спасибо.
Comment 3 Vitaly Chikunov 2024-02-12 06:56:40 MSK
Изменять текст это пусть в кроличью нору. Потому что кроме download, есть новое ядро, не новое ядро, реинсталл, не реинсталл, соответственно, там далее уже может быть не "updates its modules", а что-другое, много модулей или 1 модуль. Так код вывода этой строки станет больше чем остальная логика, ведь он должен отражать все возможные ситуации в системе и выборы пользователя.

--verbose просто недокументированная опция не предназначенная пользователю.
Comment 4 Сергей Сысоев 2024-02-12 06:59:38 MSK
Ок, согласен. Но с первой частью комментария - про порядок аргументов - это всё же существенная проблема, как мне кажется
Comment 5 Vitaly Chikunov 2024-02-12 08:25:03 MSK
Предварительно собрано в задании 340445 если захотите протестировать или у вас возникнут еще какие идеи. Заинсталлировать можно так:

  # apt-repo test 340445

Пожалуйста _после закрытия_ бага открывайте новый (если там не та же самая проблема), иначе постоянное закрытие одного и того же бага выглядит странно в changelog.
Comment 6 Сергей Сысоев 2024-02-12 09:04:01 MSK
(In reply to Vitaly Chikunov from comment #5)
> Предварительно собрано в задании 340445 если захотите протестировать или у
> вас возникнут еще какие идеи. Заинсталлировать можно так:
> 
>   # apt-repo test 340445

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

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

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

Можете подсказать, когда изменение в репозиторий P10 попадет?
Comment 7 Vitaly Chikunov 2024-02-12 09:27:10 MSK
Спасибо за тестирование.

Не планировал сдать в p10, но если вы хотите, то пошлю - и тогда пара недель.
Comment 8 Сергей Сысоев 2024-02-12 11:19:06 MSK
(In reply to Vitaly Chikunov from comment #7)
> Не планировал сдать в p10, но если вы хотите, то пошлю - и тогда пара недель.

Да, хотел бы, спасибо.
Comment 9 Repository Robot 2024-02-12 13:01:38 MSK
update-kernel-1.15-alt1 -> sisyphus:

 Mon Feb 12 2024 Vitaly Chikunov <vt@altlinux> 1.15-alt1
 - Make --dry-run and --download-only non-exclusive (ALT#49340).
 - Improve informativeness of confirmation dialog.
Comment 10 Vitaly Chikunov 2024-02-26 22:24:27 MSK
Попало в p10.
Comment 11 Сергей Сысоев 2024-02-27 05:56:01 MSK
Виталий, спасибо!