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

Create /var/spool/postfix/etc/ and /var/spool/postfix/private/ if they don't exist. #55

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from

Conversation

jroose
Copy link

@jroose jroose commented Feb 3, 2024

🗣 Description

This change creates /var/spool/postfix/etc and /var/spool/postfix/private if they do not exist.

💭 Motivation and context

The documentation recommends using /var/spool/postfix as a volume, but if an empty directory gets mounted there, then the ./etc and ./private directories don't exist and the entrypoint fails.

🧪 Testing

I just built the container, pushed it to my registry, and ran it on my K8S cluster. It no longer threw an error.

✅ Pre-approval checklist

  • This PR has an informative and human-readable title.
  • Changes are limited to a single goal - eschew scope creep!
  • All relevant type-of-change labels have been added.
  • I have read the CONTRIBUTING document.
  • These code changes follow cisagov code standards.
  • All new and existing tests pass.

✅ Pre-merge checklist

  • Finalize version.

✅ Post-merge checklist

  • Create a release.

@jsf9k jsf9k added the improvement This issue or pull request will add or improve functionality, maintainability, or ease of use label Feb 5, 2024
@jsf9k
Copy link
Member

jsf9k commented Feb 5, 2024

Thanks for the contribution @jroose!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvement This issue or pull request will add or improve functionality, maintainability, or ease of use
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants