Bug 34606
| Summary: | [FR] если нет ifplugd, ничего не сообщать о состоянии подключения | ||
|---|---|---|---|
| Product: | Sisyphus | Reporter: | Michael Shigorin <mike> |
| Component: | alterator-net-eth | Assignee: | Mikhail Efremov <sem> |
| Status: | NEW --- | QA Contact: | qa-sisyphus |
| Severity: | enhancement | ||
| Priority: | P3 | CC: | boyarsh, manowar, rider, sem, vitty, zerg |
| Version: | unstable | Keywords: | usability |
| Hardware: | all | ||
| OS: | Linux | ||
Поведение не изменилось: третьего состояния "неизвестно" в коде нет, при отсутствии средств проверки печатается "unplugged".
alterator-net-eth 5.2.12-alt1, backend3/net-eth:506:
netdev_is_plugged "$name" && tmp="plugged" || tmp="unplugged"
alterator-hw-functions 0.7.8-alt1, netdev_is_plugged() (строки 90-101): если есть /usr/sbin/ifplugstatus — используется он, иначе /usr/sbin/ethtool (разбор "Link detected: yes"); если нет ни того, ни другого — возвращается 1, что на строке 506 неотличимо от реально выдернутого кабеля. Фолбэк на ethtool существовал уже на момент отчёта (коммит a808576, 2011, 0.7-alt6).
При этом ни ethtool, ни ifplugd в зависимостях не значатся: rpm -q --requires alterator-hw-functions даёт findutils, udevadm, sfdisk, sed, blkid, coreutils, gawk, lsblk, grep, util-linux. Так что в install2 описанная ситуация воспроизводится по коду. ifplugd в репозитории есть (0.28-alt6).
Правка небольшая: вернуть из netdev_is_plugged отдельный код "нет данных" и не печатать строку о состоянии в этом случае.
|
Description
Michael Shigorin 2018-03-02 18:49:16 MSK