Skip to content

[examine-stack] Try to unblock Ubuntu 20.04#32975

Merged
ctiller merged 1 commit intogrpc:masterfrom
ctiller:better-examination-2
May 1, 2023
Merged

[examine-stack] Try to unblock Ubuntu 20.04#32975
ctiller merged 1 commit intogrpc:masterfrom
ctiller:better-examination-2

Conversation

@ctiller
Copy link
Copy Markdown
Member

@ctiller ctiller commented Apr 30, 2023

Try a different approach to this test and check some non-leaf functions in the returned text - looks like we're running into problems getting the leaf function out of the stack trace on that platform (which is probably fine): https://source.cloud.google.com/results/invocations/09e8e1ea-df48-4fdb-96dd-916bd5014f90/targets/%2F%2Ftest%2Fcore%2Fgprpp:examine_stack_test/tests

Needed to unblock #32748

@ctiller ctiller requested a review from veblush April 30, 2023 21:48
@ctiller ctiller added the release notes: no Indicates if PR should not be in release notes label Apr 30, 2023
@ctiller ctiller enabled auto-merge (squash) April 30, 2023 23:20
@ctiller ctiller merged commit 9f00eda into grpc:master May 1, 2023
@copybara-service copybara-service bot added the imported Specifies if the PR has been imported to the internal repository label May 1, 2023
paulosjca pushed a commit to paulosjca/grpc that referenced this pull request May 4, 2023
Try a different approach to this test and check some non-leaf functions
in the returned text - looks like we're running into problems getting
the leaf function out of the stack trace on that platform (which is
probably fine):
https://source.cloud.google.com/results/invocations/09e8e1ea-df48-4fdb-96dd-916bd5014f90/targets/%2F%2Ftest%2Fcore%2Fgprpp:examine_stack_test/tests

Needed to unblock grpc#32748

<!--

If you know who should review your pull request, please assign it to
that
person, otherwise the pull request would get assigned randomly.

If your pull request is for a specific language, please add the
appropriate
lang label.

-->
wanlin31 pushed a commit that referenced this pull request May 18, 2023
Try a different approach to this test and check some non-leaf functions
in the returned text - looks like we're running into problems getting
the leaf function out of the stack trace on that platform (which is
probably fine):
https://source.cloud.google.com/results/invocations/09e8e1ea-df48-4fdb-96dd-916bd5014f90/targets/%2F%2Ftest%2Fcore%2Fgprpp:examine_stack_test/tests

Needed to unblock #32748

<!--

If you know who should review your pull request, please assign it to
that
person, otherwise the pull request would get assigned randomly.

If your pull request is for a specific language, please add the
appropriate
lang label.

-->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bloat/none imported Specifies if the PR has been imported to the internal repository lang/core per-call-memory/neutral per-channel-memory/neutral release notes: no Indicates if PR should not be in release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants