Actions
action #162044
openopenQA Project - coordination #112862: [saga][epic] Future ideas for easy multi-machine handling: MM-tests as first-class citizens
openQA Project - coordination #111929: [epic] Stable multi-machine tests covering multiple physical workers
Ensure proper indendation yamllint checks in our salt-states-openqa+salt-pillars-openqa
Start date:
2024-06-03
Due date:
% Done:
0%
Estimated time:
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: 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+AC3 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
Updated by okurz 20 days ago
- Copied from action #161396: gitlab CI jobs do not fail on invalid YAML size:S added
Actions