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

Update docker-compose.yml #13

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

seritalien
Copy link

corrected the execution mode to make it work properly

corrected the execution mode to make it work properly
@seritalien
Copy link
Author

tested on my own VPS and works well now with these changes

Copy link
Member

@netroy netroy left a comment

Choose a reason for hiding this comment

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

These changes will actually break the stack

Comment on lines +70 to +71
environnement:
- EXECUTIONS_MODE=queue
Copy link
Member

Choose a reason for hiding this comment

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

This bit is shared, and should be present on all containers when running in queue mode

@@ -67,6 +66,8 @@ services:

n8n-worker:
<<: *shared
command: worker
command: n8n worker
Copy link
Member

Choose a reason for hiding this comment

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

On any recent version of n8n, this will most definitely exit the container with the message Error: command n8n not found

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