Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ansible 3 #8

Merged
merged 21 commits into from
Mar 22, 2018
Merged

Ansible 3 #8

merged 21 commits into from
Mar 22, 2018

Conversation

a-vorobyev
Copy link
Collaborator

ДЗ 12 сделано. Медленно и, надеюсь, верно )

Друзья, не нашел Юрия в списке, добавил Дмитрия.

Copy link

@Nklya Nklya left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Добрый день.

Андрей, условия ДЗ выполнены, все выглядит ок 👍

Мои комментарии ниже.

Усвоил практику разделения окружений и организации репозитория.

В основном работать пришлось с terrafrom скриптами. Сделал рефакторинг, теперь модули больше соответствуют правилам SOLID. Модуль создания машины отвечает только за создание машины, правила создаются отдельным модулем. В результате шаблон окружения выглядит целостно и из него контроллируются все атрибуты сайта. Также теперь оба окружения могут запускаться одновременно и не зависят друг от друга. Пришлось пожервовать статическим адресом, так как по лицензии он может быть только один. Могу выделить несколько негативных моментов при работе с HCL:

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Могу только добавить что в хашикорпе про эти моменты знают и считается, что это не баг а фича))
Т.е. они специально не хотят делать многие вещи в HCL.


- Условный оператор может возвращать только скалярные значения. Про листы и таблицы забудьте.

- Условное создание ресурса только через count.
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

И они его сломали в 0.11 частично))
hashicorp/terraform#16726


## Дополнительное задание 2

Всего 200 коммитов, и оно зеленое )
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Я забыл добавить в слайды trytravis как раз для отладки тревиса
https://medium.com/@Nklya/локальное-тестирование-в-travisci-2b5ef9adb16e


Усвоил практику разделения окружений и организации репозитория.

В основном работать пришлось с terrafrom скриптами. Сделал рефакторинг, теперь модули больше соответствуют правилам SOLID. Модуль создания машины отвечает только за создание машины, правила создаются отдельным модулем. В результате шаблон окружения выглядит целостно и из него контроллируются все атрибуты сайта. Также теперь оба окружения могут запускаться одновременно и не зависят друг от друга. Пришлось пожервовать статическим адресом, так как по лицензии он может быть только один. Могу выделить несколько негативных моментов при работе с HCL:
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

В stage и prod добавь пожалуйста terraform.tfvars.example чтобы было видно что и как создается.


## Дополнительное задание 1

Доработал решение с прошлого задания. Учел замечания, бинарники не храню в репозитории. При запуске вычекиваю в /tmp, там уже gradle собирает и выполняет. Вариант с хранием group_vars в папке окружения не применим с динамическим инвентарем, потому что формально файла инвентаря нет. Положил рядом с плейбуками, заодно проверил, что это работает и документация не врет )
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

То, что теперь нет бинарников 👍

Вариант с хранием group_vars в папке окружения не применим с динамическим инвентарем, потому что формально файла инвентаря нет.

Почему? Не вижу в этом проблем.

@Nklya Nklya added the _⭐_ label Mar 17, 2018
@a-vorobyev a-vorobyev merged commit 50560cb into master Mar 22, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants