Ответ
Да, регулярно. В DevOps-практике код инфраструктуры (IaC) и конфигурации так же важен, как и код приложения, поэтому ревью Pull Request'ов коллег — стандартная часть рабочего процесса.
Например, я проводил ревью изменений в Terraform-модулях для AWS, которые готовил коллега. Я проверял:
- Безопасность и соответствие стандартам: Нет ли хардкода секретов, правильно ли настроены Security Groups (минимально необходимые правила), используются ли корректные IAM-роли.
- Надежность и идемпотентность: Корректно ли используются
depends_on, не вызовет лиterraform applyнеожиданного разрушения ресурсов (например, пересоздания инстанса БД). - Масштабируемость и читаемость: Четко ли описаны переменные (variables.tf), нет ли излишней сложности, которую можно вынести в отдельный модуль.
Перед мержем PR я запускал terraform plan в тестовом окружении, чтобы убедиться, что изменения отрабатывают корректно и не конфликтуют с существующей инфраструктурой. После мержа я следил за пайплайном CI/CD (например, в GitLab CI), который применял эти изменения в staging-окружении, и проверял, что деплой прошел успешно и мониторинг не показывает аномалий.