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

[Azure Pipelines] Use Python 3.8.x for Safari test-suite runs #25044

Merged
merged 1 commit into from Nov 23, 2020

Conversation

stephenmcgruer
Copy link
Contributor

@stephenmcgruer stephenmcgruer commented Aug 17, 2020

@wpt-pr-bot wpt-pr-bot temporarily deployed to wpt-preview-25044 August 17, 2020 17:02 Inactive
@wpt-pr-bot wpt-pr-bot temporarily deployed to wpt-preview-25044 August 18, 2020 01:09 Inactive
@foolip
Copy link
Member

foolip commented Aug 18, 2020

The run for 2089041 finished and the differences aren't massive:
https://wpt.fyi/results/?q=is%3Adifferent&run_id=627220001&run_id=648510001

@wpt-pr-bot wpt-pr-bot temporarily deployed to wpt-preview-25044 October 1, 2020 12:37 Inactive
@stephenmcgruer
Copy link
Contributor Author

stephenmcgruer commented Oct 1, 2020

I have pushed 78424f2 to triggers/safari_preview. The nearest parent is 6ceb598 which is unfortunately a few commits back but should be roughly ok for a comparison.

Once the run has finished I will post the wpt.fyi comparison url.

cc @ziransun

@wpt-pr-bot wpt-pr-bot temporarily deployed to wpt-preview-25044 October 1, 2020 19:17 Inactive
@stephenmcgruer
Copy link
Contributor Author

https://wpt.fyi/results/?diff&filter=ADC&run_id=694810001&run_id=702810004 is the diff, but there's something strange there. Multiple directories have hundreds of tests now MISSING, and unless someone deleted them in 6ceb598...78424f2 (which I doubt) then something is wrong with the run.

@ziransun FYI as you might want to take a look anyway, but really I need to dig into the Azure Pipelines logs for the run.

@stephenmcgruer
Copy link
Contributor Author

I wasn't able to find anything obvious in the logs at a quick look, so I'm being lazy and just triggering another run to see. eb419a7 is the run, the parent is c960a24 aka https://wpt.fyi/results/?run_id=683470005

@wpt-pr-bot wpt-pr-bot temporarily deployed to wpt-preview-25044 October 5, 2020 14:31 Inactive
@stephenmcgruer
Copy link
Contributor Author

Ok, the new run seems much more reasonable: https://wpt.fyi/results/?diff&filter=ADC&run_id=683470005&run_id=695000002 . A little concerning to not figure out what was happening last time, but for now I think we can proceed with using this as the newest Safari run.

cc @ziransun

@stephenmcgruer stephenmcgruer changed the title [WIP] Use Python 3.8.x for Safari Preview runs [WIP] Use Python 3.8.x for Safari runs Oct 29, 2020
@stephenmcgruer stephenmcgruer changed the title [WIP] Use Python 3.8.x for Safari runs [Azure Pipelines] Use Python 3.8.x for Safari test-suite runs Oct 29, 2020
@wpt-pr-bot wpt-pr-bot temporarily deployed to wpt-preview-25044 October 29, 2020 11:49 Inactive
@stephenmcgruer stephenmcgruer merged commit 21d61bd into master Nov 23, 2020
@stephenmcgruer stephenmcgruer deleted the smcgruer/macos-py3.8 branch November 23, 2020 14:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants