Bug 40688 - apachectl2 переопределяет LANG, LANGUAGE и LC_ALL
Summary: apachectl2 переопределяет LANG, LANGUAGE и LC_ALL
Status: CLOSED FIXED
Alias: None
Product: Sisyphus
Classification: Development
Component: apache2-base (show other bugs)
Version: unstable
Hardware: x86_64 Linux
: P5 normal
Assignee: Anton Farygin
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-08-09 17:56 MSK by Stanislav Levin
Modified: 2021-09-06 11:28 MSK (History)
2 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Stanislav Levin 2021-08-09 17:56:32 MSK
Невозможно кастомизировать 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
, чтобы не вводить пользователей в заблуждение, что это работает, и добавить полезный комментарий, как переопределить в АЛЬТЕ?
Comment 1 Stanislav Levin 2021-08-09 17:58:49 MSK
Зачем это нужно описано тут
http://blog.dscpl.com.au/2014/09/setting-lang-and-lcall-when-using.html
Comment 2 Stanislav Levin 2021-08-10 10:47:46 MSK
> В настоящее время существует возможность кастомизировать указанные
> переменные _только_ через `EnvironmentFile=/etc/sysconfig/httpd2` или
> какой-то другой EnvironmentFile.
> 

Неправильно выразился. Конечно, EnvironmentFile не работает также, как и Environment. Помогает установка переменных через /etc/sysconfig/httpd2 (сорсится apachectl2).
Comment 3 Anton Farygin 2021-08-10 10:57:44 MSK
Стас, сделай так как тебе нужно и удобно.
Я приму изменения.
Comment 4 Stanislav Levin 2021-08-10 11:01:27 MSK
Я бы, например, не использовал apachectl2 в systemd юните совсем, но это на уровне идеи.
Comment 5 Egor Ignatov 2021-08-18 13:21:14 MSK
(In reply to Stanislav Levin from comment #4)
> Я бы, например, не использовал apachectl2 в systemd юните совсем, но это на
> уровне идеи.

Убрал apachectl2 из юнит файла в таске 283135, apache запускается с правильным LANG.
Стас, можешь проверить у себя пожалуйста.
Comment 6 Stanislav Levin 2021-08-19 13:22:24 MSK
(Ответ для Egor Ignatov на комментарий #5)
> (In reply to Stanislav Levin from comment #4)
> > Я бы, например, не использовал apachectl2 в systemd юните совсем, но это на
> > уровне идеи.
> 
> Убрал apachectl2 из юнит файла в таске 283135, apache запускается с
> правильным LANG.
> Стас, можешь проверить у себя пожалуйста.

Слепое(?) копирование патчей может быть опасно.
Пожалуйста, укажите причину зачем нужен патч из Федоры и почему его нет в апстриме(если нет).
Comment 7 Stanislav Levin 2021-08-19 13:25:07 MSK
Вы точно хотите включать mod_systemd на системах без systemd?
Comment 8 Anton Farygin 2021-08-19 14:12:24 MSK
Мы уже обсудили это с Егором и он сейчас переосмыслит сделанные изменения.
Comment 9 Egor Ignatov 2021-08-19 17:05:12 MSK
(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.
Comment 10 Anton Farygin 2021-08-19 17:09:11 MSK
если модуль ничего плохого на sysvinit не сделает, то лучше если его отключит администратор.
Comment 11 Stanislav Levin 2021-08-19 18:06:46 MSK
(Ответ для 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.
Лучше так и сделать.
Comment 12 Egor Ignatov 2021-08-20 12:10:44 MSK
Собрал таск без патча, еще раз проверил на системах с systemd и sysv, все работает. Отправляю в сизиф.
Comment 13 Repository Robot 2021-08-20 12:50:50 MSK
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)
Comment 14 Stanislav Levin 2021-08-24 12:33:07 MSK
Спасибо, проверил, оригинальной проблемы нет, но есть проблема при обновлении:

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 для применения изменений.
Открыть новый тикет?
Comment 15 Stanislav Levin 2021-09-06 11:28:12 MSK
Открыл https://bugzilla.altlinux.org/40861