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

Should Edge Canary have the 'experimental' label? #1635

Open
stephenmcgruer opened this issue Nov 13, 2019 · 6 comments
Open

Should Edge Canary have the 'experimental' label? #1635

stephenmcgruer opened this issue Nov 13, 2019 · 6 comments
Labels
blocked This issue is blocked on the resolution of another.

Comments

@stephenmcgruer
Copy link
Contributor

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.

@mustjab
Copy link

mustjab commented Nov 13, 2019

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.

@foolip
Copy link
Member

foolip commented Nov 13, 2019

Sounds like we should do it, but also at the same time stop applying the label to Edge Dev. #1575 probably needs to be fixed first though so that one can find older Dev and Canary runs without knowing when the experimental label flipped.

#1519 is also tangentially related.

@stephenmcgruer
Copy link
Contributor Author

I agree; marking this blocked on #1575

@stephenmcgruer stephenmcgruer added the blocked This issue is blocked on the resolution of another. label Nov 13, 2019
@Hexcles
Copy link
Member

Hexcles commented Nov 13, 2019

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 experimental label in the processor is technically correct because, incidentally, Edge Canary does not have experimental web platform features enabled. We should fix that and start labelling new Edge Canary runs as experimental, but we probably shouldn't go back and "fix" the old runs -- that'd skew our definition of "experimental".

@foolip
Copy link
Member

foolip commented Apr 29, 2022

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?

@mwjacksonmsft
Copy link
Contributor

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?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocked This issue is blocked on the resolution of another.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants