We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
@stephenmcgruer noticed in web-platform-tests/wpt#19975 (comment) that https://ecosystem-infra-rotation.appspot.com/ isn't reflecting the current problem with old Edge Dev results. (I assumed that it did but didn't check.)
The problem is here: https://github.com/foolip/ecosystem-infra-rotation/blob/3a52deb7efc4caeede433b58e850623d681737aa/webapp/components/ecosystem-infra-rotation.html#L116-L125
Because the run end time is used, the check passes because the now most recent Edge Dev run started quite long after the commit itself.
Unfortunately, we don't store the commit time. web-platform-tests/wpt.fyi#310 was about this and mostly resolved by web-platform-tests/wpt.fyi#349, but it doesn't really help if the run itself starts very late, as was the case here.
Another issue here is that each run is checked individually so this check can pass even if there aren't any aligned runs.
The text was updated successfully, but these errors were encountered:
Also need to check for WebKitGTK runs.
Sorry, something went wrong.
We added WebKitGTK in #28
stephenmcgruer
No branches or pull requests
@stephenmcgruer noticed in web-platform-tests/wpt#19975 (comment) that https://ecosystem-infra-rotation.appspot.com/ isn't reflecting the current problem with old Edge Dev results. (I assumed that it did but didn't check.)
The problem is here:
https://github.com/foolip/ecosystem-infra-rotation/blob/3a52deb7efc4caeede433b58e850623d681737aa/webapp/components/ecosystem-infra-rotation.html#L116-L125
Because the run end time is used, the check passes because the now most recent Edge Dev run started quite long after the commit itself.
Unfortunately, we don't store the commit time. web-platform-tests/wpt.fyi#310 was about this and mostly resolved by web-platform-tests/wpt.fyi#349, but it doesn't really help if the run itself starts very late, as was the case here.
Another issue here is that each run is checked individually so this check can pass even if there aren't any aligned runs.
The text was updated successfully, but these errors were encountered: