Skip to content

Fix: Use named volume for Kestra temp directory#794

Merged
wrussell1999 merged 1 commit intoDataTalksClub:mainfrom
dboyraz:main
Feb 6, 2026
Merged

Fix: Use named volume for Kestra temp directory#794
wrussell1999 merged 1 commit intoDataTalksClub:mainfrom
dboyraz:main

Conversation

@dboyraz
Copy link
Contributor

@dboyraz dboyraz commented Feb 2, 2026

Changes:

  • Replace bind mount /tmp/kestra-wd:/tmp/kestra-wd with named volume kestra_tmp:/tmp/kestra-wd
  • Add kestra_tmp to volumes definition

Reason:

  • Fixes "Permission denied" error when Kestra tries to execute Python interpreters
  • Bind mounts on Windows don't preserve Linux execute permissions
  • Named volumes are cross-platform compatible and avoid host filesystem permission conflicts

… issues

Changes:
- Replace bind mount `/tmp/kestra-wd:/tmp/kestra-wd` with named volume `kestra_tmp:/tmp/kestra-wd`
- Add `kestra_tmp` to volumes definition

Reason:
- Fixes "Permission denied" error when Kestra tries to execute Python interpreters
- Bind mounts on Windows don't preserve Linux execute permissions
- Named volumes are cross-platform compatible and avoid host filesystem permission conflicts
Copilot AI review requested due to automatic review settings February 2, 2026 14:17
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This pull request fixes a cross-platform compatibility issue with Kestra's temporary directory by replacing a bind mount with a named Docker volume. The change specifically addresses "Permission denied" errors on Windows where bind mounts don't preserve Linux execute permissions.

Changes:

  • Added kestra_tmp named volume definition to the volumes section
  • Replaced bind mount /tmp/kestra-wd:/tmp/kestra-wd with named volume kestra_tmp:/tmp/kestra-wd

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@alexeygrigorev
Copy link
Member

@wrussell1999

@wrussell1999 wrussell1999 merged commit 2b4ad5f into DataTalksClub:main Feb 6, 2026
5 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants