Skip to content

fix(profiler): handle non-frame objects while stack-unwinding [backport 1.7]#5090

Merged
brettlangdon merged 2 commits into
1.7from
backport-5019-to-1.7
Feb 10, 2023
Merged

fix(profiler): handle non-frame objects while stack-unwinding [backport 1.7]#5090
brettlangdon merged 2 commits into
1.7from
backport-5019-to-1.7

Conversation

@github-actions

@github-actions github-actions Bot commented Feb 9, 2023

Copy link
Copy Markdown
Contributor

Backport ea7c9c6 from #5019 to 1.7.

Following reports of Python 3.11 returning non-frame objects while unwinding the stack of a running thread, this change adds handling code that prevents exceptions from being raised.

Checklist

  • Change(s) are motivated and described in the PR description.
  • Testing strategy is described if automated tests are not included in the PR.
  • Risk is outlined (performance impact, potential for breakage, maintainability, etc).
  • Change is maintainable (easy to change, telemetry, documentation).
  • Library release note guidelines are followed.
  • Documentation is included (in-code, generated user docs, public corp docs).

Reviewer Checklist

  • Title is accurate.
  • No unnecessary changes are introduced.
  • Description motivates each change.
  • Avoids breaking API changes unless absolutely necessary.
  • Testing strategy adequately addresses listed risk(s).
  • Change is maintainable (easy to change, telemetry, documentation).
  • Release note makes sense to a user of the library.

@github-actions
github-actions Bot requested a review from a team as a code owner February 9, 2023 16:26
@github-actions github-actions Bot added the Profiling Continous Profling label Feb 9, 2023
brettlangdon
brettlangdon previously approved these changes Feb 9, 2023
P403n1x87
P403n1x87 previously approved these changes Feb 9, 2023
Following reports of Python 3.11 returning non-frame objects while
unwinding the stack of a running thread, this change adds handling code
that prevents exceptions from being raised.

(cherry picked from commit ea7c9c6)
@brettlangdon
brettlangdon dismissed stale reviews from P403n1x87 and themself via 8df5f95 February 10, 2023 15:40
@brettlangdon
brettlangdon enabled auto-merge (squash) February 10, 2023 19:12
@brettlangdon
brettlangdon merged commit d72b0d1 into 1.7 Feb 10, 2023
@brettlangdon
brettlangdon deleted the backport-5019-to-1.7 branch February 10, 2023 20:03
@github-actions github-actions Bot added this to the v1.7.5 milestone Feb 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Profiling Continous Profling

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants