GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — gh-91048: Add better error messages for remote debugging for CI builds by pablogsal · Pull Request #134682 · python/cpython

Conversation

@pablogsal

@pablogsal pablogsal commented May 26, 2025 •
edited by bedevere-app Bot
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Member
  • Issue: Async Call-Stack Reconstruction #91048

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
@pablogsal pablogsal added the needs backport to 3.14 bugs and security fixes label May 26, 2025
@bedevere-app bedevere-app Bot added the awaiting core review label May 26, 2025
@bedevere-app bedevere-app Bot mentioned this pull request May 26, 2025
Async Call-Stack Reconstruction #91048
Closed
@pablogsal
pablogsal force-pushed the better_errors branch from 5366a19 to 72f9ada Compare May 26, 2025 01:16
@pablogsal

pablogsal commented May 26, 2025 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Member Author

These only trigger in the test suite mode and are mainly to help diagnose errors in CI in the future

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@pablogsal pablogsal added the skip news label May 26, 2025
@pablogsal
pablogsal force-pushed the better_errors branch 3 times, most recently from 9d3c02d to 2a13bc5 Compare May 26, 2025 01:25
@pablogsal

pablogsal commented May 26, 2025 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Member Author

Example chain:

RuntimeError: mach_vm_read_overwrite failed for PID 74282 at address 0x1 (size 216): kern_return_t 1

The above exception was the direct cause of the following exception:

RuntimeError: Failed to read code object

The above exception was the direct cause of the following exception:

RuntimeError: Failed to parse frame object in chain

The above exception was the direct cause of the following exception:

RuntimeError: Failed to process frame chain

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/Users/pablogsal/github/python/main/benchmark_get_stack_trace.py", line 189, in benchmark
    stack_trace = unwinder.get_stack_trace()
RuntimeError: Failed to unwind stack for thread
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@pablogsal
pablogsal force-pushed the better_errors branch 7 times, most recently from 6b35763 to a15c91e Compare May 26, 2025 02:09
@pablogsal
pablogsal enabled auto-merge (squash) May 26, 2025 02:09
@pablogsal
pablogsal force-pushed the better_errors branch 3 times, most recently from 1a97fe4 to 69ad823 Compare May 26, 2025 02:13
@pablogsal pablogsal changed the title gh-91048: Add better error messages for remote debugging in debug versions gh-91048: Add better error messages for remote debugging for CI builds May 26, 2025
@pablogsal
pablogsal force-pushed the better_errors branch 3 times, most recently from ac5b598 to b90ef14 Compare May 26, 2025 03:07
@pablogsal
pablogsal disabled auto-merge May 26, 2025 03:35
@pablogsal
pablogsal force-pushed the better_errors branch 4 times, most recently from e8bb64b to 2b67d04 Compare May 26, 2025 13:35
@pablogsal
pythongh-91048: Add better error messages for remote debugging in deb…
e0dad63
…ug versions

Signed-off-by: Pablo Galindo <[email protected]>
@pablogsal
pablogsal force-pushed the better_errors branch from 2b67d04 to e0dad63 Compare May 26, 2025 13:58
@pablogsal
pablogsal merged commit 0909d6d into python:main May 26, 2025
@pablogsal
pablogsal deleted the better_errors branch May 26, 2025 14:31
@miss-islington-app

miss-islington-app Bot commented May 26, 2025

Copy link
Copy Markdown

Thanks @pablogsal for the PR 🌮🎉.. I'm working now to backport this PR to: 3.14.
🐍🍒⛏🤖

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@bedevere-app bedevere-app Bot removed the awaiting core review label May 26, 2025
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request May 26, 2025
@pablogsal @miss-islington
pythongh-91048: Add better error messages for remote debugging for CI…
37c5a32
… builds (pythonGH-134682)

(cherry picked from commit 0909d6d)

Co-authored-by: Pablo Galindo Salgado <[email protected]>
@bedevere-app

bedevere-app Bot commented May 26, 2025

Copy link
Copy Markdown

GH-134719 is a backport of this pull request to the 3.14 branch.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@bedevere-app bedevere-app Bot removed the needs backport to 3.14 bugs and security fixes label May 26, 2025
pablogsal added a commit to miss-islington/cpython that referenced this pull request May 26, 2025
@pablogsal
pythongh-91048: Add better error messages for remote debugging for CI…
4a0da8b
… builds (pythonGH-134682)

(cherry picked from commit 0909d6d)

Co-authored-by: Pablo Galindo Salgado <[email protected]>
pablogsal pushed a commit that referenced this pull request May 26, 2025
@miss-islington
[3.14] gh-91048: Add better error messages for remote debugging for C…
aded785
…I builds (GH-134682) (#134719)
@chris-eibl chris-eibl mentioned this pull request Jun 9, 2025
Fix warnings set but not used [-Wunused-but-set-variable] in remote_debug.h #135290
Merged
Pranjal095 pushed a commit to Pranjal095/cpython that referenced this pull request Jul 12, 2025
@pablogsal @Pranjal095
pythongh-91048: Add better error messages for remote debugging for CI…
b34f9bd
… builds (python#134682)
taegyunkim pushed a commit to taegyunkim/cpython that referenced this pull request Aug 4, 2025
@pablogsal @taegyunkim
pythongh-91048: Add better error messages for remote debugging for CI…
a990ffb
… builds (python#134682)
gostak-dd pushed a commit to gostak-dd/cpython that referenced this pull request Jun 2, 2026
@pablogsal @gostak-dd
pythongh-91048: Add better error messages for remote debugging for CI…
ff60a93
… builds (python#134682)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

skip news

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

1 participant

@pablogsal
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.