Skip to content
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

Action's cache dirs are stored in working directory #63

Closed
BjoernPetersen opened this issue Feb 23, 2023 · 2 comments · Fixed by #64
Closed

Action's cache dirs are stored in working directory #63

BjoernPetersen opened this issue Feb 23, 2023 · 2 comments · Fixed by #64
Milestone

Comments

@BjoernPetersen
Copy link

BjoernPetersen commented Feb 23, 2023

When using the v3 version of this action, I ran into an issue because the action stores its cache dirs docker-kube-config-[...] and helm-cache-[...] in the working directory. That will lead to an error 413 (Request Entity Too Large) when trying to install the chart in the working directory.

Anyone else affected reading this: add these lines to your .helmignore (create one if it doesn't exist) to work around this issue:

helm-cache-*/
docker-kube-config-*/
@WyriHaximus WyriHaximus added this to the v3.0.1 milestone Feb 23, 2023
@WyriHaximus
Copy link
Owner

Oof good point, I'll update

const dockerKubeConfigDir = process.cwd() + '/docker-kube-config-' + Math.random().toString(36).replace(/[^a-z]+/g, '').substr(0, 13);
fs.mkdirSync(dockerKubeConfigDir, {
mode: 0o777,
});
const dockerKubeConfig = dockerKubeConfigDir + '/config';
const helmCacheDir = process.cwd() + '/helm-cache-' + Math.random().toString(36).replace(/[^a-z]+/g, '').substr(0, 13);
to use the system tmp location.

@WyriHaximus
Copy link
Owner

v3.0.1 is out to fix this. Also found some other minor things to improve, but this one was causing issues, thank you for reporting 👍

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 a pull request may close this issue.

2 participants