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

Better log for debugging #46825

Merged
merged 1 commit into from
Jun 20, 2024
Merged

Better log for debugging #46825

merged 1 commit into from
Jun 20, 2024

Conversation

WeizhongX
Copy link
Contributor

We are seeing logs as below from Chromium bots. Add more information to log to make debugging easier.

Two kind of error log to address here:

  1. No handler for resource: '/404'
  2. WebDriver was not accessible within the timeout:

Bug: chromedriver:4793

We are seeing logs as below from Chromium bots. Add more information to
log to make debugging easier.

Two kind of error log to address here:
1) No handler for resource: '/404'
2) WebDriver was not accessible within the timeout:

Bug: chromedriver:4793
@WeizhongX WeizhongX requested a review from a team as a code owner June 19, 2024 00:50
@wpt-pr-bot wpt-pr-bot added infra third_party wptrunner The automated test runner, commonly called through ./wpt run labels Jun 19, 2024
@WeizhongX WeizhongX requested review from jonathan-j-lee and removed request for jgraham, foolip and DanielRyanSmith June 19, 2024 00:51
@WeizhongX WeizhongX closed this Jun 20, 2024
@WeizhongX WeizhongX reopened this Jun 20, 2024
@WeizhongX
Copy link
Contributor Author

Unexpected Results

/infrastructure/server/context.any.serviceworker.html
TIMEOUT /infrastructure/server/context.any.serviceworker.html - TestRunner hit external timeout (this may indicate a hang)
/infrastructure/server/order-of-metas.any.worker.html
TIMEOUT /infrastructure/server/order-of-metas.any.worker.html - TestRunner hit external timeout (this may indicate a hang)
SUITE-END | took 746s

@WeizhongX
Copy link
Contributor Author

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

@jcscottiii can you help admin merge? Thanks!

@jcscottiii jcscottiii merged commit 3bd9ff9 into master Jun 20, 2024
31 of 35 checks passed
@jcscottiii jcscottiii deleted the better-logging branch June 20, 2024 18:30
sadym-chromium pushed a commit that referenced this pull request Jul 18, 2024
We are seeing logs as below from Chromium bots. Add more information to
log to make debugging easier.

Two kind of error log to address here:
1) No handler for resource: '/404'
2) WebDriver was not accessible within the timeout:

Bug: chromedriver:4793
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
infra third_party wptrunner The automated test runner, commonly called through ./wpt run
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants