OpenVZ Forum


Home » International » Russian » баг в vzctl ?
Re: баг в vzctl ? [message #7414 is a reply to message #7413] Fri, 13 October 2006 07:50 Go to previous messageGo to previous message
Igor Sukhih is currently offline  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

 
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Previous Topic: Количество процессов
Next Topic: /proc/meminfo, free и все остальное...
Goto Forum:
  


Current Time: Wed Aug 14 04:25:26 GMT 2024

Total time taken to generate the page: 0.02970 seconds