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

ability to update apt's cache was added #127

Closed
wants to merge 1 commit into from

Conversation

gwaewion
Copy link

No description provided.

@nickjj
Copy link
Owner

nickjj commented May 29, 2024

Hi,

Thanks for the PR, have you checked this comment btw #59 (comment)?

@gwaewion
Copy link
Author

@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
Copy link
Owner

nickjj commented May 31, 2024

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
Copy link
Owner

nickjj commented Jul 30, 2024

Going to close this one, sorry.

@nickjj nickjj closed this Jul 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants