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

package.json: add support for replay and core debug modes #1268

Closed
wants to merge 22 commits into from

Conversation

lggomez
Copy link
Contributor

@lggomez lggomez commented Mar 4, 2021

New launch modes: replay and core

The following modes are added:

  • replay: Replays an rr trace, allowing backwards flows (reverse continue and stepback). Requires a traceDirPath property on launch.json pointing to a valid rr trace directory.
    Equivalent to dlv replay <tracedir> command.
  • core: Replays a core dump file, showing its callstack and the file matching the callsite. Requires a coreFilePath property on launch.json pointing to a valid coredump file.
    Equivalent to dlv core <exe> <corefile> command.

With the new dap service, most of the heavy work will go to delve, so this PR mainly involves getting around passing the corresponding launchArgs to delve. It´s service counterpart lies on go-delve/delve#2367

Fixes #168
Fixes #1243
Updates #110

@google-cla google-cla bot added the cla: yes Used by googlebot to label PRs as having a valid CLA. The text of this label should not change. label Mar 4, 2021
@gopherbot
Copy link
Collaborator

This PR (HEAD: bc89abb) has been imported to Gerrit for code review.

Please visit https://go-review.googlesource.com/c/vscode-go/+/298569 to see it.

Tip: You can toggle comments from me using the comments slash command (e.g. /comments off)
See the Wiki page for more info

@gopherbot
Copy link
Collaborator

Message from Hyang-Ah Hana Kim:

Patch Set 1:

(1 comment)


Please don’t reply on this GitHub thread. Visit golang.org/cl/298569.
After addressing review feedback, remember to publish your drafts!

@gopherbot
Copy link
Collaborator

This PR (HEAD: 6075579) has been imported to Gerrit for code review.

Please visit https://go-review.googlesource.com/c/vscode-go/+/298569 to see it.

Tip: You can toggle comments from me using the comments slash command (e.g. /comments off)
See the Wiki page for more info

@gopherbot
Copy link
Collaborator

This PR (HEAD: 9520700) has been imported to Gerrit for code review.

Please visit https://go-review.googlesource.com/c/vscode-go/+/298569 to see it.

Tip: You can toggle comments from me using the comments slash command (e.g. /comments off)
See the Wiki page for more info

@gopherbot
Copy link
Collaborator

This PR (HEAD: 3ea3d73) has been imported to Gerrit for code review.

Please visit https://go-review.googlesource.com/c/vscode-go/+/298569 to see it.

Tip: You can toggle comments from me using the comments slash command (e.g. /comments off)
See the Wiki page for more info

@gopherbot
Copy link
Collaborator

This PR (HEAD: 0d57463) has been imported to Gerrit for code review.

Please visit https://go-review.googlesource.com/c/vscode-go/+/298569 to see it.

Tip: You can toggle comments from me using the comments slash command (e.g. /comments off)
See the Wiki page for more info

@gopherbot
Copy link
Collaborator

This PR (HEAD: c0af810) has been imported to Gerrit for code review.

Please visit https://go-review.googlesource.com/c/vscode-go/+/298569 to see it.

Tip: You can toggle comments from me using the comments slash command (e.g. /comments off)
See the Wiki page for more info

@gopherbot
Copy link
Collaborator

This PR (HEAD: edf0858) has been imported to Gerrit for code review.

Please visit https://go-review.googlesource.com/c/vscode-go/+/298569 to see it.

Tip: You can toggle comments from me using the comments slash command (e.g. /comments off)
See the Wiki page for more info

@gopherbot
Copy link
Collaborator

This PR (HEAD: 366fa88) has been imported to Gerrit for code review.

Please visit https://go-review.googlesource.com/c/vscode-go/+/298569 to see it.

Tip: You can toggle comments from me using the comments slash command (e.g. /comments off)
See the Wiki page for more info

@gopherbot
Copy link
Collaborator

This PR (HEAD: 3c2f52e) has been imported to Gerrit for code review.

Please visit https://go-review.googlesource.com/c/vscode-go/+/298569 to see it.

Tip: You can toggle comments from me using the comments slash command (e.g. /comments off)
See the Wiki page for more info

@gopherbot
Copy link
Collaborator

This PR (HEAD: 9134201) has been imported to Gerrit for code review.

Please visit https://go-review.googlesource.com/c/vscode-go/+/298569 to see it.

Tip: You can toggle comments from me using the comments slash command (e.g. /comments off)
See the Wiki page for more info

@gopherbot
Copy link
Collaborator

This PR (HEAD: ec0b234) has been imported to Gerrit for code review.

Please visit https://go-review.googlesource.com/c/vscode-go/+/298569 to see it.

Tip: You can toggle comments from me using the comments slash command (e.g. /comments off)
See the Wiki page for more info

@gopherbot
Copy link
Collaborator

This PR (HEAD: d85f75a) has been imported to Gerrit for code review.

Please visit https://go-review.googlesource.com/c/vscode-go/+/298569 to see it.

Tip: You can toggle comments from me using the comments slash command (e.g. /comments off)
See the Wiki page for more info

@gopherbot
Copy link
Collaborator

This PR (HEAD: 2a608ca) has been imported to Gerrit for code review.

Please visit https://go-review.googlesource.com/c/vscode-go/+/298569 to see it.

Tip: You can toggle comments from me using the comments slash command (e.g. /comments off)
See the Wiki page for more info

@gopherbot
Copy link
Collaborator

This PR (HEAD: 39e88b2) has been imported to Gerrit for code review.

Please visit https://go-review.googlesource.com/c/vscode-go/+/298569 to see it.

Tip: You can toggle comments from me using the comments slash command (e.g. /comments off)
See the Wiki page for more info

@gopherbot
Copy link
Collaborator

Message from Hyang-Ah Hana Kim:

Patch Set 27: Run-TryBot+1 Trust+1

(1 comment)


Please don’t reply on this GitHub thread. Visit golang.org/cl/298569.
After addressing review feedback, remember to publish your drafts!

@gopherbot
Copy link
Collaborator

Message from kokoro:

Patch Set 27:

Kokoro presubmit build starting for golang/vscode-go/gcp_ubuntu/presubmit
Logs at:
https://source.cloud.google.com/results/invocations/091f1fa0-41cb-46fa-a04d-31be3b8c0411


Please don’t reply on this GitHub thread. Visit golang.org/cl/298569.
After addressing review feedback, remember to publish your drafts!

@gopherbot
Copy link
Collaborator

Message from kokoro:

Patch Set 27: TryBot-Result+1

Kokoro presubmit build finished with status: SUCCESS
Logs at: https://source.cloud.google.com/results/invocations/091f1fa0-41cb-46fa-a04d-31be3b8c0411


Please don’t reply on this GitHub thread. Visit golang.org/cl/298569.
After addressing review feedback, remember to publish your drafts!

@gopherbot
Copy link
Collaborator

Message from Hyang-Ah Hana Kim:

Patch Set 28: Commit message was updated.


Please don’t reply on this GitHub thread. Visit golang.org/cl/298569.
After addressing review feedback, remember to publish your drafts!

@gopherbot
Copy link
Collaborator

Message from Hyang-Ah Hana Kim:

Patch Set 28: Code-Review+2 Trust+1


Please don’t reply on this GitHub thread. Visit golang.org/cl/298569.
After addressing review feedback, remember to publish your drafts!

@gopherbot
Copy link
Collaborator

Message from Suzy Mueller:

Patch Set 29: Trust+1


Please don’t reply on this GitHub thread. Visit golang.org/cl/298569.
After addressing review feedback, remember to publish your drafts!

@hyangah hyangah added this to the v0.27.1 milestone Aug 10, 2021
@gopherbot
Copy link
Collaborator

Message from Hyang-Ah Hana Kim:

Patch Set 30: Published edit on patch set 29.


Please don’t reply on this GitHub thread. Visit golang.org/cl/298569.
After addressing review feedback, remember to publish your drafts!

@gopherbot
Copy link
Collaborator

Message from Hyang-Ah Hana Kim:

Patch Set 30: Run-TryBot+1 Code-Review+2 Trust+1

(1 comment)


Please don’t reply on this GitHub thread. Visit golang.org/cl/298569.
After addressing review feedback, remember to publish your drafts!

@gopherbot
Copy link
Collaborator

