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 pipeline error, Install Safari Technology Preview "no download for 10.16" #34926

Closed
TalbotG opened this issue Jul 21, 2022 · 7 comments · Fixed by #34942
Closed

Azure pipeline error, Install Safari Technology Preview "no download for 10.16" #34926

TalbotG opened this issue Jul 21, 2022 · 7 comments · Fixed by #34942

Comments

@TalbotG
Copy link
Contributor

TalbotG commented Jul 21, 2022

@foolip , @jcscottiii , @gsnedders , @whimboo
Dear github admins and colleagues,

I am running again into this error in PR34925 , just like in issue 34752
{
Check failure on line 30 in Build log
@azure-pipelines azure-pipelines / Azure Pipelines
Build log #L30
Bash exited with code '1'.

View raw log

Starting: Install Safari Technology Preview

Task : Command line
Description : Run a command line script using Bash on Linux and macOS and cmd.exe on Windows
Version : 2.201.1
Author : Microsoft Corporation
Help : https://docs.microsoft.com/azure/devops/pipelines/tasks/utility/command-line

Generating script.
Starting Command Output
/bin/bash --noprofile --norc /Users/runner/work/_temp/2aebd30f-ed7f-43f3-8a3d-c1f902896ae4.sh

  • ./wpt install --channel preview --download-only -d . --rename STP safari browser
    INFO:install:Now installing safari browser...
    INFO:install:Finding STP download URLs
    INFO:install:Found STP URLs for macOS ==13.*, ~=12.3
    Traceback (most recent call last):
    File "/Users/runner/work/1/s/./wpt", line 10, in
    wpt.main()
    File "/Users/runner/work/1/s/tools/wpt/wpt.py", line 220, in main
    rv = script(*args, **kwargs)
    File "/Users/runner/work/1/s/tools/wpt/install.py", line 89, in run
    install(browser, kwargs["component"], destination, channel, logger=logger,
    raise ValueError(f"no download for {system_version}")
    ValueError: no download for 10.16
    ##[error]Bash exited with code '1'.
    Finishing: Install Safari Technology Preview
    }
@TalbotG
Copy link
Contributor Author

TalbotG commented Jul 21, 2022

Trying to reach @web-platform-tests/admins , @web-platform-tests/wpt-core-team , @stephenmcgruer

@foolip
Copy link
Member

foolip commented Jul 21, 2022

I don't know exactly what's gone wrong here, but it's related to scraping https://developer.apple.com/safari/resources/ implemented in #34553.

@foolip
Copy link
Member

foolip commented Jul 21, 2022

@gsnedders what would you think of extracting the download URL into a file in the repo to pin it, so that we can keep running tests even when https://developer.apple.com/safari/resources/ changes?

@gsnedders
Copy link
Member

gsnedders commented Jul 22, 2022

@TalbotG what is echo $SYSTEM_VERSION_COMPAT?

presumably you have something that is setting this such that macOS is pretending to be 10.16 to avoid breaking applications that assume macOS versions would begin with 10.x for all eternity

[edit: doh, this is Azure Pipelines again, sorry!]

@gsnedders what would you think of extracting the download URL into a file in the repo to pin it, so that we can keep running tests even when https://developer.apple.com/safari/resources/ changes?

This is unlikely to work (as the outdated STPs cease being available), and regardless we need to know the system version (which is the problem here) to be able to know which URL to download.

gsnedders added a commit to gsnedders/web-platform-tests that referenced this issue Jul 22, 2022
…ling STP

Presumably Azure Pipelines (temporarily?) had an image which didn't
set SYSTEM_VERSION_COMPAT=1 by default, hence this previously appeared
to work without this.
@foolip
Copy link
Member

foolip commented Jul 22, 2022

Hmm, old STP downloads get deleted? In 2021 we had a pinned version for months, so I guess this is new, or they get deleted only after a long time?

gsnedders added a commit that referenced this issue Jul 22, 2022
Presumably Azure Pipelines (temporarily?) had an image which didn't
set SYSTEM_VERSION_COMPAT=1 by default, hence this previously appeared
to work without this.
@gsnedders
Copy link
Member

Hmm, old STP downloads get deleted? In 2021 we had a pinned version for months, so I guess this is new, or they get deleted only after a long time?

It is… not perfectly consistent as to whether it actually happens. 🙃

@TalbotG
Copy link
Contributor Author

TalbotG commented Jul 22, 2022

Sam Sneddon , Philip Jägenstedt ,

A few min. ago, I have created PR34943 and things appear to be okay, normal, as expected now.

Thank you for your assistance, your time and this fix.

Gérard

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants