Bug 50755 - Не работает выключение механизмов групповых политик
Summary: Не работает выключение механизмов групповых политик
Status: CLOSED FIXED
Alias: None
Product: Sisyphus
Classification: Development
Component: gpupdate (show other bugs)
Version: unstable
Hardware: x86_64 Linux
: P5 major
Assignee: Danila Skachedubov
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2024-06-26 12:36 MSK by Сергей Сысоев
Modified: 2024-06-27 17:07 MSK (History)
5 users (show)

See Also:


Attachments
пример исправления (279 bytes, patch)
2024-06-26 12:36 MSK, Сергей Сысоев
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Сергей Сысоев 2024-06-26 12:36:42 MSK
Created attachment 16336 [details]
пример исправления

Альт Рабочая Станция К 10.3, 6.1.94-un-def-alt1, все последние обновления
+ установлен gpupdate 0.10.3 из задания 351174.

Отключаем в групповой политике какой-либо из механизмов GPUpdate (например, Поддержка конфигурирования брандмауэра).

В /etc/dconf/db/policy.d/policy.ini видим как и должно быть:
[Software/BaseALT/Policies/GPUpdate]
FirewallApplier = 0

НО! Политики по брандмауэру всё равно применяются:
2024-06-26 13:09:14.383|[D00117]| Запуск применение настроек Firewall для машины|{}
2024-06-26 13:09:14.383|[D00119]| Firewall отключен, настройки будут сброшены|{}

Немного поизучали проблему, дошли до:
https://github.com/altlinux/gpupdate/blob/c57d1bac9e813d85663e29e02bfdc1a800411344/gpoa/frontend/applier_frontend.py#L52

Получается, что когда параметр не сконфигурирован (str("")) или параметр отключен (установлен в int(0)), проверка "if flag:" всегда ложная, и до нужной ветки result=false никогда не добраться.

Топорное, но рабочее решение - заменить
    if flag:
на 
    if len(str(flag))>0:

Прошу сразу собрать таск также в p10

Спасибо!
Comment 1 Сергей Сысоев 2024-06-26 13:15:40 MSK
Добвавлю, что проверка в функции check_windows_mapping_enabled(storage) скорей всего тоже некорректно работает:
    if flag and '0' == str(flag):

https://github.com/altlinux/gpupdate/blob/c57d1bac9e813d85663e29e02bfdc1a800411344/gpoa/frontend/applier_frontend.py#L40
Comment 2 Repository Robot 2024-06-27 17:07:41 MSK
gpupdate-0.10.4-alt1 -> sisyphus:

 Thu Jun 27 2024 Valery Sinelnikov <greh@altlinux> 0.10.4-alt1
 - Fixed the definition of the module activation check (closes: 50755)
 - Fixed sorting of scripts (closes: 50756)
 - Fixed reading key values from dconf
 - Changed the method for getting the list of packages for pkcon_runner