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

Allow live debugging tasks with actual sources #19978

Merged
merged 12 commits into from
Jun 17, 2024
Merged

Conversation

kboom
Copy link
Collaborator

@kboom kboom commented Jun 10, 2024

This change makes it much easier to troubleshoot problems with tasks or develop new features. It makes it possible to connect to a specific task which is being executed by the agent and debug this in Visual Studio Code.

Note: this recording is based on the previous parameter set. After review, we've managed to simplify this to just one parameter passed to an existing build command.

short

There is a single new parameter to an existing command introduced --debug-agent-dir, which can be used like this:
node make.js build --task AzureCLIV2 --debug-agent-dir "C:\src\azure-pipelines-agent\_layout\win-x64"
or like this, to do this for all tasks
node make.js build --debug-agent-dir "C:\src\azure-pipelines-agent\_layout\win-x64

This works with the change on the agent side.

@kboom kboom requested review from manolerazvan and a team as code owners June 10, 2024 13:29
make.js Outdated Show resolved Hide resolved
make.js Outdated Show resolved Hide resolved
@kboom kboom force-pushed the users/kboom/debugging_tasks branch from bafc18b to 782aa75 Compare June 13, 2024 11:23
@kboom kboom requested a review from a team as a code owner June 13, 2024 19:43
@kboom kboom enabled auto-merge (squash) June 17, 2024 19:23
@kboom kboom merged commit 3385262 into master Jun 17, 2024
11 checks passed
@kboom kboom deleted the users/kboom/debugging_tasks branch June 18, 2024 07:20
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.

5 participants