Skip to content
Snippets Groups Projects

chore: add MediaConch profile symlinks

Closed Jens Steidl requested to merge add_mediaconch_symlinks into master

Bitte nochmal prüfen und ggf anpassen, Symlinks der default Plugin Parameter waren noch nicht Teil des Playbooks.

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Jens Steidl requested review from @sachse

    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

  • closed

Please register or sign in to reply
Loading