1.2 KiB
1.2 KiB
Причесать роль 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 §5.
Связано: roles/owner, roles/eget (образец), roles/secrets.