handle working directories in base PulpTask
Creation and cleanup of `working_dir` should owned by the `PulpTask` class . Creation of a new `working_dir` should occur on `PulpTask.__call__()` and cleanup should occur on `PulpTask.on_success()` and `PulpTask.on_failure()`.
`PulpTask`'s child, `UserFacingTask` will need to be updated to use super() to pick up this functionality.
#1 Updated by bmbouter over 2 years ago
While working on sync() it was determined that very few tasks actually need working directories and it would be inefficient to have them created/deleted with every Task (which is what this issue is for). It probably is only sync() and publish(). Since those stories include the working dir create+delete we should close this as WONTFIX.
I'm waiting for another dev who agrees to actually close it.
#2 Updated by email@example.com over 2 years ago
- Status changed from NEW to CLOSED - NOTABUG
Agreed. This work is now done by this context manager:
Too bad we don't have a CLOSED-IRRELEVANT :)
Please register to edit this issue