21 lines
1.2 KiB
Markdown
21 lines
1.2 KiB
Markdown
# Причесать роль `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.
|