Docs: собрать беклог задач из drafts
This commit is contained in:
@@ -0,0 +1,20 @@
|
||||
# Причесать роль `owner` под конвенции
|
||||
|
||||
**Приоритет:** низкий
|
||||
|
||||
Роль `owner` разошлась по стилю с `eget`/`secrets`:
|
||||
|
||||
- **`roles/owner/tasks/main.yml:2-10`** — валидация аргументов через `fail` +
|
||||
`when`, причём две задачи с **идентичным именем**. `eget` для того же делает
|
||||
`assert` (`roles/eget/tasks/main.yml:15`). Привести к одному стилю — `assert`
|
||||
либо декларативный `meta/argument_specs.yml`.
|
||||
- **`roles/owner/tasks/main.yml:32,53`** — устаревшие `with_items`/`with_dict`;
|
||||
конвенция — `loop` (`loop: "{{ owner_ssh_keys }}"`,
|
||||
`loop: "{{ owner_env_dict | dict2items }}"`).
|
||||
- У `owner` нет `meta/main.yml` и README, тогда как у `eget` и `secrets` есть.
|
||||
- Имена задач с точкой на конце (`"Prepare env variables."`) — ansible-lint в
|
||||
строгом профиле это ловит.
|
||||
|
||||
Контекст: [docs/drafts/ansible-review.md](../drafts/ansible-review.md) §5.
|
||||
|
||||
Связано: roles/owner, roles/eget (образец), roles/secrets.
|
||||
Reference in New Issue
Block a user