  - name: Примонтировать файл
    command: mount "/opt/{{ path_to_iso | basename }}" /media/cdrom
  - name: Скопировать пакеты 
    shell: cp -a /media/cdrom/* /srv/repo/{{ name_dir_repo }}

Серьезно? :)

MikeyTide Jul 5 at 23:46

То, что я не использовать модуль ?

Tamerlan666 Jul 5 at 23:38

      - name: создание ссылки на папку
        command: ln -s /srv/repo /var/www/html/
        ignore_errors: yes
      - name: Проверить наличие строки в файле /etc/apache2/sites-enabled/000-default.conf
        command: grep -q "<Directory /var/www/html/repo>" /etc/apache2/sites-enabled/000-default.conf
        register: grep_result
        changed_when: false
        ignore_errors: true 
      - name: Дописать в /etc/apache2/sites-enabled/000-default.conf
        command: sed -i '/DocumentRoot \/var\/www\/html/a \ \ \ \ <Directory \/var\/www\/html\/repo>\n \ \ \ \ \ \ Options Indexes MultiViews FollowSymLinks\n \ \ \ \ \ \ AllowOverride None\n \ \ \ \ \ \ Order Deny,Allow\n \ \ \ \ \ \ Require all granted\n \ \ \ \ <\/Directory>' /etc/apache2/sites-enabled/000-default.conf
        when: grep_result.rc != 0    
      - name: Измнение AstraMode on
        command: sed -i 's/# AstraMode on/AstraMode off/g'  /etc/apache2/apache2.conf

Вы решили поставить рекорд по количеству антипаттернов в статье с примерами кода Ansible?

MikeyTide Jul 5 at 23:46

То что я не кладу готовый кофеин в правлю его на удаленной тачке ?

Tamerlan666 Jul 6 at 06:24

Правка конфига вместо темплейта
Башсибл на пустом месте вместо стандартных модулей
ignore_errors без хендлинга исключений

MikeyTide Jul 6 at 07:08

Да согласен, что все можно было использовать.

Но тут и простая работа написана 2 года назад.

Я раньше не топил за красоту кода.

Щас я уже пишу нормальные роли:) просто нет времени переписать весь код. Об этом еще в 1 статье говорил. И про fqcn и так далее.

MikeyTide Jul 6 at 07:34

В планах конечно и переписать на питон.

И оформить в роли может быть некоторые таски.

Ну и добавить хендлеры, темплейты для конфигов.

Просто раньше писал, чтобы работало , так как хотел много функционала сразу реализовать.

Все замечания учитываю и поддерживаю , что best practices никто не отменял

past Jul 6 at 19:24

Почитайте про разницу между императивным и декларативным подходами. Bash, типичный представитель первого. Ansible, второго.

MikeyTide Jul 6 at 19:59

спасибо я знаю в чем разница.

Проекту более 2-х лет. Использования bash не более чем мое удобство, и не желания плодить множество файлов с шаблонами.

Данный код не виден конечному пользователю, а красота для меня стоит на втором месте, после функционала полезного конечному пользователю.

past Jul 7 at 08:44

Видимо, не до конца понимаете разницу, сравнивая теплое с мягким.
Если бы можно было бы сетапить сервера с помощью bash, не появились бы инструменты декларативного описания инфраструктуры.
Должен быть не просто чистый Ansible, а Ansible + тесты на Molecule. Тогда только появляется уверенность в том, что по исполнении произойдет именно то, чего ожидает разработчик.

MikeyTide Jul 7 at 09:36

Так я разработчик AstraWizard, я пишу то, что хочу.

Я делаю тесты и показываю их не видео.

Я не пишу что-то просто, чтобы делать тесты на молекуле.

Это не обычные роли или коллекции, которые легко проверить, учитывая всю мою обертку на баш и гуи.

Я не заявлял , что я пишу что-то на ansible , я сказал я его использую под капотом, чтобы удаленно выполнять то, что моя первая версия софта делала локально через баш.

Year Jul 8 at 11:12

Самое сомнительное в этом то, как передаются пароли на удаленную машину.
Например, ввод машины в виндовый домен предполагает ввод кредов, которые потом просто летят открытой строкой в вызов ансибла, откуда передаются на удаленную машину, где также открытой строкой попадают в вызов команды...

И вроде бы мы применяем ансибл, который со всем этим умеет достаточно безопасно обращаться, но ничего этого не применено.

MikeyTide Jul 8 at 11:31

Согласен, есть что доработать.
Но пароль не видно в открытом виде нигде.
History и логи не видно пароля, насколько я смотрел.

MikeyTide Jul 8 at 11:49

Можно оформить ПР. Я буду только рад развитию проекта с помощью сообщества