search

le-phare / le-phare.github.io

The Faros site
https://faros.lephare.com
4 stars 2 forks source link

Add getfacl and setfacl binaries #128

Open pierreboissinot opened 8 months ago

pierreboissinot commented 8 months ago

Some tasks from lephare/ansible-deploy role require ACL related binaries: setfacl and getfacl.

These tasks are skipped by default: https://github.com/le-phare/ansible-deploy/blob/641c82192a8efeffd2ac0abfc6c88f7997c82d73/defaults/main.yml#L63

thislg commented 8 months ago

Ce n'est pas activé par défaut et l'option est très peu utilisée donc je ne suis pas convaincu du besoin de mettre ça en prérequis.

pierreboissinot commented 8 months ago

Ce n'est pas activé par défaut et l'option est très peu utilisée donc je ne suis pas convaincu du besoin de mettre ça en prérequis.

Oui, ceci dit l'impact est faible; ce n'est pas comme si on installait un elasticsearch par défaut.

Ca ne me dérange pas car j'ajoute ce genre de binaires "optionnels" dans les pré-requis que j'envoie; autrement dit c'est la responsabilité du lead dev. C'est Vincent qui s'en es plaint à la base et je me disais que ce n'était pas grave de mettre ça en pré requis tout de mm.

@caillaudv tu es d'accord avec @thislg ?

pierreboissinot commented 5 months ago

@thislg je vous propose qu'on mentionne dans dans le readme ok pr vous ?