gitops-engine
gitops-engine copied to clipboard
Enable consumers of gitops-engine API to specify an alternative temp directory to '/dev/shm', for generated manifests
At present, gitops-engine (and consumers of gitops-engine API) use /dev/shm
to store temporary files that are passed to kubectl, such as manifests and cluster credentials. The temp directory logic can be found within pkg/utils/io/io.go
.
One disadvantage of this is that since /dev/shm
is backed by RAM, it has limited size. For many use cases this is not an issue, however, when concurrently deploying to a large number of Kubernetes clusters, or deploying large k8s manifests, /dev/shm
can run out of disk space.
One option is that we could just move to using /tmp
, rather than /dev/shm
. But, it appears that /dev/shm
was chosen intentionally to ensure that sensitive data (such as credentials) are kept off disk (absent disk caching of memory).
Thus, rather than moving to use /tmp
, I propose instead allowing API consumers to optionally use a different path, which can then by enabled by API consumers (for example, Argo CD configmap value or env var)
I've also been tackling this problem by reducing the leaking of files into /dev/shm
. You can see that work below.
Related issues:
- In 'ManageResources' and 'runResourceCommand', temporary files written to '/dev/shm' may not be cleaned up, in rare cases
- https://github.com/argoproj/gitops-engine/issues/568
- https://github.com/argoproj/gitops-engine/pull/569
- 'Apply/ReplaceResource' in resource_ops.go may leak files to '/dev/shm' since the kubectl 'apply/replace' commands never time out
- https://github.com/argoproj/gitops-engine/issues/572
- 'util/git/creds.go' will leak temporary files to '/dev/shm' if proxy URL cannot be parsed, or if write fails
- https://github.com/argoproj/argo-cd/issues/17658
- https://github.com/argoproj/argo-cd/pull/17659