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

func start on Linux won't stop upon recieving SIGTERM #3074

Open
UnknownJoe796 opened this issue Jun 9, 2022 · 7 comments · May be fixed by #3148
Open

func start on Linux won't stop upon recieving SIGTERM #3074

UnknownJoe796 opened this issue Jun 9, 2022 · 7 comments · May be fixed by #3148

Comments

@UnknownJoe796
Copy link

If I execute func start on Linux and then send a termination request (kill -s SIGTERM <pid>), the local runner will ignore the request.

Sending a termination signal like this should result in the program closing. Instead, the process stays open indefinitely.

Interestingly, SIGINT works just fine.

I'm running Linux and I've tested this with both a Node project and a Java project.

This is currently preventing the Azure Functions Gradle Plugin from shutting down down the azureFunctionsRun task correctly, which is a huge usability problem for that plugin. However, the root issue remains in func start.

@robmoo
Copy link

robmoo commented Aug 22, 2022

Is there any update on this issue? We are getting this issue as well in our build pipelines when trying to run integration tests

@UnknownJoe796
Copy link
Author

We don't have anything yet. I tried looking through the source myself to figure something out but I'm not a C# guy.

@beyerleinf beyerleinf linked a pull request Sep 9, 2022 that will close this issue
5 tasks
@beyerleinf
Copy link

beyerleinf commented Sep 9, 2022

Just stumbled upon this same issue and opened a PR, at least for the version published on npm.

@beyerleinf
Copy link

Has anyone had time to look a this yet? I need this fix...

@paulsmithkc
Copy link

Same problem exists on MacOS.

@beyerleinf
Copy link

Would be nice if this could get some attention.

@paulsmithkc
Copy link

Link #3222

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants