| Re: Патчик для vzctl. [message #6930 is a reply to message #6493] |
Thu, 28 September 2006 05:28   |
Umka
Messages: 56
Registered: September 2006
|
Member |
|
|
Там несколько частей.
часть 1. Игорь просил разделить патчи - на что ответ простой.
части которые управляют разной функциональностью - выделены в отдельные функции внутри vps-functions
а) роутинг - функции vzaddsourcerouting/vzdelsourcerouting
б) шейпер - функции vzgetbw и vzsetuptcclass/vzflushtcclass и vzaddshaper/vzdelshaper
в) акаунтинг - vzaddipaccount/vzdelippaccount/vzgetipaccount
и общая функция vzsetupnet которая разбирает переданое окружение и /etc/vz/subnets
Откровенно не вижу сложностей в review 7кб патча на vps-functions.
Тем более там даже кой где коментарии стоят.
часть 2. Касающаяся добавления возможности использовать глобальные объявления и возможность выключения шейперов для конкретно взятого VPS. Требование логичное - будет реализовано.
часть 3. Обсуждение того что щас задаются шейпера по имени интерфейса, а Игорю хотелось бы что бы задавалось по имени группы. Ответ - такая возможность заложена в формат /etc/vz/subnet, но на момент первой публикации патча была не реализована. Собственно потребуется только выделение кода ответственного за формирование fw id для роутинга в отдельную функцию (для обеспечения повторного использования кода при формировании classid) и для формирования идентификатора tc classid использовать не формат devnum:veid, а заменить veid на вычисляемый classid.
часть 4. возможность отключения шейпера для отдельных классов - делается через паравку функции vzgetbw и обработку значения unlim в case.
Все эти пункты будут реализованы на следующей неделе.
PS не совсем понятная позиция SWSoft который вырезал реализацию шейперов из шеловой обвязки vzctl которая _уже_ существует в рамках Virtuozzo.
[Updated on: Thu, 28 September 2006 05:30] Report message to a moderator |
|
|
|
OpenVZ Forum
Members
Search
Help
Register
Login
Home
