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

debuginfo: Add extended support for Go binaries #27

Merged
merged 6 commits into from
Sep 16, 2021
Merged

Conversation

kakkoyun
Copy link
Member

@kakkoyun kakkoyun commented Sep 14, 2021

Signed-off-by: Kemal Akkoyun kakkoyun@gmail.com

Refers #4

Signed-off-by: Kemal Akkoyun <kakkoyun@gmail.com>
Signed-off-by: Kemal Akkoyun <kakkoyun@gmail.com>
Signed-off-by: Kemal Akkoyun <kakkoyun@gmail.com>
@kakkoyun kakkoyun changed the title Add extended support for Go binaries debuginfo: Add extended support for Go binaries Sep 15, 2021
Copy link
Member

@brancz brancz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very nice!


var errNotFound = errors.New("not found")

type DebugInfoClient interface {
Copy link
Member

@brancz brancz Sep 15, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a side note, nothing for now, I was thinking of moving the client wrapper that chunks the upload to this repo, as then in the future we can use buf remote-generation, and not even have any dependency on Parca at all.

@kakkoyun kakkoyun changed the title debuginfo: Add extended support for Go binaries [WIP] debuginfo: Add extended support for Go binaries Sep 15, 2021
Signed-off-by: Kemal Akkoyun <kakkoyun@gmail.com>
@kakkoyun kakkoyun changed the title [WIP] debuginfo: Add extended support for Go binaries debuginfo: Add extended support for Go binaries Sep 15, 2021
Signed-off-by: Kemal Akkoyun <kakkoyun@gmail.com>
Signed-off-by: Kemal Akkoyun <kakkoyun@gmail.com>
@kakkoyun kakkoyun merged commit 5ed28de into main Sep 16, 2021
@kakkoyun kakkoyun deleted the go_support branch September 16, 2021 13:22
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.

None yet

2 participants