Change "task log storage" to become "job-local file storage" #99409

New Issue

Sybren A. Stüvel · 2022-07-04T15:11:38+02:00

Sybren A. Stüvel commented

2022-07-04 15:11:38 +02:00

Some context: Before the "last-rendered image" feature, the only files saved locally on the Manager were the task logs. This was then expanded to become more of a "job-specific files" location, so that the last-rendered images also could be stored there.

This task:

- Refactor internal/manager/task_logs so that it uses internal/manager/local_storage to manage the file locations.
- Change the configuration option task_logs_path to something like local_storage_path, so that it matches the new semantics.

Some context: Before the "last-rendered image" feature, the only files saved locally on the Manager were the task logs. This was then expanded to become more of a "job-specific files" location, so that the last-rendered images also could be stored there. This task: - - [x] Refactor `internal/manager/task_logs` so that it uses `internal/manager/local_storage` to manage the file locations. - - [x] Change the configuration option `task_logs_path` to something like `local_storage_path`, so that it matches the new semantics.

Sybren A. Stüvel commented

2022-07-04 15:11:38 +02:00

Added subscriber: @dr.sybren

Sybren A. Stüvel self-assigned this 2022-07-05 17:26:57 +02:00

Sybren A. Stüvel commented

2022-07-05 17:26:57 +02:00

Changed status from 'Needs Triage' to: 'Confirmed'