opensafely-core / job-runner

A client for running jobs in an OpenSAFELY secure environment, requested via job-server (q.v.)
Other
4 stars 5 forks source link

Refactor VolumeAPI #748

Open bloodearnest opened 1 month ago

bloodearnest commented 1 month ago

Volumes are abstract objects that can be docker volumes proper, or bind mounts.

In various places we need the name, in some places we need the type of mount, and also the source/target.

It would be cleaner to have an instance of a Volume object that captures this abstraction better, rather than various calls to VolumeAPI.foo.. scattered throughout the codebase.