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

Fix install scripts to remove temp directories #258

Merged
merged 1 commit into from
Sep 8, 2023

Conversation

etremel
Copy link
Contributor

@etremel etremel commented Sep 8, 2023

According to Linux standards, the /var/tmp directory should be preserved across reboots and should not be deleted frequently by the OS, and many Linux distributions never delete files from /var/tmp. Installers and programs that use /var/tmp are supposed to clean up after themselves and delete the temporary directories they create in /var/tmp. Since our install scripts do not clean up after themselves, the Fractus nodes suffer from an ever-growing number of temporary directories in /var/tmp (each containing copies of libfabric, nlohmann_json, and mutils) that are not deleted automatically.

(See https://superuser.com/questions/168125, https://superuser.com/questions/499039/, and https://unix.stackexchange.com/questions/698677)

According to Linux standards, the /var/tmp directory should be preserved
across reboots and should not be deleted frequently by the OS, and many
Linux distributions never delete files from /var/tmp. Installers and
programs that use /var/tmp are supposed to clean up after themselves and
delete the temporary directories they create in /var/tmp. Since our
install scripts do not clean up after themselves, the Fractus nodes
suffer from an ever-growing number of temporary directories in /var/tmp
(each containing copies of libfabric, nlohmann_json, and mutils) that
are not deleted automatically.

(See https://superuser.com/questions/168125,
https://superuser.com/questions/499039/, and
https://unix.stackexchange.com/questions/698677)
@songweijia songweijia merged commit b6fc974 into master Sep 8, 2023
@etremel etremel deleted the fix_prerequisite_scripts branch September 8, 2023 20:16
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