Opened 7 years ago

Closed 6 years ago

#278 closed улучшение (fixed)

Окно GE-xx: вместо пустой таблицы создавать дефолтный VLAN

Reported by: san Owned by: alx
Priority: средний Milestone: 1 очередь
Component: web-интерфейс (sw) Keywords:
Cc: andrei, artem

Description

При переключении из режима Port-based в 802.1q предлагаю если таблица пустая, создавать в ней VLAN №1 с включенными в него всеми портами в режиме Untagged.

Change History (12)

comment:1 by alx, 7 years ago

Хотелось бы узнать цель и мотивацию такого улучшения. Что именно в результате улучшится?

comment:2 by san, 7 years ago

Цель - небольшое улучшение восприятия этих настроек пользователем

  1. В коммутаторе sw на вкладке VLAN по дефолту присутствует 1-й vlan, хочется подобия/единообразия, хоть и в данном случае этот VLAN не является необходимостью
  2. Прописав дефолтный VLAN мы сделаем некий пример для пользователя, таблица со строчкой воспринимается лучше чем пустая
  3. Не знаю как это объяснить логически, но не первый раз встречаюсь ситуацией когда пользователи переводят плату из Port-based в 802.1q, и не трогая таблицу, применяют настройки, естественно подключены при этом к блоку они через эту плату GE, и как следствие, теряют связь с блоком. Не пытаюсь оправдать их действия, но меня это наводит на мысль, что есть некая неинтуитивность в этих настройках, что собственно и пытаюсь поправить.

comment:3 by san, 7 years ago

Cc: andrei artem added

in reply to:  2 comment:4 by alx, 7 years ago

Replying to san:

Спасибо за пояснения.

  1. В коммутаторе sw на вкладке VLAN по дефолту присутствует 1-й vlan, хочется подобия/единообразия, хоть и в данном случае этот VLAN не является необходимостью

Цель присутствия этого VLAN в коммутаторе VE-01 - чтобы по умолчанию фреймы форвардились из любого порта в любой. В конфигурация по умолчанию плат GE-XX также разрешен форвардинг из любого в любой. Таким образом, аналогия и единообразие уже есть. Другое дело, что это реализовано с помощью другого режима...

  1. Прописав дефолтный VLAN мы сделаем некий пример для пользователя, таблица со строчкой воспринимается лучше чем пустая

Я вижу в этом такой плюс, что это экономит один клик мышкой (ведь, как минимум, один VLAN надо будет создать). Предлагаю вместо VLAN 1 оставлять поле номера пустым - тогда, если оператор забудет вписать нужный номер, при нажатии "Применить" получит сообщение об ошибке. Если же там сразу будет 1, то это может оказаться совсем не тем, что требуется - и будет та же фигня, от которой мы хотим избавиться...

меня это наводит на мысль, что есть некая неинтуитивность в этих настройках, что собственно и пытаюсь поправить.

Тут, я считаю, надо просто выдавать предупреждение. См. #279.

Last edited 7 years ago by alx (previous) (diff)

comment:5 by san, 7 years ago

Я хотел бы повысить интуитивность интерфейса ДО применения настроек, а не указать на ошибки после

in reply to:  5 comment:6 by alx, 7 years ago

Replying to san:

Я хотел бы повысить интуитивность интерфейса ДО применения настроек, а не указать на ошибки после

В чем именно заключается интуитивность/неинтуитивность? Если появляется строка с полем ввода, то, по-моему, интуитивно понятно, что в него надо что-то ввести. :) Если оператор не знает, что у каждой VLAN должен быть определенный идентификатор, и, поэтому, не знает, что его надо указать, то тут никакая интуиция не поможет. Если нужен пример - то давайте сделаем именно пример - добавим строку перед таблицей, которая не будет попадать в реальный конфиг. И напишем рядом с ней, что это именно пример конфигурации VLAN...

Далее. Значение 1 с очень большой вероятностью может оказаться не тем, которое действительно требуется. Если уж автоматически создавать VLAN с уже заполненным полем VID, то будет логично использовать значения из полей "Untagged VID" портов. Логика в том, что если трафику присваивается некий идентификатор, то и VLAN с данным идентификатором должен существовать. Тут есть проблема: до включения режима 802.1q эти поля заблокированы, поэтому пользователь сначала включает режим 802.1q, а уже потом вписывает нужные значения. Следовательно, недостающие VLAN должны создаваться не в момент смены режима, а после изменения значения поля... Есть еще один момент: если предполагается, что весь трафик на входе в плату уже тегирован (что вполне может быть), значение поля "Untagged VID" никакой роли не играет, и в нем вполне может быть все что угодно - создавать VLAN с таким идентификатором нет никакой необходимости...

Кроме того, предложение включать по умолчанию все порты тоже сомнительно: если исходить из логики, что по умолчанию должен предлагаться некий наиболее "безопасный" вариант, то вариант "все включены" точно так же может привести к потере связи из-за лишних петель, как и вариант "все выключены"...

Поэтому мое мнение - не надо придумывать конфигурацию за оператора. Подсказать в виде примера - можно. Предупредить о возможной ошибке - можно (и нужно). Но конфигурировать за него - нет. Живую голову не заменишь... :)

comment:7 by san, 7 years ago

Андрей, Артём, ваше мнение?
(плата GE-12 для посмотреть есть в демоблоке)

Last edited 7 years ago by san (previous) (diff)

comment:8 by andrei, 7 years ago

Я с Алексеем согласен. Слишком много "но" и условий. Нужно читать описание, скоро в нем будет все подробно изложено).
Проверка и предупреждения тикета #279 думаю будут полезны.

comment:9 by san, 7 years ago

Какие "но"?
И в тикете одно условие: "если таблица пустая".

Нужно читать описание

Конечно нужно читать, но это не мешает нам попробовать сделать интерфейс чуть удобней.

Last edited 7 years ago by san (previous) (diff)

in reply to:  9 comment:10 by alx, 7 years ago

Replying to san:

Какие "но"?

Может быть Андрей неправильно понял мое мнение? Я не против автоматического добавления VLAN как такового. Мнения разошлись по вопросу, как эта VLAN должна быть сконфигурирована. Я предложил создавать заведомо невалидную конфигурацию VLAN, чтобы оператор не смог записать ее в плату "как есть", не подумав и не изменив на действительно требуемую.

comment:11 by alx, 6 years ago

Component: swweb-интерфейс (sw)

comment:12 by alx, 6 years ago

Resolution: fixed
Status: newclosed

In 1640/sw:

В веб-интерфейсе при переключении плат GE-04 и PE-04 в режим IEEE 802.1,
если в таблице VLAN нет ни одной записи, автоматически создается пустая
(незаполненная) запись VLAN. Closes #278.

Note: See TracTickets for help on using tickets.