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

To make sure the firefox menu can show up #14421

Merged
merged 1 commit into from
Mar 7, 2022

Conversation

tinawang123
Copy link
Contributor

@tinawang123 tinawang123 commented Mar 4, 2022

@hjluo
Copy link
Contributor

hjluo commented Mar 4, 2022

LGTM

tests/x11/firefox.pm Outdated Show resolved Hide resolved
To make sure the firefox menu can show up
@lemon-suse lemon-suse merged commit 4587e37 into os-autoinst:master Mar 7, 2022
@DimStar77
Copy link
Contributor

Based on feeling, it is worse:
https://openqa.opensuse.org/tests/2231586#step/firefox/13

before, I'd only seen the issue on 42.x upgrade scenarios, now I see it on regular installs

@okurz
Copy link
Member

okurz commented Mar 7, 2022

Just revert it. You can't verify a fix for a sporadic issue with just a single verification run. I suggest to use https://progress.opensuse.org/projects/openqatests/wiki/Wiki#Statistical-investigation for both the analysis of the original issue and fail rate as well as any attempts of trying to improve the situation

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