Skip to content

Settings: rewrite #154

Open
Open
@saltydk

Description

Clean up the settings role to use dynamic include_tasks instead of the current implementation that is fairly spammy and confusing to debug.

Current implementation:

- name: "Settings | Migrator"
  ansible.builtin.include_tasks: "subtasks/migrator.yml"
  vars:
    file: "{{ outer_item }}"

Initial suggestion would be:

- name: "Settings | Migrator"
  ansible.builtin.include_tasks: "subtasks/migrator/{{ outer_item }}.yml"

This would avoid looping over tasks that aren't going to do anything over and over.

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions