ContentSquare / chproxy

Open-Source ClickHouse http proxy and load balancer
https://www.chproxy.org/
MIT License
1.29k stars 259 forks source link

fix: rely on OS temp directory #486

Closed vfoucault closed 4 days ago

vfoucault commented 4 days ago

Description

This is an issue to impose a `/tmp` as a default temporary directory, even thought this is the right one. On some case, you'd end up having a `/tmp` which is not writable (k8s: securityContext -> readOnlyRootFilesystem: true) and you'd prefer to have a way to override this. ## Pull request type Please check the type of change your PR introduces: - [x] Bugfix - [ ] Feature - [ ] Code style update (formatting, renaming) - [ ] Refactoring (no functional changes, no api changes) - [ ] Build related changes - [ ] Documentation content changes - [ ] Other (please describe): ### Checklist - [ ] Linter passes correctly - [ ] Add tests which fail without the change (if possible) - [ ] All tests passing - [ ] Extended the README / documentation, if necessary ## Does this introduce a breaking change?

Further comments