chore: add MediaConch profile symlinks
Bitte nochmal prüfen und ggf anpassen, Symlinks der default Plugin Parameter waren noch nicht Teil des Playbooks.
Merge request reports
Activity
requested review from @sachse
assigned to @sachse
Hi Jens,
danke dir, ich find‘ den Commit inhaltlich gut und korrekt, hab‘ aber noch zwei kleine Formsachen, deswegen schließ‘ ich den Merge Request ohne Merge (was hoffentlich sowas wie „Reject“ bedeuten sollte , ich bin damit auch noch nicht so wahnsinnig fit).
- Der Task ab Zeile 50 braucht noch eine explizite Angabe des File Modes, z.B. ‘mode: “0755“‘, sonst meckert ansible-lint.
- Zumindest deine neuen Tasks sollten FQCNs (Fully Qualified Collection Names) für die Modulnamen verwenden, z.B. „ansible.builtin.file“. Das ist seit einer Weile empfohlen und wird demnächst auch durchgesetzt, wenn ich für die Rolle die CI gebaut hab‘. Ich wollte da jetzt nicht in deinem Branch rumpfuschen, deshalb geht das an dich zurück. Stell‘ mir gerne einfach einen neuen Merge Request, dann wink‘ ich den nachher durch.
Beste Grüße, Jörg
Please register or sign in to reply