Closed Bug 1821063 Opened 2 years ago Closed 2 years ago

fix tempdir usage

Categories

(Tecken :: General, defect, P2)

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: willkg, Assigned: willkg)

References

Details

Attachments

(1 file)

Tecken has a UPLOAD_TEMPDIR_PREFIX which is uses in conjunction with whatever Python's gettempdir returns. Then in the infrastructure, /tmp is volume mounted because that's what Python's gettempdir returns. However, other things get stored in /tmp when Linux is running and it'd be better if Tecken kept all its temp stuff isolated from everything else.

We should switch to a configuration setting that specifies an absolute path to where Tecken can store its temporary stuff. Then we use that as the root path for generating new temporary directories for handling upload files. That way, we're using an absolute path, it's not intertwined with operating system things, and it's easier to reason about everything involved.

willkg merged PR #2700: "bug 1821063: fix temporary directory usage" in 974115e.

I'll wait until this deploys to stage and then verify it there.

I uploaded a bunch of symbols and watched the disk usage graph and everything looks good.

I pushed this out as part of bug #1825022 just now.

Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: