40688 2021-08-09 17:56:32 +0300 apachectl2 переопределяет LANG, LANGUAGE и LC_ALL 2021-09-06 11:28:12 +0300 1 1 4 Development Sisyphus apache2-base unstable x86_64 Linux CLOSED FIXED P5 normal --- 1 slev rider egori rider qa-sisyphus oldest_to_newest 201363 0 slev 2021-08-09 17:56:32 +0300 Невозможно кастомизировать LC_ALL через опцию systemd юнита Environment. Например, FreeIPA использует drop-in конфиг для apache именно с такой опцией, из-за этого не работает часть функционала из коробки. Воспроизведение: - используется система с systemd. - по умолчанию в юните для httpd2 указано `Environment=LANG=C` или кастомный drop-in конфиг с идентичным `Environment=` - httpd получает LANG=LANGUAGE=LC_ALL=POSIX Причина: - apachechtl2 импортирует `/etc/init.d/functions`, в котором: ```bash if [ "$UID" = 0 ]; then LANG=POSIX LANGUAGE=POSIX LC_ALL=POSIX ``` `/usr/sbin/envvars` не поможет, так как импорт вышеуказанного файла происходит после импорта envvars. В настоящее время существует возможность кастомизировать указанные переменные _только_ через `EnvironmentFile=/etc/sysconfig/httpd2` или какой-то другой EnvironmentFile. Если так и задумано, то можно удалить из юнита httpd2.service Environment=LANG=C , чтобы не вводить пользователей в заблуждение, что это работает, и добавить полезный комментарий, как переопределить в АЛЬТЕ? 201364 1 slev 2021-08-09 17:58:49 +0300 Зачем это нужно описано тут http://blog.dscpl.com.au/2014/09/setting-lang-and-lcall-when-using.html 201389 2 slev 2021-08-10 10:47:46 +0300 > В настоящее время существует возможность кастомизировать указанные > переменные _только_ через `EnvironmentFile=/etc/sysconfig/httpd2` или > какой-то другой EnvironmentFile. > Неправильно выразился. Конечно, EnvironmentFile не работает также, как и Environment. Помогает установка переменных через /etc/sysconfig/httpd2 (сорсится apachectl2). 201391 3 rider 2021-08-10 10:57:44 +0300 Стас, сделай так как тебе нужно и удобно. Я приму изменения. 201392 4 slev 2021-08-10 11:01:27 +0300 Я бы, например, не использовал apachectl2 в systemd юните совсем, но это на уровне идеи. 201797 5 egori 2021-08-18 13:21:14 +0300 (In reply to Stanislav Levin from comment #4) > Я бы, например, не использовал apachectl2 в systemd юните совсем, но это на > уровне идеи. Убрал apachectl2 из юнит файла в таске 283135, apache запускается с правильным LANG. Стас, можешь проверить у себя пожалуйста. 201851 6 slev 2021-08-19 13:22:24 +0300 (Ответ для Egor Ignatov на комментарий #5) > (In reply to Stanislav Levin from comment #4) > > Я бы, например, не использовал apachectl2 в systemd юните совсем, но это на > > уровне идеи. > > Убрал apachectl2 из юнит файла в таске 283135, apache запускается с > правильным LANG. > Стас, можешь проверить у себя пожалуйста. Слепое(?) копирование патчей может быть опасно. Пожалуйста, укажите причину зачем нужен патч из Федоры и почему его нет в апстриме(если нет). 201852 7 slev 2021-08-19 13:25:07 +0300 Вы точно хотите включать mod_systemd на системах без systemd? 201853 8 rider 2021-08-19 14:12:24 +0300 Мы уже обсудили это с Егором и он сейчас переосмыслит сделанные изменения. 201857 9 egori 2021-08-19 17:05:12 +0300 (In reply to Stanislav Levin from comment #6) > Слепое(?) копирование патчей может быть опасно. > Пожалуйста, укажите причину зачем нужен патч из Федоры и почему его нет в > апстриме(если нет). Патч взят с апстрима федоры (httpd-2.4.43-mod_systemd.patch) и добавляет вывод доп информации в логи systemd. Он не является обязательным, но не вижу причин его не брать. (In reply to Stanislav Levin from comment #7) > Вы точно хотите включать mod_systemd на системах без systemd? Включение mod_systemd на системах с sysv, не делает ничего плохого кроме того, что добавляется runtime зависимость на libsystemd. А вот вынос его в отдельный пакет создает проблему с обновлением httpd.service. Могу, разве что, добавить post скрип убирающий автозагрузку mod_systemd для sysv. 201858 10 rider 2021-08-19 17:09:11 +0300 если модуль ничего плохого на sysvinit не сделает, то лучше если его отключит администратор. 201859 11 slev 2021-08-19 18:06:46 +0300 (Ответ для Egor Ignatov на комментарий #9) > (In reply to Stanislav Levin from comment #6) > > Слепое(?) копирование патчей может быть опасно. > > Пожалуйста, укажите причину зачем нужен патч из Федоры и почему его нет в > > апстриме(если нет). > Патч взят с апстрима федоры (httpd-2.4.43-mod_systemd.patch) и добавляет > вывод доп информации в логи systemd. > Он не является обязательным, но не вижу причин его не брать. А я не вижу причин его брать. Посмотрел историю патча, он только в федоре, без каких-либо привязок к багзилле или каким-либо тикетам. Не надо тащить то, что хз что исправляет/улучшает. > > (In reply to Stanislav Levin from comment #7) > > Вы точно хотите включать mod_systemd на системах без systemd? > Включение mod_systemd на системах с sysv, не делает ничего плохого кроме > того, что добавляется runtime зависимость на libsystemd. > А вот вынос его в отдельный пакет создает проблему с обновлением > httpd.service. > > Могу, разве что, добавить post скрип убирающий автозагрузку mod_systemd для > sysv. Лучше так и сделать. 201886 12 egori 2021-08-20 12:10:44 +0300 Собрал таск без патча, еще раз проверил на системах с systemd и sysv, все работает. Отправляю в сизиф. 201887 13 repository-robot 2021-08-20 12:50:50 +0300 apache2-1:2.4.48-alt5 -> sisyphus: Wed Aug 18 2021 Egor Ignatov <egori@altlinux> 1:2.4.48-alt5 - enable mod_systemd, drop apachectl2 in unit file (closes: #40688) 201995 14 slev 2021-08-24 12:33:07 +0300 Спасибо, проверил, оригинальной проблемы нет, но есть проблема при обновлении: filetrigger 90-apache2-base-httpd.filetrigger срабатывает раньше, чем systemd.filetrigger (который перечитывает юниты), поэтому первый должен это сделать самостоятельно или надо переименовать. 14: apache2-base-1:2.4.48-alt3.1 ####################################### [100%] Warning: The unit file, source configuration file or drop-ins of httpd2.service changed on disk. Run 'systemctl daemon-reload' to reload units. То есть после обновления нужно перезапустить httpd для применения изменений. Открыть новый тикет? 202354 15 slev 2021-09-06 11:28:12 +0300 Открыл https://bugzilla.altlinux.org/40861