search

8gears / n8n-helm-chart

A Kubernetes Helm chart for n8n a Workflow Automation Tool. Easily automate tasks across different services.
https://artifacthub.io/packages/helm/open-8gears/n8n
Apache License 2.0
187 stars 97 forks source link

When scaling.webhook is enabled webhook-test paths are returning 404 #75

Open aeb-dev opened 7 months ago

aeb-dev commented 7 months ago 
ingress:
  enabled: true
  ...
scaling:
  enabled: true
  webhook:
    enabled: true

I have the configuration above for webhooks. When I deploy n8n and trigger a webhook based on test-url, I always get 404. I changed the ingress definition of path:/webhook-test/ to use main n8n pod it works as expected.

My guess is that setting https://github.com/n8n-io/n8n/blob/master/packages/cli/src/config/schema.ts#L718 (N8N_DISABLE_PRODUCTION_MAIN_PROCESS) to true only affects the production url, so the test url should still point to main n8n

Vad1mo commented 7 months ago

Hmm, what do you suggest could be a solution.

aeb-dev commented 7 months ago

I'm not 100% sure but I feel like webhook-test part of Ingress should point to main n8n process

egandro commented 5 months ago

N8N_DISABLE_PRODUCTION_MAIN_PROCESS

I think the -test is missing here:

https://github.com/8gears/n8n-helm-chart/blob/master/templates/service.webhooks.yaml

egandro commented 5 months ago

That looks ok...

https://github.com/8gears/n8n-helm-chart/blob/4f70095930313ef2bd1dfd05de570afe91175ce4/templates/ingress.yaml#L56