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

[cmake] Disable test-stressgraphics-chrome #16684

Closed

Conversation

dpiparo
Copy link
Member

@dpiparo dpiparo commented Oct 15, 2024

Until failures on Linux Fedora 39 are understood.

This PR is marked as emergency since all fedora39 builds are failing because of this test failure.

Until failures on Linux Fedora 39 are understood.
@dpiparo dpiparo added the pr:emergency🚨 This PR is fixing breakage, it's urgent and may overtake others label Oct 15, 2024
@dpiparo dpiparo requested a review from linev October 15, 2024 07:43
@linev
Copy link
Member

linev commented Oct 15, 2024

I submit #16685 which may improve chrome behavior.
If it does not help - we can merge this PR.

Copy link

Test Results

    14 files      14 suites   2d 13h 46m 7s ⏱️
 2 703 tests  2 703 ✅ 0 💤 0 ❌
35 751 runs  35 751 ✅ 0 💤 0 ❌

Results for commit 72c3868.

@guitargeek
Copy link
Contributor

guitargeek commented Oct 15, 2024

@dpiparo, Fedora 39 is end-of-life in less than a month: https://endoflife.date/fedora

Do we really want to disable the test on all platforms just for that? Personally, I would rather remove Fedora 39, and maybe add Fedora 41 in a few weeks when it comes out.

@linev
Copy link
Member

linev commented Oct 16, 2024

web-based stressGraphics tests disabled now

With next PR #16691 I trying to re-enable them - after adjusting chrome handling.
For last several test builds I saw no problem with stressgraphics-chrome on Fedora39

@linev linev closed this Oct 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr:emergency🚨 This PR is fixing breakage, it's urgent and may overtake others
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants