Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add getfacl and setfacl binaries #128

Open
pierreboissinot opened this issue Jan 17, 2024 · 3 comments · May be fixed by #129
Open

Add getfacl and setfacl binaries #128

pierreboissinot opened this issue Jan 17, 2024 · 3 comments · May be fixed by #129
Assignees

Comments

@pierreboissinot
Copy link
Member

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

@pierreboissinot pierreboissinot self-assigned this Jan 17, 2024
pierreboissinot added a commit that referenced this issue Jan 17, 2024
@pierreboissinot pierreboissinot linked a pull request Jan 17, 2024 that will close this issue
@thislg
Copy link
Member

thislg commented Jan 18, 2024

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
Copy link
Member Author

pierreboissinot commented Jan 18, 2024

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
Copy link
Member Author

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants