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

[Bug] Remove apt cache after apt-get #9825

Closed
areusch opened this issue Jan 3, 2022 · 3 comments
Closed

[Bug] Remove apt cache after apt-get #9825

areusch opened this issue Jan 3, 2022 · 3 comments
Assignees
Labels
type:ci Relates to TVM CI infrastructure

Comments

@areusch
Copy link
Contributor

areusch commented Jan 3, 2022

Per #9060 (comment), best practice for apt-get install in docker is to rm -rf /var/lib/apt/lists/* after installation. we should change all docker apt-get command to do this.

@areusch areusch added the type:ci Relates to TVM CI infrastructure label Jan 3, 2022
@areusch areusch self-assigned this Jan 3, 2022
@leandron
Copy link
Contributor

leandron commented Jan 6, 2022

Per #9060 (comment), best practice for apt-get install in docker is to rm -rf /var/lib/apt/lists/* after installation. we should change all docker apt-get command to do this.

Also it is implied by this change that all our docker/install/*.sh files should not assume there is an apt cache, and therefore, they should all apt-get update prior to installing any package.

@areusch
Copy link
Contributor Author

areusch commented Jan 25, 2022

maybe we should just write an apt wrapper that handles this and only apt-get update one per CI image build?

@driazati
Copy link
Member

driazati commented Aug 9, 2022

Fixed in #11470

@driazati driazati closed this as completed Aug 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:ci Relates to TVM CI infrastructure
Projects
None yet
Development

No branches or pull requests

3 participants