-
Notifications
You must be signed in to change notification settings - Fork 56
fix: add sandbox endpoint to worker so that it can be triggered by tr… #150
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
base: master
Are you sure you want to change the base?
Conversation
…iggers Signed-off-by: sheltonsuen <[email protected]>
|
Thank you for your PR. This configuration is indeed required by both backend services, API and worker. Currently, we have only added the corresponding environment variable to the API. In fact, we have already designed a Helm function charts/charts/dify/templates/_helpers.tpl Lines 98 to 176 in 83587c2
|
|
Also, you need to bump the chart version to get it merged |
after upgrade to dify 1.10.0 without RC tag, the triggers not able to exec workflows automatcily due to workers not able to find the sandbox service