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

Add ability to override command for main container #40

Closed
wants to merge 2 commits into from

Conversation

tomaszkiewicz
Copy link
Collaborator

This simple change adds ability to override entrypoint for the container.


Story behind the change: (if someone is interested or have similar issue)

I encountered strange issue while using volumes based on NFS as a storage for n8n and n8n was unable to execute entrypoint script due to immutable .snapshot directory on the mounted filesystem, so the only way to temporary hack it was to override entrypoint with my own version that does chown optionally and runs as root:

          command:
            - tini
            - --
            - /bin/sh
            - -c
            - chmod o+rx /root; chown -R node /root/.n8n || true; chown -R node /root/.n8n; ln -s /root/.n8n /home/node; chown -R node /home/node || true; node /usr/local/bin/n8n

Copy link
Member

@Vad1mo Vad1mo left a comment

Choose a reason for hiding this comment

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

looks good to me. Can be accepted.

Please also update the README file

values.yaml Show resolved Hide resolved
@tomaszkiewicz
Copy link
Collaborator Author

For some strange reason GH doesn't see new commit and doesn't allow me to open another PR for the same branch... but I also got error that they're under heavy load, so let's wait and see if the commit will show up soon.

@Vad1mo
Copy link
Member

Vad1mo commented Jun 29, 2023

@tomaszkiewicz ping

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