search

nickjj / ansible-docker

Install / Configure Docker and Docker Compose using Ansible.
MIT License
750 stars 224 forks source link

ability to update apt's cache was added #127

Closed gwaewion closed 3 months ago

nickjj commented 5 months ago

Hi,

Thanks for the PR, have you checked this comment btw https://github.com/nickjj/ansible-docker/pull/59#issuecomment-513504141?

gwaewion commented 5 months ago

@nickjj I've read it and I get the point. But I suggest only ability to update apt's cache. If anyone wants to do that he will be able, but by default nobody is going to touch that functionality.

nickjj commented 5 months ago

I'm on the fence for this one because if we do it here it's saying every role should have this behavior. It really feels like a responsibility outside of a role installing and configuring a specific package.

If you don't have a dedicated bootstrap role, what about running the update in pre_tasks?

nickjj commented 3 months ago

Going to close this one, sorry.