9886 2006-08-16 13:34:08 +0400 При использовании veth etcnet-add_ip.sh этого не учитывает 2007-03-04 20:13:38 +0300 1 1 4 Development Sisyphus vzctl unstable all Linux CLOSED FIXED P2 enhancement --- 1 dubrsl andy andy qa-sisyphus oldest_to_newest 39582 0 dubrsl 2006-08-16 13:34:08 +0400 При использовании veth etcnet-add_ip.sh все равно настраивает venet внутри VE и совершенно не учитывает возможность использования veth. Т.е. поднимает venet0 и устанавливает default маршрут. Предлагаю изменить скрипт так, чтобы проводилась проверка на наличие переменной veth в конфиге и проводилась соответствующая настройка. 39587 1 1589 dubrsl 2006-08-16 16:52:40 +0400 Created attachment 1589 Пропускает конфигурацию venet в VE для etcnet если не установлена IP_ADDR 39598 2 ldv 2006-08-17 00:15:49 +0400 Совсем пропускать запуск create_venet_config в setup_network тоже нельзя, поскольку setup_network выполняется только один раз при включении контейнера. 39599 3 dubrsl 2006-08-17 10:12:53 +0400 (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 еще непочатый край работы. 42957 4 dubrsl 2006-12-12 12:08:06 +0300 Уже сделали в апстриме http://git.openvz.org/?p=vzctl;a=commit;h=32a571fe8861c8dbde388f8b61c396737bc936ae 45665 5 ldv 2007-02-21 14:50:27 +0300 Вячеслав, вы могли бы сделать патч на etcnet-{add,del}_ip.sh по аналогии с процитированным изменением в upstream? 45671 6 dubrsl 2007-02-21 15:07:03 +0300 (In reply to comment #5) > Вячеслав, вы могли бы сделать патч на etcnet-{add,del}_ip.sh по аналогии с > процитированным изменением в upstream? Ok. Только дома попробую. Пока на работе нет возможности работать с vz от ALT. 45747 7 dubrsl 2007-02-22 15:38:10 +0300 Посмотрел 14 версию. Также просмотрел add_ip от разных дистрибутивов входящих в пакет. Или я не правильно понял или поправьте меня, но Upstream сделал только чтобы не добавлялся IP для venet если нет IP. Но сам интерфейс создается. Считаю это не правильно. Не логично и только сбивает с толку. Предлагаю такую логику: При старте проверяем наличие IP. если нет IP то даже не создавать venet. А при удалении IP проверять, остался ли хоть один IP и если нет, то удаляем venet. Скажите пожалуйста Ваше мнение? Нужен ли venet без IP? 45986 8 dubrsl 2007-02-28 13:12:10 +0300 (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 всегда и в нем делал все проверки. Но не знаю как это отразится на других скриптах для других дистрибутивов. 45987 9 1842 dubrsl 2007-02-28 13:15:27 +0300 Created attachment 1842 Удаление 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 всегда и в нем делал все проверки. Но не знаю как это отразится на других скриптах для других дистрибутивов. 45990 10 dubrsl 2007-02-28 15:22:23 +0300 Вот и баг повесили. http://bugzilla.openvz.org/show_bug.cgi?id=478 Я попробую сделать нормальный патч, чтобы работало при любых вариантах. 46062 11 ldv 2007-03-02 04:04:35 +0300 Я тоже склонен откатить это изменение в src/lib/net.c, заменив его соответствующей проверкой в add_ip(), например, как в 3.0.14-alt1-5-g1be1f6a. 46146 12 ldv 2007-03-04 20:13:37 +0300 etcnet-add_ip.sh я исправил, а вот с остальными *-add_ip.sh проблема осталась. 1589 2006-08-16 16:52:40 +0400 2006-08-16 16:52:40 +0400 Пропускает конфигурацию venet в VE для etcnet если не установлена IP_ADDR skip-network-if-no-IP.patch text/plain 416 dubrsl LS0tIGV0Y25ldC1hZGRfaXAuc2gub3JnCTIwMDYtMDgtMTUgMTc6NTU6NTggKzA0MDAKKysrIGV0 Y25ldC1hZGRfaXAuc2gJMjAwNi0wOC0xNiAxNTo0MTozMSArMDQwMApAQCAtNjYsOCArNjYsMTAg QEAgVFlQRT1sbycgPiIkZGlyL29wdGlvbnMiIHx8CiAKIHNldHVwX25ldHdvcmsoKQogewotCSMg U2V0IHVwIHZlbmV0MCBtYWluIGludGVyZmFjZQotCWNyZWF0ZV92ZW5ldF9jb25maWcgIiRWRU5F VF9ERVYiCisJaWYgWyAtbiAiJElQX0FERFIiIF07IHRoZW4KKwkgICAgIyBTZXQgdXAgdmVuZXQw IG1haW4gaW50ZXJmYWNlCisJICAgIGNyZWF0ZV92ZW5ldF9jb25maWcgIiRWRU5FVF9ERVYiCisJ ZmkKIAogCSMgU2V0IC9ldGMvc3lzY29uZmlnL25ldHdvcmsKIAlwdXRfcGFyYW0gIiRORVRGSUxF IiBORVRXT1JLSU5HIHllcwo= 1842 2007-02-28 13:15:27 +0300 2007-02-28 13:15:27 +0300 Удаление venet0 при старте VE если удаляется все IP vzctl-3.0.14-alt1-etcnet_add_ip.patch text/plain 1007 dubrsl ZGlmZiAtTnVycFAgdnpjdGwtMy4wLjE0LWFsdDFfb3JnL2V0Yy9kaXN0cy9zY3JpcHRzL2V0Y25l dC1hZGRfaXAuc2ggdnpjdGwtMy4wLjE0LWFsdDEvZXRjL2Rpc3RzL3NjcmlwdHMvZXRjbmV0LWFk ZF9pcC5zaAotLS0gdnpjdGwtMy4wLjE0LWFsdDFfb3JnL2V0Yy9kaXN0cy9zY3JpcHRzL2V0Y25l dC1hZGRfaXAuc2gJMjAwNy0wMi0xOSAxNzo0NToyMCArMDIwMAorKysgdnpjdGwtMy4wLjE0LWFs dDEvZXRjL2Rpc3RzL3NjcmlwdHMvZXRjbmV0LWFkZF9pcC5zaAkyMDA3LTAyLTI1IDAwOjU4OjQ1 ICswMjAwCkBAIC0xMTYsMTEgKzExNiwyMSBAQCBmaW5kX3VudXNlZF9hbGlhcygpCiBhZGRfaXAo KQogewogCWxvY2FsIGkgaXAKIAorCSMgSW4gY2FzZSB3ZSBhcmUgc3RhcnRpbmcgVkUKIAlpZiBb ICIkVkVfU1RBVEUiID0gInN0YXJ0aW5nIiBdOyB0aGVuCisJCSMgUmVtb3ZlIGFsbCBWRU5FVCBj b25maWcgZmlsZXMKKwkJcm0gLWZyICIkVkVORVRfREVWIiA+L2Rldi9udWxsIDI+JjEKKwlmaQor CWlmIFsgISAtZCAiJFZFTkVUX0RFViIgXTsgdGhlbgogCQlzZXR1cF9uZXR3b3JrCiAJZmkKLQor ICAgICAgICBpZiBbICIke0lQREVMQUxMfSIgPSAieWVzIiBdOyB0aGVuCisgICAgICAgICAgICAg ICAgaWZkb3duICR7VkVORVRfREVWfSA+L2Rldi9udWxsIDI+JjEKKwkJcm0gLWZyICIkVkVORVRf REVWIiA+L2Rldi9udWxsIDI+JjEKKyAgICAgICAgZmkKKwkJCQkJCQogCWJhY2t1cF9jb25maWdz ICIkSVBERUxBTEwiCiAKIAlpPTAKQEAgLTEzMyw4ICsxNDMsOCBAQCBhZGRfaXAoKQogCiAJaWYg WyAiJFZFX1NUQVRFIiA9ICJydW5uaW5nIiBdOyB0aGVuCiAJCSMgc3luY2hyb255emUgY29uZmln IGZpbGVzICYgaW50ZXJmYWNlcwotCQlpZmRvd24gIiRWRU5FVF9ERVYiCi0JCWlmdXAgIiRWRU5F VF9ERVYiCisJCWlmZG93biAke1ZFTkVUX0RFVn0gID4vZGV2L251bGwgMj4mMQorCQlpZnVwICR7 VkVORVRfREVWfSAgPi9kZXYvbnVsbCAyPiYxCiAJZmkKIH0KIAo=