georchestra / helm-georchestra

geOrchestra helm chart
3 stars 5 forks source link

Remove usage of emptyDir for datafeeder upload #80

Closed edevosc2c closed 5 months ago

edevosc2c commented 5 months ago

Doesn't make sense to use an emptyDir for that because the ephemeral filesystem of the container is already enough.

It's also using precious emptyDir disk space.

In the future, maybe useful to allow this to be configured in order to for example use a PVC so that it doesn't use the local disk space. Not sure if this directory is just for "temporary" files and these files are immediately deleted once the import is done.

edevosc2c commented 5 months ago

Confirmed by @pmauduit on slack that it's not worth to keep.

Not sure if this directory is just for "temporary" files and these files are immediately deleted once the import is done.

I created a separate issue here: https://github.com/georchestra/georchestra/issues/4140