Re: баг в vzctl ? [message #7414 is a reply to message #7413] |
Fri, 13 October 2006 07:50 |
Igor Sukhih
Messages: 21 Registered: May 2006
|
Junior Member |
|
|
Umka wrote on Fri, 13 October 2006 03:29 |
Igor Sukhih wrote on Fri, 13 October 2006 03:06 | О каких ошибках идет речь, все ошибки должны преверятся на этапе парсинга параметров (или имеются в виду конфликты)
Если ВЕ не ранится для Вас уже не важно что в конфиг попадут "некоректные" параметры?
|
Собственно в оригинальном варианте у вас есть 3 точки в которых может быть вызван vzerror.
Quote: |
# grep vzerror vps-functions
vzerror "Missing parameter: $VAR" $VZ_INVALID_PARAMETER_SYNTAX
vzerror "Unable to get source ip [${device}]" $VZ_CANT_ADDIP
vzerror "Unable to add route ${IP_CMD} route add $1 dev venet0 src ${src_addr}" $VZ_CANT_ADDI
|
кроме того это функции vzcheckvar, которые присутствуют внутри каждого из shell сприптов. Да и мало ли по какое еще причине надо выйти из shell скрипта с указазанием ошибки. Я не прав?
|
Из ошибки в шел скрипте не всегда следует что этот параметр неверный и его не следует сохранять,
как вы правильно заметили его (скрипт) могут вообше прибить, и мы получаем, чтобы сохранит параметры нужно
стопать ВЕ.
На данный момент ip адрес является единственным параметром который откатывается при ошибе.
Quote: |
Кроме того любое нарушение формата логичнее проверять в той точке где оно обрабатывается. Или я не прав?
|
Да прав, потому они обрабатываются на входе т.е. командная строка/конфиг.
Quote: |
А обрабатываются настройки шейпера у меня в шел скриптах - поэтому я добавил проверок при создании объектов шейпера, На что собственно и накололся сегодня утром - когда для теста скормил строку с неправильным форматом. Результатом был вывод error`а на экран и испорченый конфиг.
|
Я понял Вашу пробл., но это не тот фикс который нужен.
[Updated on: Fri, 13 October 2006 10:17] by Moderator Report message to a moderator
|
|
|