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: Slow startup and performance on NFS or slow volume mounts #188 #189

Closed
wants to merge 3 commits into from

Conversation

capile
Copy link
Contributor

@capile capile commented Nov 27, 2024

Addresses issue #188

Adds a bit of documentation on how to enable a different ATTACHMENTS_DIR and what other folders need to be mounted instead of the /var/www/MISP/app/files.

Places the attachments folder outside of the files/ structure, that will remain basically the same as the distribution, with the exception of files/img/and files/terms/, folders that should be affected by a NFS filesystem.

During initialization, checks if the files/ folder was mounted before performing any tasks on it.

FILES_VERSION=
MISP_APP_FILES_PATH=/var/www/MISP/app/files
if [ -f ${MISP_APP_FILES_PATH}/VERSION ]; then
FILES_VERSION=$(cat ${MISP_APP_FILES_PATH}/VERSION)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you add a line echoing the parsed version?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ola Stefano, sure, but I messed up and used the master branch instead of an issue, and now it's all messed up -- I'm re-creating this PR with the echo and the right branch

Please ignore the other updates on this PR

Copy link
Collaborator

@ostefano ostefano left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As per comment

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