This PR (HEAD: e055352) has been imported to Gerrit for code review.

Please visit https://go-review.googlesource.com/c/vscode-go/+/298569 to see it.

Tip: You can toggle comments from me using the comments slash command (e.g. /comments off)
See the Wiki page for more info

@gopherbot
Copy link
Collaborator

Message from Hyang-Ah Hana Kim:

Patch Set 31: Run-TryBot+1 Code-Review+2 Trust+1


Please don’t reply on this GitHub thread. Visit golang.org/cl/298569.
After addressing review feedback, remember to publish your drafts!

@hyangah hyangah changed the title Support for replay and core modes package.json: add support for replay and core debug modes Aug 10, 2021
gopherbot pushed a commit that referenced this pull request Aug 10, 2021
## New launch modes: replay and core

The following modes are added:

- **replay**: Replays an rr trace, allowing backwards flows (reverse continue and stepback). Requires a `traceDirPath` property on `launch.json` pointing to a valid rr trace directory.
Equivalent to `dlv replay <tracedir>` command.
- **core**: Replays a core dump file, showing its callstack and the file matching the callsite. Requires a `coreFilePath` property on `launch.json` pointing to a valid coredump file.
Equivalent to `dlv core <exe> <corefile>` command.

With the new dap service, most of the heavy work will go to delve, so this PR mainly involves getting around passing the corresponding launchArgs to delve. It´s service counterpart lies on go-delve/delve#2367

Change-Id: Idc21f27152387c07c844fa471b89d95f38c2545b
GitHub-Last-Rev: e055352
GitHub-Pull-Request: #1268
Reviewed-on: https://go-review.googlesource.com/c/vscode-go/+/298569
Reviewed-by: Hyang-Ah Hana Kim <hyangah@gmail.com>
Trust: Hyang-Ah Hana Kim <hyangah@gmail.com>
Trust: Suzy Mueller <suzmue@golang.org>
@gopherbot
Copy link
Collaborator

Message from kokoro:

Patch Set 31:

Kokoro presubmit build starting for golang/vscode-go/gcp_ubuntu/presubmit
Logs at:
https://source.cloud.google.com/results/invocations/0ce723b1-6dc5-48f8-aac6-30a17d8629a6


Please don’t reply on this GitHub thread. Visit golang.org/cl/298569.
After addressing review feedback, remember to publish your drafts!

@gopherbot
Copy link
Collaborator

This PR is being closed because golang.org/cl/298569 has been merged.

@gopherbot gopherbot closed this Aug 10, 2021
gopherbot pushed a commit that referenced this pull request Aug 30, 2021
The following modes are added:

- **replay**: Replays an rr trace, allowing backwards flows (reverse continue and stepback). Requires a `traceDirPath` property on `launch.json` pointing to a valid rr trace directory.
Equivalent to `dlv replay <tracedir>` command.
- **core**: Replays a core dump file, showing its callstack and the file matching the callsite. Requires a `coreFilePath` property on `launch.json` pointing to a valid coredump file.
Equivalent to `dlv core <exe> <corefile>` command.

With the new dap service, most of the heavy work will go to delve, so this PR mainly involves getting around passing the corresponding launchArgs to delve. It´s service counterpart lies on go-delve/delve#2367

GitHub-Last-Rev: e055352
GitHub-Pull-Request: #1268
Reviewed-on: https://go-review.googlesource.com/c/vscode-go/+/298569
Reviewed-by: Hyang-Ah Hana Kim <hyangah@gmail.com>
Trust: Hyang-Ah Hana Kim <hyangah@gmail.com>
Trust: Suzy Mueller <suzmue@golang.org>
Change-Id: I7b6b3b7b9d7c30d552a4401892492c74ec2e3023
Reviewed-on: https://go-review.googlesource.com/c/vscode-go/+/346092
Run-TryBot: Hyang-Ah Hana Kim <hyangah@gmail.com>
Reviewed-by: Suzy Mueller <suzmue@golang.org>
TryBot-Result: kokoro <noreply+kokoro@google.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes Used by googlebot to label PRs as having a valid CLA. The text of this label should not change.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

debug: add replay mode to launch.json debug: UI to examine core dumps
3 participants