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

Check thread id is in current frames #46266

Merged
merged 1 commit into from
May 15, 2024
Merged

Check thread id is in current frames #46266

merged 1 commit into from
May 15, 2024

Conversation

WeizhongX
Copy link
Contributor

@WeizhongX WeizhongX commented May 14, 2024

On Chromium Windows bot, we are seeing KeyError as below. Check the thread id is in sys._current_frames() before format_stack, otherwise do not include the stack in the error message.

Error message:
00:02:16.943 WARNING: Exception in TestExecutor.run:
File "C:\b\s\w\ir\third_party\wpt_tools\wpt\tools\wptrunner\wptrunner\executors\base.py", line 219, in run
message += "".join(traceback.format_stack(sys._current_frames()[executor.ident]))
KeyError: 8372

On Chromium Windows bot, we are seeing KeyError below. Check the thread
id is in sys._current_frames() before format_stack, otherwise do not
include the stack in the error message.

Error message:
00:02:16.943 WARNING: Exception in TestExecutor.run:
  File "C:\b\s\w\ir\third_party\wpt_tools\wpt\tools\wptrunner\wptrunner\executors\base.py", line 219, in run
    message += "".join(traceback.format_stack(sys._current_frames()[executor.ident]))
KeyError: 8372
@wpt-pr-bot wpt-pr-bot added infra wptrunner The automated test runner, commonly called through ./wpt run labels May 14, 2024
@WeizhongX
Copy link
Contributor Author

/infrastructure/testdriver/actions/multiTouchPointsSimultaneousMove.html
TIMEOUT /infrastructure/testdriver/actions/multiTouchPointsSimultaneousMove.html - TestRunner hit external timeout (this may indicate a hang)

The failure on infrastructure/ tests is not related with this change.

@WeizhongX
Copy link
Contributor Author

@KyleJu can you help admin merge? Thanks!

@KyleJu KyleJu merged commit c55cca3 into master May 15, 2024
31 of 33 checks passed
@KyleJu KyleJu deleted the check-thread-id branch May 15, 2024 17:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
infra wptrunner The automated test runner, commonly called through ./wpt run
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants