-
Notifications
You must be signed in to change notification settings - Fork 237
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: configure temporary directory for local execute #967
feat: configure temporary directory for local execute #967
Conversation
7d70984
to
431ba76
Compare
Thanks for making this PR -- it seems to resolve the "/tmp/local_build_config.yml: is a directory" issue with Rancher Desktop. |
431ba76
to
6903815
Compare
Would like to see this change go in, or if someone could point me to the workaround. I'm on Intel Mac and Rancher Desktop. |
+1 this would solve issues I see related to #716 |
6903815
to
b9e1e9f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Glad this finally gets out!
Checklist
=========
Internal Checklist
Changes
During
circleci local execute
, make it possible to configure temporary directory to store config file.Rationale
This was needed to support alternative (aka "not Docker") container distributions on macOS, particularly lima-based distributions, such as colima or Rancher Desktop, that do not support mounting
/tmp
(or other "system paths") to their Linux VM.Considerations
local execute
, or inside thecli.yml
settings file with thetemp_dir
property, otherwise it defaults to the system's temporary dir/tmp
, instead reverting to system temp dir, as Docker Desktop now adds/var/folders
to its mount paths