Skip to content

create separate cache directory for cli#8005

Merged
thrau merged 1 commit intomasterfrom
fix-cli-cache-dir
Mar 29, 2023
Merged

create separate cache directory for cli#8005
thrau merged 1 commit intomasterfrom
fix-cli-cache-dir

Conversation

@thrau
Copy link
Member

@thrau thrau commented Mar 29, 2023

This PR adds a separate cache dir for the CLI. I saw errors in the log:

[16:16:34] starting LocalStack in Docker mode 🐳                                                                                                                                                  localstack.py:142
2023-03-29T16:16:34.656  WARN --- [  MainThread] localstack.utils.functions : error calling function save: [Errno 13] Permission denied: '/home/thomas/.cache/localstack/cli'

because /home/thomas/.cache/localstack is created by the container. So instead, we now have /home/thomas/.cache/localstack-cli which is managed by the cli and won't have permission errors.

@thrau thrau requested a review from dfangl March 29, 2023 14:50
Copy link
Member

@dfangl dfangl left a comment

Choose a reason for hiding this comment

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

Seems sensible, LGTM!

@thrau thrau merged commit a23e539 into master Mar 29, 2023
@thrau thrau deleted the fix-cli-cache-dir branch March 29, 2023 16:19
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.

2 participants