Skip to content

Temporary File Persistent Path Workaround Should Be Removed #9501

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

Open
dficker opened this issue Mar 28, 2025 · 3 comments · May be fixed by #9522
Open

Temporary File Persistent Path Workaround Should Be Removed #9501

dficker opened this issue Mar 28, 2025 · 3 comments · May be fixed by #9522
Assignees
Labels
topic: App Servers / Environments Issues around env variables and other things within applications containers Type: Fix Content Issue or PR to resolve incorrect information in the docs

Comments

@dficker
Copy link
Contributor

dficker commented Mar 28, 2025

Expected behavior

Actual behavior

  • It needs to be removed and/or has to have have the most dire message ever. Something like, "If you do this, your site will probably go down. Don't do this!"

Steps to reproduce the problem

  1. Doing this causes the file system to fill up with temporary files, and sooner or later things will break. The Pantheon file system is not designed for storage of temporary files.

Suggested Resolution

Additional information

@rachelwhitton
Copy link
Member

rachelwhitton commented Apr 9, 2025

@rachelwhitton to work with @djschaap to review and confirm the culprit is over populated files dir

"dir that's being set as tmp is not being treated as a tmp folder" it would be best if we recommended a path that the platform treated as tmp and auto cleaned up

@rachelwhitton rachelwhitton self-assigned this Apr 9, 2025
@rachelwhitton rachelwhitton added Type: Fix Content Issue or PR to resolve incorrect information in the docs topic: App Servers / Environments Issues around env variables and other things within applications containers labels Apr 9, 2025
@djschaap
Copy link
Member

This isn't necessarily related to an over-populated files directory (a dir that contains "too many" files) - but it could be related, sometimes.

The focus on temporary files suggests this is more of a rapid write situation - which is a separate issue.

I do agree this workaround is something we shouldn't be recommending, in general, due to the potential for it to create other (disastrous) problems down the road.

@rachelwhitton
Copy link
Member

Continued discussion internally here in this slack thread, resulting todo is to add a disclaimer into the recommendation saying "this is dangerous and may take down your site if the offending plugin is very active writing to the temporary files"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic: App Servers / Environments Issues around env variables and other things within applications containers Type: Fix Content Issue or PR to resolve incorrect information in the docs
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants