samba-in-kubernetes / samba-operator

An operator for a Samba as a service on PVCs in kubernetes
Apache License 2.0
117 stars 24 forks source link

refactor how we set up volumes in our pods #107

Closed phlogistonjohn closed 3 years ago

phlogistonjohn commented 3 years ago

This is an attempt, perhaps quixotic, to simplify how we work with volumes in mounts in our pods. Starting by moving volume constructor calls into a new file.

Previously, we had to juggle volumes, volume mounts, the full list of volumes and the individual list of mounts. Now we return a simple struct tracking both volume and mount but unfortunately still need to track separate lists for to complete set of volumes (for the pod) and the individualized lists of mounts for each container.