Re: баг в vzctl ? [message #7439 is a reply to message #7438] |
Fri, 13 October 2006 11:51 ![Go to previous message Go to previous message](/theme/ovz3/images/up.png) |
Umka
Messages: 56 Registered: September 2006
|
Member |
|
|
kir wrote on Fri, 13 October 2006 07:36 | Я ещё раз всё перечитал. Ваш патч (в первом посте этого треда) изменяет функциональность vzctl так, что он не вносит изменения в конфиг, если они не приложились. Поправьте меня, если я это не так понял.
|
Почти. Только сформулировать иначе. Если функция применения параметров вернула ошибку - указав основному коду, что в текущей ситуаци параметры команды не валидны.
Quote: |
Такое изменение нежелательно по причинам, изложенным мной выше.
Да, проблема существует, но её нельзя однозначно решить таким патчем. Необходимо, видимо, разнести функции валидации параметров и функции приложения параметров, и изменить логику на следующую:
1. проверить параметры, при ошибке -- вываливаться
2. попытаться приложить параметры
3. (как и сейчас) безусловно записать параметры в конфиг.
|
4. заставить функции изменения параметров к VE разбираться - что есть фатальная ошибка (которая может повлиять на последующее поведение - в частности старт), а что не фатальная, которую можно игнорировать.
Реализовав такой алгоритм мы убираем "знание" о поведение отдельно взятой команды внутрь обработчики выполняющего эту команду. От него потребуется только одно - вернуть 0 если данные коректны и допустимы, или вернуть код ошибки которая идентифицирует проблему. Возможно это потребует правки в логике обработчиков - но IMHO это самый "дешевый" способ получить нужную функциональность.
|
|
|