Summary: | При использовании veth etcnet-add_ip.sh этого не учитывает | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | Sisyphus | Reporter: | Slava Dubrovskiy <dubrsl> | ||||||
Component: | vzctl | Assignee: | Andrew Vasilyev <andy> | ||||||
Status: | CLOSED FIXED | QA Contact: | qa-sisyphus | ||||||
Severity: | enhancement | ||||||||
Priority: | P2 | CC: | andy | ||||||
Version: | unstable | ||||||||
Hardware: | all | ||||||||
OS: | Linux | ||||||||
Attachments: |
|
Description
Slava Dubrovskiy
2006-08-16 13:34:08 MSD
Created attachment 1589 [details]
Пропускает конфигурацию venet в VE для etcnet если не установлена IP_ADDR
Совсем пропускать запуск create_venet_config в setup_network тоже нельзя, поскольку setup_network выполняется только один раз при включении контейнера. (In reply to comment #2) > Совсем пропускать запуск create_venet_config в setup_network тоже нельзя, > поскольку setup_network выполняется только один раз при включении контейнера. Он (create_venet_config) устанавливает конфигурацию для venet, которая совершенно не нужна при использовании veth. Специально задавал вопрос на форуме http://forum.openvz.org/index.php?t=rview&goto=5223#msg_5223 Идеальный вариант, еслибы запускался create_veth_config в котором бы устанавливались настройки для veth. С этим veth еще непочатый край работы. Уже сделали в апстриме http://git.openvz.org/?p=vzctl;a=commit;h=32a571fe8861c8dbde388f8b61c396737bc936ae Вячеслав, вы могли бы сделать патч на etcnet-{add,del}_ip.sh по аналогии с процитированным изменением в upstream? (In reply to comment #5) > Вячеслав, вы могли бы сделать патч на etcnet-{add,del}_ip.sh по аналогии с > процитированным изменением в upstream? Ok. Только дома попробую. Пока на работе нет возможности работать с vz от ALT. Посмотрел 14 версию. Также просмотрел add_ip от разных дистрибутивов входящих в пакет. Или я не правильно понял или поправьте меня, но Upstream сделал только чтобы не добавлялся IP для venet если нет IP. Но сам интерфейс создается. Считаю это не правильно. Не логично и только сбивает с толку. Предлагаю такую логику: При старте проверяем наличие IP. если нет IP то даже не создавать venet. А при удалении IP проверять, остался ли хоть один IP и если нет, то удаляем venet. Скажите пожалуйста Ваше мнение? Нужен ли venet без IP? (In reply to comment #5) > Вячеслав, вы могли бы сделать патч на etcnet-{add,del}_ip.sh по аналогии с > процитированным изменением в upstream? Я вот сделал по аналогии. Но не очень вижу в этих изменениях необходимости, т.к. при отсутствии IP вообще не запускается add_ip.sh. Это проверяется в http://git.openvz.org/?p=vzctl;a=blobdiff;f=src/lib/net.c;h=426edfbf84ebdf02c02fcef35f9bd48a6b1a9ad5;hp=7754878c58523b67638e56bcbfb9f595ac0b07ff;hb=32a571fe8861c8dbde388f8b61c396737bc936ae;hpb=c332d0410878d3f4c1d18f87d8a621bc769addcf Считаю это не правильным, т.к. возможна ситуация, когда добавили IP, потом удалили при выключенном VE то настройки venet остаются и venet всегда настраивается. До тех пор, пока руками не удалишь /etc/net/venet0 внутри VE. Я бы запускал add_ip.sh всегда и в нем делал все проверки. Но не знаю как это отразится на других скриптах для других дистрибутивов. Created attachment 1842 [details] Удаление venet0 при старте VE если удаляется все IP (In reply to comment #5) > Вячеслав, вы могли бы сделать патч на etcnet-{add,del}_ip.sh по аналогии с > процитированным изменением в upstream? Я вот сделал по аналогии. Но не очень вижу в этих изменениях необходимости, т.к. при отсутствии IP вообще не запускается add_ip.sh. Это проверяется в http://git.openvz.org/?p=vzctl;a=blobdiff;f=src/lib/net.c;h=426edfbf84ebdf02c02fcef35f9bd48a6b1a9ad5;hp=7754878c58523b67638e56bcbfb9f595ac0b07ff;hb=32a571fe8861c8dbde388f8b61c396737bc936ae;hpb=c332d0410878d3f4c1d18f87d8a621bc769addcf Считаю это не правильным, т.к. возможна ситуация, когда добавили IP, потом удалили при выключенном VE то настройки venet остаются и venet всегда настраивается. До тех пор, пока руками не удалишь /etc/net/venet0 внутри VE. Я бы запускал add_ip.sh всегда и в нем делал все проверки. Но не знаю как это отразится на других скриптах для других дистрибутивов. Вот и баг повесили. http://bugzilla.openvz.org/show_bug.cgi?id=478 Я попробую сделать нормальный патч, чтобы работало при любых вариантах. Я тоже склонен откатить это изменение в src/lib/net.c, заменив его соответствующей проверкой в add_ip(), например, как в 3.0.14-alt1-5-g1be1f6a. etcnet-add_ip.sh я исправил, а вот с остальными *-add_ip.sh проблема осталась. |