action #162044
closedopenQA Project (public) - coordination #112862: [saga][epic] Future ideas for easy multi-machine handling: MM-tests as first-class citizens
openQA Project (public) - coordination #111929: [epic] Stable multi-machine tests covering multiple physical workers
Ensure proper indentation yamllint checks in our salt-states-openqa+salt-pillars-openqa size:S
0%
Description
Motivation¶
See
https://gitlab.suse.de/openqa/salt-pillars-openqa/-/merge_requests/832?commit_id=8412fb44dee81426a40e1f114aed3e62ca7e0667#note_633754
With the merge request we align both repos to use the same YAML style rules but it would be better to explicitly check for indendation. So we should enable indendation check, at best by removing any customization in the yamllint config file, in both salt-states-openqa+salt-pillars-openqa and apply the according necessary style adaptations in both repos.
Acceptance criteria¶
- AC1: Line https://gitlab.suse.de/openqa/salt-states-openqa/-/blob/master/.yamllint?ref_type=heads#L6 is absent
- AC2: yamllint ensures proper indendation in salt-states-openqa
- AC3: Same as AC1+AC2 but for salt-pillars-openqa
Suggestions¶
- Remove https://gitlab.suse.de/openqa/salt-states-openqa/-/blob/master/.yamllint?ref_type=heads#L6, call yamllint, see where it fails and adapt the code
- As necessary add a custom rule but at best we can just live with the default
- Apply the same for salt-pillars-openqa