Bug 28647 - Не удаляются записи о старых ядрах в lilo.conf
Summary: Не удаляются записи о старых ядрах в lilo.conf
Status: CLOSED FIXED
Alias: None
Product: Branch t6
Classification: Distributions
Component: bootloader-utils (show other bugs)
Version: не указана
Hardware: all Linux
: P3 critical
Assignee: Nobody's working on this, feel free to take it
QA Contact: QA t6
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-03-08 20:58 MSK by Roman Savochenko
Modified: 2013-04-07 21:41 MSK (History)
3 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Roman Savochenko 2013-03-08 20:58:12 MSK 
В T6 переехал с grub2 на lilo и всёбы хорошо, но при удалении старых ядер записи о них вроде image="/boot/vmlinuz-3.0.65-std-def-alt0.M60P.1" не удаляются из lilo.conf, что приводит к сообщениям ошибки LILO при обновлении загрузчика и невозможности вообще загрузиться если одновременно ставилось новое и удалялось старое ядро.
Comment 1 Michael Shigorin 2013-03-08 21:00:32 MSK 
Сам поправишь?  IIRC обсуждалось и вопрос перекликается с /usr/lib/rpm/boot_kernel.filetrigger
Comment 2 Roman Savochenko 2013-03-09 18:53:23 MSK 
(В ответ на комментарий №1)
> Сам поправишь?  IIRC обсуждалось и вопрос перекликается с
Если-бы я сам мог быстро поправить то я и баги тут не создавал.
Собственно поглядел на /usr/share/loader/lilo, потом не понял откуда он вызывается и что возможно сверху не вызывается remove_kernel().

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

> /usr/lib/rpm/boot_kernel.filetrigger
Похоже именно отсюда оно и вызывается, но ещё через какого-то посредника.
Comment 3 Roman Savochenko 2013-03-13 23:15:27 MSK 
(В ответ на комментарий №1)
> Сам поправишь?  IIRC обсуждалось и вопрос перекликается с
> /usr/lib/rpm/boot_kernel.filetrigger
Исправил:

--- boot_kernel.filetrigger_    2013-03-13 20:54:45.400836619 +0200
+++ boot_kernel.filetrigger     2013-03-13 20:49:30.782812824 +0200
@@ -63,6 +63,7 @@
                /sbin/installkernel "$VERSION"
        else
                # file does not exist -> uninstall kernel
+               /sbin/installkernel -r "$VERSION"
                rm -f /boot/initrd-"$VERSION".img

                # update default kernel if necessary
Comment 4 Michael Shigorin 2013-03-13 23:38:32 MSK 
Прошу глянуть патч.
Comment 5 led 2013-03-13 23:45:40 MSK 
(В ответ на комментарий №4)
> Прошу глянуть патч.

AFAIK
/sbin/installkernel -r "$VERSION"
уже предполагает
rm -f /boot/initrd-"$VERSION".img
Comment 6 Roman Savochenko 2013-03-14 00:12:36 MSK 
(В ответ на комментарий №5)
> (В ответ на комментарий №4)
> > Прошу глянуть патч.
> 
> AFAIK
> /sbin/installkernel -r "$VERSION"
> уже предполагает
> rm -f /boot/initrd-"$VERSION".img
Да, строку "rm -f /boot/initrd-"$VERSION".img" можно убрать.
Проверил — удаляет.
Comment 7 Roman Savochenko 2013-04-07 18:09:42 MSK 
Что дальше?
Мне это исправление добавить или как?
Comment 8 Repository Robot 2013-04-07 18:35:56 MSK 
bootloader-utils-0.4.21-alt1 -> sisyphus:

* Sun Apr 07 2013 Dmitry V. Levin <ldv@altlinux> 0.4.21-alt1
- kernel.filetrigger: fixed handling of uninstalled kernels
  (by Roman Savochenko; closes: #28647).
Comment 9 Michael Shigorin 2013-04-07 21:41:49 MSK 
Спасибо!  Осталось и в t6 :)