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

[Gecko Bug 1584192] Fix handling multiple repeats in wptreport logs #19697

Merged
merged 3 commits into from Oct 16, 2019

Conversation

moz-wptsync-bot
Copy link
Collaborator

Previously this would have ended up with multiple JSON objects all put
on the same line and we'd be unable to update from such
metadata. Ensure that there's one line per JSON object and the update
code is able to consume this format.

Differential Revision: https://phabricator.services.mozilla.com/D47248

bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1584192
gecko-commit: 83dc6a6ed93137707ecfd785f10a721e2e1abca7
gecko-integration-branch: autoland
gecko-reviewers: maja_zf

Previously this would have ended up with multiple JSON objects all put
on the same line and we'd be unable to update from such
metadata. Ensure that there's one line per JSON object and the update
code is able to consume this format.

Differential Revision: https://phabricator.services.mozilla.com/D47248

bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1584192
gecko-commit: 83dc6a6ed93137707ecfd785f10a721e2e1abca7
gecko-integration-branch: autoland
gecko-reviewers: maja_zf
@wpt-pr-bot wpt-pr-bot added infra wptrunner The automated test runner, commonly called through ./wpt run labels Oct 15, 2019
Copy link
Collaborator

@wpt-pr-bot wpt-pr-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Already reviewed downstream.

DONTBUILD

Differential Revision: https://phabricator.services.mozilla.com/D49283

bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1584192
gecko-commit: 491690739db4e4ad8eaa26494d45d3a0430b2ac1
gecko-integration-branch: autoland
gecko-reviewers: ato
Previously when running with --rerun=N we added all the subtest results
to the previous test results and so got all N iterations of each
subtest result, but only one of each test result. Instead ensure that
top-level tests are added each time they are run, allowing multiple
test results.

Differential Revision: https://phabricator.services.mozilla.com/D47249

bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1584192
gecko-commit: 325b5278961fb32b8fd6d4dcc481c4dd7890f33c
gecko-integration-branch: autoland
gecko-reviewers: maja_zf
@moz-wptsync-bot moz-wptsync-bot merged commit e5c5b3e into master Oct 16, 2019
@moz-wptsync-bot moz-wptsync-bot deleted the gecko/1584192 branch October 16, 2019 16:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
infra mozilla:gecko-sync 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

3 participants