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

aws cli cache directory unwritable #380

Open
ryan-dyer-sp opened this issue Apr 29, 2024 · 2 comments · May be fixed by #385
Open

aws cli cache directory unwritable #380

ryan-dyer-sp opened this issue Apr 29, 2024 · 2 comments · May be fixed by #385
Labels
enhancement New feature or request

Comments

@ryan-dyer-sp
Copy link

The default aws cli cache directory (/home/atlantis/.aws/cli) is unwritable as the aws secret is mounted as as the entire /home/atlantis/.aws directory.

When dealing with lots of profiles, etc. it is nice to have the aws cli tool available in atlantis (we build our own image w/ additional toolsets installed). But the helm chart's existing method for managing the aws secret doesnt allow the aws cli to work out of the box.

I will submit a PR shortly to iterate through the aws objects and mount them individually as files within the .aws directory instead.

@GMartinez-Sisti
Copy link
Member

Hi @ryan-dyer-sp, as replied on our Community Slack:

Hi Ryan, thanks in advance for the contribution 😄 . I’d say it’s best to send the PR and then we can review it. One important detail is to ensure there are no breaking changes, otherwise it needs to be explained with migration steps.

@GMartinez-Sisti GMartinez-Sisti added the enhancement New feature or request label May 4, 2024
@GMartinez-Sisti
Copy link
Member

Would a similar approach work for this? #384

@ryan-dyer-sp ryan-dyer-sp linked a pull request May 7, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants