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

WPT test warning after all runs #1266

Closed
Lightning00Blade opened this issue Sep 7, 2023 · 6 comments
Closed

WPT test warning after all runs #1266

Lightning00Blade opened this issue Sep 7, 2023 · 6 comments
Labels

Comments

@Lightning00Blade
Copy link
Collaborator

Lightning00Blade commented Sep 7, 2023

In local and CI runs we get the following error:

0:00.05 WARNING Falling back to getting metadata update properties from wptrunner browser product file, this will be removedTraceback (most recent call last):  File "/usr/local/google/home/nvitkov/bidi/wpt/wpt", line 10, in <module>    wpt.main()  File "/usr/local/google/home/nvitkov/bidi/wpt/tools/wpt/wpt.py", line 233, in main    rv = script(*args, **kwargs)         ^^^^^^^^^^^^^^^^^^^^^^^  File "/usr/local/google/home/nvitkov/bidi/wpt/tools/wpt/update.py", line 44, in update_expectations    update_properties = metadata.get_properties(properties_file=kwargs["properties_file"],                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^  File "/usr/local/google/home/nvitkov/bidi/wpt/tools/wptrunner/wptrunner/metadata.py", line 107, in get_properties    properties, dependents = products.load_product_update(config, product)                             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^  File "/usr/local/google/home/nvitkov/bidi/wpt/tools/wptrunner/wptrunner/products.py", line 54, in load_product_update    module = product_module(config, product)             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^  File "/usr/local/google/home/nvitkov/bidi/wpt/tools/wptrunner/wptrunner/products.py", line 9, in product_module    raise ValueError("Unknown product %s" % product)

Most likely there was a WPT infra change related to metadata, that could explain why our auto-update expectations have not been working lately.

@thiagowfx
Copy link
Contributor

Possibly related: web-platform-tests/wpt#41184

@thiagowfx thiagowfx added the wpt label Sep 7, 2023
@thiagowfx
Copy link
Contributor

Yup, this is the same as the issue above.

"product" is deprecated: https://github.com/web-platform-tests/wpt/blame/49d6aa17ff973bd570aec96f322b145b47977360/tools/wptrunner/wptrunner/metadata.py#L4

So we should remove --product chrome from our script.

That said, when we do it, we hit the same error as gsnedders@ in the bug above.

@thiagowfx
Copy link
Contributor

Possible culprit: web-platform-tests/wpt@64082e7

@thiagowfx
Copy link
Contributor

@foolip: Do you know how this issue could be resolved?

@foolip
Copy link
Member

foolip commented Sep 15, 2023

@thiagowfx I'm not familiar with the changes that caused this breakage, so I don't know off hand. But feel free to ping me if you try to debug this and can't work out how to fix it.

@thiagowfx thiagowfx removed their assignment Nov 23, 2023
@Lightning00Blade
Copy link
Collaborator Author

Closing as not an issue on our end, and I can't repo any more.

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

No branches or pull requests

3 participants