29236 2013-07-29 14:17:12 +0400 Некорректная остановка squid 2013-08-18 01:08:52 +0400 1 1 4 Development Sisyphus squid unstable all Linux CLOSED FIXED P3 normal --- 1 mav shaba egori led shaba qa-sisyphus oldest_to_newest 141876 0 mav 2013-07-29 14:17:12 +0400 В init-скрипте squid останавливается командой "stop_daemon --pidfile "$PIDFILE" --lockfile "$LOCKFILE" --expect-user squid -- $SQUID", из-за чего при повторном запуске он заново проверяет свой кэш. В процессе проверки squid не обслуживает запросы пользователей, а при достаточно большом объеме кэша проверка занимает значительное время (десятки минут). Вместо этого squid рекомендуется останавливать командой squid -k shutdown (в .service-файле для systemd, кстати, именно так и сделано) - при этом состояние кэша записывается на диск корректно и squid начинает работать без его проверки при старте. 141886 1 led 2013-07-30 08:16:05 +0400 В 3.2.13-alt2 вроде бы исправил. Когда дойдёт до репозитория посмотрите, пожалуйста, устраивает ли такое поведение (stop и restart) 141890 2 repository-robot 2013-07-30 11:26:36 +0400 squid-3.2.13-alt2 -> sisyphus: * Tue Jul 30 2013 Led <led@altlinux> 3.2.13-alt2 - squid.init: fixed stopping service (ALT#29236) 141936 3 mav 2013-07-31 13:54:03 +0400 Спасибо, поведение сильно лучше, чем раньше. :) Есть пара мыслей по этому поводу: 1. Есть смысл ввести обработку SQUID_SHUTDOWN_TIMEOUT из /etc/sysconfig/squid. В текущем варианте, если процесс squid помирать откажется (по разным причинам - сбой диска, например), мы получим бесконечное ожидание при его завершении. Предлагаю ждать определенное время, настраиваемое в /etc/sysconfig/squid. 2. Возможно, есть смысл обработать cache_effective_user из /etc/squid/squid.conf аналогично pid-файлу, и использовать полученное значение, вместо жесткого задания в squid. 142090 4 5906 mav 2013-08-14 15:36:54 +0400 Created attachment 5906 Патч на squid.init Вариант патча для squid.init, реализующий таймаут при остановке и обработку cache_effective_user 142092 5 mav 2013-08-14 15:38:22 +0400 Переоткрою багу, так как думаю, что обработка таймаута при остановке squid все-таки нужна. 142096 6 led 2013-08-14 19:09:41 +0400 (В ответ на комментарий №4) > Created an attachment (id=5906) [details] > Патч на squid.init > > Вариант патча для squid.init, реализующий таймаут при остановке и обработку > cache_effective_user С SQUIDUSER - согласен. С SQUID_SHUTDOWN_TIMEOUT=60 - не очень. Таймаут задаётся в конфиге параметром shutdown_lifetime - его и нужно использовать (или дефолтные 30 секунд, если не задан), а не абстрактные 60 секунд. Плюс 5 секунд накидываем сверху "на всякий случай" (по тестам получается, что нужно сверх 1-3 секунды). См. http://git.altlinux.org/people/led/packages/?p=squid.git;a=commit;h=89d9197e7d5217a775c3264f63d0a338165f266b Ну и следующий за ним http://git.altlinux.org/people/led/packages/?p=squid.git;a=commit;h=9b5cb51a1c09333a780686a3ace00bd375c12e6b в котором опять всё упрощаем и используем для остановки по таймауту стандартный stop_daemon(). При этом, если не дождались (почему-то), процесс прибивается с -9. Если конструктивных возражений не будет, этот вариант и пойдёт на сборку. 142097 7 mav 2013-08-14 19:19:11 +0400 По-моему, в http://git.altlinux.org/people/led/packages/?p=squid.git;a=commit;h=9b5cb51a1c09333a780686a3ace00bd375c12e6b все упростили настолько, что потеряли собственно цель - гасить squid с помощью squid -k shutdown. :) Предлагаю "упростить" только цикл остановки по таймауту, а функцию stop() начать с чего-то типа if start-stop-daemon -q --pid "$PIDFILE" --user $SQUID_USER --stop --test >/dev/null; then msg_stopping $"squid" $SQUID -f "$SQUID_CONF" -k shutdown *** далее остановка по таймауту *** fi 142098 8 led 2013-08-14 19:25:50 +0400 (В ответ на комментарий №7) > По-моему, в > http://git.altlinux.org/people/led/packages/?p=squid.git;a=commit;h=9b5cb51a1c09333a780686a3ace00bd375c12e6b > все упростили настолько, что потеряли собственно цель - гасить squid с помощью > squid -k shutdown. :) stop_daemon() делает ровно то же, что и "squid -k shutdown" - посылает процессу с указанным pid'ом сигнал TERM. Просто без --retry он, не дожидаясь, посылал сразу же KILL. Поэтому разницы никакой, зато на порядок проще и "стандартнее". 142099 9 mav 2013-08-14 19:38:44 +0400 (В ответ на комментарий №8) > stop_daemon() делает ровно то же, что и "squid -k shutdown" - посылает процессу > с указанным pid'ом сигнал TERM. Просто без --retry он, не дожидаясь, посылал > сразу же KILL. > Поэтому разницы никакой, зато на порядок проще и "стандартнее". Если так - тогда возражений нет. 142100 10 repository-robot 2013-08-14 21:38:43 +0400 squid-3.2.13-alt3 -> sisyphus: * Wed Aug 14 2013 Led <led@altlinux> 3.2.13-alt3 - squid.init: use config values for shutdown timeout and effective squid user (ALT#29236) 142238 11 mav 2013-08-16 16:27:36 +0400 (В ответ на комментарий №6) > (В ответ на комментарий №4) > С SQUID_SHUTDOWN_TIMEOUT=60 - не очень. > > Таймаут задаётся в конфиге параметром shutdown_lifetime - его и нужно > использовать (или дефолтные 30 секунд, если не задан), а не абстрактные 60 > секунд. Плюс 5 секунд накидываем сверху "на всякий случай" (по тестам > получается, что нужно сверх 1-3 секунды). А как Вы считаете, не следует ли абстрактные 5 секунд, накинутые сверху, заменить переменной типа SQUID_EXTRA_SHUTDOWN_TIMEOUT (вместо SQUID_SHUTDOWN_TIMEOUT, которую вообще убрать из /etc/sysconfig/squid), и дать возможность настраивать эти "на всякий случай накидываемые" секунды, с умолчанием в те же 5 секунд например? Случаи разные могут быть, 5 секунд может и не хватить (пример - параллельная остановка нескольких контейнеров OpenVZ, squid в одном из них. 5 секунд может не хватить для корректной остановки). PS Состояние баги не меняю, так как она таки RESOLVED. Если этот пакет скопировать в p7, то я его протестирую и базу закрою/переоткрою. 142240 12 led 2013-08-16 17:53:05 +0400 (В ответ на комментарий №11) http://git.altlinux.org/people/led/packages/?p=squid.git;a=commit;h=5b12e87c8a54916050aa3ce675ebe844c6c15fd2 * Fri Aug 16 2013 Led <led@altlinux.ru> 3.2.13-alt4 - squid.init: made SQUID_EXTRA_SHUTDOWN_TIMEOUT configurable 142250 13 mav 2013-08-18 01:08:52 +0400 Спасибо, работает замечательно. 5906 2013-08-14 15:36:54 +0400 2013-08-14 15:36:54 +0400 Патч на squid.init squid.init.patch text/plain 1698 mav LS0tIHNxdWlkLmluaXQub2xkCTIwMTMtMDctMzEgMTM6Mzg6NDUuNjQzMDg5MjgxICswNDAwCisr KyBzcXVpZC5pbml0CTIwMTMtMDgtMTQgMTU6MDk6MDYuMDE0OTQxNzAyICswNDAwCkBAIC0yOSw2 ICsyOSw3IEBACiAKIFNRVUlEPS91c3Ivc2Jpbi9zcXVpZAogU1FVSURfVUxJTUlUPTQwOTYKK1NR VUlEX1NIVVRET1dOX1RJTUVPVVQ9NjAKIFNRVUlEX0NPTkY9Ii9ldGMvc3F1aWQvc3F1aWQuY29u ZiIKIAogIyBPdmVyd3JpdGUgc29tZXRoaW5nCkBAIC0zOSw2ICs0MCw4IEBACiBMT0NLRklMRT0v dmFyL2xvY2svc3Vic3lzL3NxdWlkCiBQSURGSUxFPSIkKHNlZCAtcm4gJy9eW1s6Ymxhbms6XV0q cGlkX2ZpbGVuYW1lW1s6Ymxhbms6XV0vcy9eW1s6Ymxhbms6XV0qcGlkX2ZpbGVuYW1lW1s6Ymxh bms6XV0rKFteWzpibGFuazpdXSkrW1s6Ymxhbms6XV0qJC9cMS8nICIkU1FVSURfQ09ORiIpIgog UElERklMRT0iJHtQSURGSUxFOi0vdmFyL3J1bi9zcXVpZC5waWR9IgorU1FVSURVU0VSPSIkKHNl ZCAtcm4gJy9eW1s6Ymxhbms6XV0qY2FjaGVfZWZmZWN0aXZlX3VzZXJbWzpibGFuazpdXS9zL15b WzpibGFuazpdXSpjYWNoZV9lZmZlY3RpdmVfdXNlcltbOmJsYW5rOl1dKyhbXls6Ymxhbms6XV0p K1tbOmJsYW5rOl1dKiQvXDEvJyAiJFNRVUlEX0NPTkYiKSIKK1NRVUlEVVNFUj0iJHtTUVVJRFVT RVI6LXNxdWlkfSIKIFJFVFZBTD0wCiAKIHN0YXJ0KCkKQEAgLTYxLDE0ICs2NCwyMSBAQAogCiBz dG9wKCkKIHsKLQlsb2NhbCBwPSQoY2F0ICIkUElERklMRSIgMj4vZGV2L251bGwpCi0JaWYgc3Rh cnQtc3RvcC1kYWVtb24gLXEgLS1waWQgIiRQSURGSUxFIiAtLXVzZXIgc3F1aWQgLS1zdG9wIC0t dGVzdCA+L2Rldi9udWxsOyB0aGVuCisJbG9jYWwgcD0kKGNhdCAiJFBJREZJTEUiIDI+L2Rldi9u dWxsKSB0CisJaWYgc3RhcnQtc3RvcC1kYWVtb24gLXEgLS1waWQgIiRQSURGSUxFIiAtLXVzZXIg JFNRVUlEVVNFUiAtLXN0b3AgLS10ZXN0ID4vZGV2L251bGw7IHRoZW4KIAkJbXNnX3N0b3BwaW5n IHNxdWlkCiAJCSRTUVVJRCAtZiAiJFNRVUlEX0NPTkYiIC1rIHNodXRkb3duCiAJCVJFVFZBTD0k PwogCQlpZiBbICRSRVRWQUwgPSAwIF07IHRoZW4KLQkJCXdoaWxlIFsgIngkKExBTkc9QyBzdGF0 IC1jICIlVSAlRiIgL3Byb2MvJHAgMj4vZGV2L251bGwpIiA9ICJ4c3F1aWQgZGlyZWN0b3J5IiBd OyBkbworCQkJdD0wCisJCQl3aGlsZSBbICJ4JChMQU5HPUMgc3RhdCAtYyAiJVUgJUYiIC9wcm9j LyRwIDI+L2Rldi9udWxsKSIgPSAieCRTUVVJRFVTRVIgZGlyZWN0b3J5IiBdOyBkbwogCQkJCXNs ZWVwIDEKKwkJCQlpZiBbICR0IC1nZSAkU1FVSURfU0hVVERPV05fVElNRU9VVCBdOyB0aGVuCisJ CQkJCWVjaG8gLW4gInRpbWVvdXQhIgorCQkJCQlmYWlsdXJlCisJCQkJCXJldHVybiAxCisJCQkJ ZmkKKwkJCQlsZXQgdD0kdCsxCiAJCQlkb25lCiAJCQlybSAtZiAiJExPQ0tGSUxFIgogCQkJc3Vj Y2VzcwpAQCAtMTIwLDcgKzEzMCw3IEBACiAJCVsgLWUgIiRMT0NLRklMRSIgXSAmJiByZWxvYWQK IAkJOzsKIAlzdGF0dXMpCi0JCXN0YXR1cyAtLXBpZGZpbGUgIiRQSURGSUxFIiAtLWV4cGVjdC11 c2VyIHNxdWlkIHNxdWlkCisJCXN0YXR1cyAtLXBpZGZpbGUgIiRQSURGSUxFIiAtLWV4cGVjdC11 c2VyICRTUVVJRFVTRVIgc3F1aWQKIAkJUkVUVkFMPSQ/CiAJCTs7CiAJKikK