Last modified: 2012-07-30 17:54:15 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T39058, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 37058 - TMH need a way to configure temporary path
TMH need a way to configure temporary path
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
TimedMediaHandler (Other open bugs)
unspecified
All All
: Normal major (vote)
: ---
Assigned To: Michael Dale
:
Depends on: 37048
Blocks:
  Show dependency treegraph
 
Reported: 2012-05-23 15:52 UTC by Antoine "hashar" Musso (WMF)
Modified: 2012-07-30 17:54 UTC (History)
3 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Antoine "hashar" Musso (WMF) 2012-05-23 15:52:46 UTC
TMH is going to transcoder video using a job which use MediaWiki temporary directory as a working directory.

The job is WebVideoTranscode, definition of path is done by getTargetEncodePath(). It eventually call TempFSFile::factory() to get a temporary file.

This part of the code need to be adapted so we could set the working directory. For example, to use a dedicated disk partition.
Comment 1 Michael Dale 2012-06-13 19:20:57 UTC
I am guessing you want to support hard coded path override ?  Or a more correct way I suppose would be to extend the TempFSFile Class with namespaced paths per application domains ? 

But it seems that it would be simpler if these transcode boxes are only going to be transcoding, then why not just change the temporary path configuration for those instances?
Comment 2 Antoine "hashar" Musso (WMF) 2012-06-13 19:42:12 UTC
The functions I have described above ultimately call wfTempDir().

With https://gerrit.wikimedia.org/r/8996 for bug 24985, wfTempDir() now uses $wgTmpDirectory when available. So we can take tweak our CommonSettings.php file to change the temp path :-]

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links