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

Sops cannot find config file if it's named .sops.yml (instead of .sops.yaml) #1567

Closed
ErraticMaker opened this issue Jul 29, 2024 · 2 comments · Fixed by #1569
Closed

Sops cannot find config file if it's named .sops.yml (instead of .sops.yaml) #1567

ErraticMaker opened this issue Jul 29, 2024 · 2 comments · Fixed by #1569

Comments

@ErraticMaker
Copy link
Contributor

I don't know if it is a hard requirement that the sops config file is exactly .sops.yaml for it to be auto-discovered, or if it should accept the alternative .yml extension too, but it took me a while to realize this was the issue I was having.

So, it might be good to have a warning in the docs stating that it should match exactly to save some time to anyone facing the same issue. Or even better would be to allow the file to have either of those extensions.

I think the code where the name is defined to match that specific file name is here just for reference.

Thank you for the great work!

@felixfontein
Copy link
Contributor

You can always use --config to specify your config file, then it doesn't matter at all how it is named.

So, it might be good to have a warning in the docs stating that it should match exactly to save some time to anyone facing the same issue. Or even better would be to allow the file to have either of those extensions.

Mentioning that in the docs is the best way forward IMO; allowing .yml as well is something I would avoid.

@ErraticMaker
Copy link
Contributor Author

ErraticMaker commented Jul 30, 2024

Okay! Made a small PR (#1569). Feel free to leave comments or just close it an replace it if you feel there is a better phrasing or place to put the note.

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