Skip to content

Enable Python debug port #521

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

Closed
asavaritayal opened this issue Jun 7, 2018 · 6 comments
Closed

Enable Python debug port #521

asavaritayal opened this issue Jun 7, 2018 · 6 comments

Comments

@asavaritayal
Copy link
Contributor

For VS Code to use /cc @nturinski @fiveisprime

@pragnagopa
Copy link
Member

@ahmelsayed - I am working on updating the functions host to parse languageWorker.{language}.arguments instead of specifically looking for woker.{language}.Debug. This will allow CLI to pass other commandline options as well.
Instead of hard coding the debug ports, can we take that as an input? for example VS Code can pass the debug port to CLI

@ahmelsayed
Copy link
Contributor

as mentioned on email, this is the syntax that vscode needs to pass in

# for node
func ... --language-worker -- "--inspect=<port>"
 
# for java
func ... --language-worker -- "<port>"
 
# for python
func ... --language-worker -- "<port>"

The CLI will have the following mapping:

{
    node: "languageWorkers:node:arguments={languageWorker}", // in this example {languageWorker} is "--inspect=<port>"
    java: "languageWorkers:java:debug={languageWorker}", // in this example {languageWorker} is "<port>"
    python: "languageWorkers:python:debug={languageWorker}" // in this example {languageWorker} is "<port>"
}

@ahmelsayed
Copy link
Contributor

I have this PR for this change #546

Please let me know if the mapping above is correct and I'll merge it

@ahmelsayed
Copy link
Contributor

@asavaritayal please make sure to follow up with vscode to pass the new parameters starting with beta.30

@pragnagopa
Copy link
Member

@jdneo - Please see sample above for setting the debug port for java

@jdneo
Copy link
Contributor

jdneo commented Jul 12, 2018

@pragnagopa Thanks for the information. So -agentlib:jdwp=transport=dt_socket,server=y,suspend=n,address={portnumber} will only has effect if user set FUNCTIONS_WORKER_RUNTIME to java in local.settings.json?

alrod pushed a commit to alrod/azure-functions-core-tools that referenced this issue Jul 17, 2018
@ghost ghost locked as resolved and limited conversation to collaborators Dec 19, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants