-
Notifications
You must be signed in to change notification settings - Fork 92
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
Should Edge Canary have the 'experimental' label? #1635
Comments
Yes, don't see any reason why not. Let me know if I need to make any change on our end to set it correctly. |
I agree; marking this blocked on #1575 |
Sorry for being late to this series of label-related issues. IIRC, we decided to have Edge Dev on the home page by default because we do not have Chrome Canary on Linux and we want a more aligned comparison. The code for assigning the |
We're still running both Edge Dev and Canary every 3 hours, but this is pretty wasteful since we don't really use the Canary results. If we label Canary as experimental then we can probably just stop running Edge Dev. @KyleJu WDYT? |
I realize this is an old issue - but I'm happy to take a look at fixing it. With this change web-platform-tests/wpt#42008, we do enable experimental web platform features (amongst other things). The Edge tests are dependent upon MojoJS, which are only produced from Linux builds, and on Linux we only support dev, beta, and stable channels. If we were to run canary builds, we'd end up with mismatched MojoJS causing test failures. Additionally, I think we probably want to stick with dev channel, to help reduce noise as we ingest changes from Chromium. I think running canary channel today, would make it hard for us to reason about the different failures. I'd propose that we leave Edge Dev labelled experimental, and Edge Canary as nightly as it is today. If any of the above changes, then we could revisit. WDYT? |
Forked from #1539. Copying some comments over:
#1539 (comment)
"Do we have an issue for the fact that Edge Canary doesn't seem to be tagged with experimental? (Or is there a valid reason it shouldn't be)."
#1539 (comment)
"There’s no issue for that AFAIK, and I don’t recall how it came about. @mustjab do you know why it’s Edge Dev and not Canary that shows on wpt.fyi by default?"
#1539 (comment)
"The code reason, incidentally, would seem to be that results-processor/wptreport.py has a CHANNEL_TO_LABEL map that doesn't include 'canary' (it maps ['release', 'stable'] --> 'stable', ['beta'] --> 'beta', and ['dev', 'experimental', 'nightly', 'preview'] --> 'experimental'), and that the report from azure doesn't include 'experimental' either."
Assigning to mustjab to answer the query about whether Edge Canary should have the experimental label.
The text was updated successfully, but these errors were encountered: