37173 2019-08-31 12:13:41 +0300 добавить имя ether в список опознаваемых как TYPE eth 2020-10-31 10:29:10 +0300 1 1 4 Development Sisyphus etcnet unstable all Linux NEW P3 enhancement --- 1 asy sem ldv mike rider sem shaba qa-sisyphus oldest_to_newest 184027 0 asy 2019-08-31 12:13:41 +0300 В рамках работы над https://bugzilla.altlinux.org/32167 Как-то вот так например: --- functions.bak 2018-12-03 16:30:57.000000000 +0400 +++ functions 2019-08-31 13:01:39.750502691 +0400 @@ -128,6 +128,9 @@ { local NAME=${1:?missing 1st arg to $FUNCNAME} local CAND=${NAME%%[0-9]*} + if [ $CAND = "ether" ]; then + CAND=eth + fi supported_type $CAND && echo $CAND } 184028 1 asy 2019-08-31 12:21:19 +0300 кавычки забыл: [ "$CAND" = "ether" ] 184034 2 asy 2019-08-31 18:12:17 +0300 Или, может, правильнее несколько симлинков просто добавить: create-ether -> create-eth destroy-ether -> destroy-eth setup-ether -> setup-eth shutdown-ether -> shutdown-eth 188453 3 asy 2020-03-10 17:04:33 +0300 Похоже вариант с тремя строчками в functions проще. Нет возражений по поводу такой доработки? 191611 4 asy 2020-07-27 19:31:57 +0300 (In reply to Sergey Y. Afonin from comment #3) > Похоже вариант с тремя строчками в functions проще. Нет возражений по поводу > такой доработки? https://lists.altlinux.org/pipermail/devel/2020-April/210614.html Как там с обновлением? 191615 5 rider 2020-07-28 08:03:28 +0300 Не совсем понятно - зачем. TYPE же всегда можно указать. 191616 6 asy 2020-07-28 08:26:41 +0300 (In reply to Anton Farygin from comment #5) > Не совсем понятно - зачем. > TYPE же всегда можно указать. Для удобства. TYPE вообще же для всего можно написать, но есть автоопределение. :-) 193642 7 sem 2020-10-30 15:23:41 +0300 Мне не очевидно почему имя ether должно быть обработано особым образом. Как я вижу оно не является умолчанием даже в udev-rule-generator сейчас. Имена интерфейсов могут быть произвольные, мы же не будем добавлять все варианты имен, которые кто-то у себя может использовать? Скорее уж логичнее добавлять поддержку для стандартной схемы именования, которая используется в udev сейчас (вида enp0s1). Но я и этого не хочу делать, текущая функция проста и понятна, а усложнение ее будет выглядеть уже как набор костылей, достаточно посмотреть на предложенный патч. Я думаю, что стоит вообще перестать полагаться на это автоопределение типа и всегда указывать тип явно в конфиге. 193643 8 rider 2020-10-30 15:28:57 +0300 Лучше тогда уж логику переписать на sysfs или udev. Там уже есть определение типа интерфейса. /sys/class/net/<name>/type = 1 это eth 193645 9 sem 2020-10-30 15:40:44 +0300 да, тоже вариант. Хотя надо посмотреть, возможно для других типов интерфейсов тип определяется до того, как интерфейсы создаются. 193647 10 sem 2020-10-30 16:01:39 +0300 (Ответ для Mikhail Efremov на комментарий #9) > да, тоже вариант. Хотя надо посмотреть, возможно для других типов > интерфейсов тип определяется до того, как интерфейсы создаются. Да, это определение происходит в самом начале, при чтении конфигов. Собственно, функция называется name2type() и делает именно то, что заявлено в названии. Добавлять в нее заглядывание в sysfs точно не надо. Можно сделать еще одну функцию для этого, но мне кажется это тоже лишнее. 193648 11 asy 2020-10-30 16:04:21 +0300 (In reply to Mikhail Efremov from comment #7) > Мне не очевидно почему имя ether должно быть обработано особым образом. Как > я вижу оно не является умолчанием даже в udev-rule-generator сейчас. Только потому, что пока от этого ломается Альтератор и etcnet интерфейсы не поднимает без type. А так всё готово. Ручка в /etc/sysconfig, можно переключить даже без обновления пакета. > Имена интерфейсов могут быть произвольные, мы же не будем добавлять все варианты Да, но какое-то должно же быть по умолчанию, и это не eth, так как eth сломали в udev. Я бы с удовольствием ничего не менял, если бы у кого-то ручки шаловливые до udev не дотянулись. 193649 12 asy 2020-10-30 16:08:17 +0300 (In reply to Mikhail Efremov from comment #7) > Я думаю, что стоит вообще перестать полагаться на это автоопределение типа и > всегда указывать тип явно в конфиге. Хотя, в общем-то, тоже вариант, но будет ли удобно и не будет ли каких-то проблем с забывчивостью. Пока-то параметр необязательный. 193650 13 sem 2020-10-30 16:18:47 +0300 (Ответ для Sergey Y. Afonin на комментарий #11) > Только потому, что пока от этого ломается Альтератор и etcnet интерфейсы не > поднимает без type. Alterator-net-eth всегда прописывал TYPE явно. > Да, но какое-то должно же быть по умолчанию, и это не eth, так как eth > сломали в udev. Я бы с удовольствием ничего не менял, если бы у кого-то > ручки шаловливые до udev не дотянулись. Это умолчание тогда уж имена вида enp0s1, как я писал выше. > Хотя, в общем-то, тоже вариант, но будет ли удобно и не будет ли каких-то > проблем с забывчивостью. Пока-то параметр необязательный. TYPE всегда обязательный, просто etcnet может попытаться угадать его по имени. Разумеется, всегда надежнее указать явно. 193653 14 asy 2020-10-30 17:59:08 +0300 (In reply to Mikhail Efremov from comment #13) > Это умолчание тогда уж имена вида enp0s1, как я писал выше. За такие имена вообще руки надо отрывать. Однообразия нет, постоянности нет. Чего добились - не ясно. 193654 15 asy 2020-10-30 18:03:43 +0300 (In reply to Mikhail Efremov from comment #13) > > Только потому, что пока от этого ломается Альтератор и etcnet интерфейсы не > > поднимает без type. > > Alterator-net-eth всегда прописывал TYPE явно. Хотя про Alterator, возможно, я поторопился. Было такое сообщение: https://lists.altlinux.org/pipermail/devel/2019-August/208342.html Я решил, что что-то сломалось, но может быть я был не прав. Уточню. 193656 16 rider 2020-10-30 19:01:45 +0300 (Ответ для Mikhail Efremov на комментарий #10) > (Ответ для Mikhail Efremov на комментарий #9) > > да, тоже вариант. Хотя надо посмотреть, возможно для других типов > > интерфейсов тип определяется до того, как интерфейсы создаются. > > Да, это определение происходит в самом начале, при чтении конфигов. > Собственно, функция называется name2type() и делает именно то, что заявлено > в названии. Добавлять в нее заглядывание в sysfs точно не надо. Можно > сделать еще одну функцию для этого, но мне кажется это тоже лишнее. Я честно говоря не понял о чём речь. Если интерфейса на момент входа в эту функцию нет, то его уже после этого никто не создаст (в случае с eth). А если есть, то тип интерфейса можно взять из sysfs, вместо автоугадава по имени. 193664 17 sem 2020-10-30 22:03:12 +0300 Тип определяет какие функции etcnet будет использовать дальше. В случае виртуальных интерфейсов (вроде bridge) как раз тип и определяет, что их нужно создавать. 193665 18 rider 2020-10-31 10:29:10 +0300 Это понятно. но в случае, если запрашиваемый интерфейс уже есть, а тип явно не определён. то разумно было бы уточнить этот тип через sysfs, а не полагаться в логике определения на имя интерфейса.