Bug 23156 - При загрузке образа не сообщается о нехватке места
Summary: При загрузке образа не сообщается о нехватке места
Status: CLOSED FIXED
Alias: None
Product: Sisyphus
Classification: Development
Component: alterator-netinst (show other bugs)
Version: unstable
Hardware: all Linux
: P3 normal
Assignee: Anton V. Boyarshinov
QA Contact: qa-sisyphus
URL:
Keywords: usability
Depends on:
Blocks: 23155
  Show dependency tree
 
Reported: 2010-03-15 17:57 MSK by Andrey Cherepanov
Modified: 2011-07-05 14:16 MSK (History)
3 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Andrey Cherepanov 2010-03-15 17:57:56 MSK
Если во время загрузки на /var заканчивается место, то модуль alterator никак не сообщает об этом, а при использовании неполностью закаченного образа выдаётся неверная диагностика.

Ожидаемый результат: модуль должен проверять целостность закаченного образа.
Comment 1 Vladislav Zavjalov 2010-11-03 13:10:49 MSK
Перевешиваю на nobody@, так как модулем заниматься не планирую. Для тех, кто захочет им заниматься, информация следующая:

1. Вся логика этого модуля вынесена за пределы альтератора, в shell-скрипт /usr/bin/alterator-netinst. Этот скрипт, запущенный без аргументов, выдает вполне подробную подсказку по использованию. Cоответственно, всю отладку можно производить с его помощью.
В данном случае я 
- поменял в скрипте значение DATADIR на то, что мне удобно
- запустил ./alterator-netinst -A file://<большой файл>
- запустил ./alterator-netinst -S
первая команда начинат скачивание, вторая - проверяет прогресс и результат.

2. В данном случае, ошибка была внесена коммитом fef59fd2e5e8684fa1da22d76e3062531adabcc3 и заключается в проверке
elif [ "$newsize" -le 0 -o "$(( $newsize - $size ))" -gt 512 ]; then
  status="fail"
...

$newsize - это размер реально скачанного файла, а $size - размер исходного.
Так что надо бы их наоборот вычитать...

Впрочем это всего лишь приведит вместо сообщения
curl: (23) Failed writing body (12472 != 16383); mount: you must specify the filesystem type; unable to mount image

к не сильно более понятному сообщению
curl: (23) Failed writing body (12472 != 16383)

Разбора (и русификации) подобных сообщений в модуле нет, если его хочется - надо дописывать. Скорее всего - научить функцию downloading_status(), чтоб она находила и заменяла наиболее часто встречающиеся сообщения на то, что надо по списку.
Comment 2 AEN 2011-06-28 18:25:07 MSK
2boyarsh@: Хорошо бы реализовать хотя бы пункт 2 из предложения slazav@ , если уже не.
Comment 3 Anton V. Boyarshinov 2011-07-05 14:16:39 MSK
(В ответ на комментарий №2)
> 2boyarsh@: Хорошо бы реализовать хотя бы пункт 2 из предложения slazav@ , если
> уже не.

Реализовано в ноябре 2010