-
-
Notifications
You must be signed in to change notification settings - Fork 330
Creating lists of intermittently failing tests #6472
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
base: master
Are you sure you want to change the base?
Creating lists of intermittently failing tests #6472
Conversation
This change lists known unreliable tests in a consistent format. Signed-off-by: Adam Farley <[email protected]>
For the July 2025 release. Signed-off-by: Adam Farley <[email protected]>
Signed-off-by: Adam Farley <[email protected]>
Signed-off-by: Adam Farley <[email protected]>
Signed-off-by: Adam Farley <[email protected]>
Signed-off-by: Adam Farley <[email protected]>
Signed-off-by: Adam Farley <[email protected]>
Signed-off-by: Adam Farley <[email protected]>
Signed-off-by: Adam Farley <[email protected]>
Signed-off-by: Adam Farley <[email protected]>
Requesting reviews from @ShelleyLambert and @sophia-guo please. |
thanks @adamfarley - a heads up that my active Github tag is @smlambert ... but I did see this review request in any event. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for compiling this list @adamfarley. Only openjdk testcase failures are handled by jtreg exclude Problemlist format. For the other test groups that are part of AQAvit, they get excluded via playlist exclusions.
I would have expected the perf and system targets to be listed via playlist files (which do not use a different file for each version as the problem lists do).
I guess it depends on what we intend to do with these lists. If we are adding a feature to exclude unreliable targets on the fly, or post-process failures against a list, we have to consider what format to use, and how to organize them within the directory (as different vendors may eventually have different lists).
I noticed that. For Perf and Systemtest, I used the jtreg format for consistency, rather than explicit usability (though a consistent format does make any future parsing easier)
I considered that, but opted not to because I'm unaware of a mechanism for disabling targets via the playlist files sometimes. As far as I know, for a given vendor, platform, and version, a target is either enabled or disabled. An openjdk test can include an extra exclude file dynamically, so we can choose whether to exclude the test in some runs or not.
My first priority with this task was to provide a unified list of all intermittently failing tests.
Specific uses were a secondary priority. My first thought was that they could be used as exclude files during releases (or during reruns, or reruns during a release), but I didn't want to sidetrack myself looking too much into applications. I figured that if we did pick an application for this data, the best thing I can do right now is to use a consistent format across the board, to make conversion/parsing easier later on. I'll add a note to the General Retrospective to start discussion into some of our options here. |
openjdk jtreg problemlists are the 'odd man out'... no other test group uses that format I do not want to store non-openjdk test material in problemlist format as it might lead people to think that is the commonly used format, which it is not. Agree that we need to have a requirements gathering session and design discussion on this (in terms of formats, and how to structure and where to locate such files). If this is merely a form of documentation, and not a plan for using for debug / temp exclude, then likely all could be in a doc folder. |
Fair. Perhaps I was over-focused on the OpenJDK ProblemList format due to the majority of the intermittent failures being OpenJDK tests.
In my mind, this is a debug/exclude tool first. Mind if I get the ball rolling on Slack? |
This change lists known unreliable tests in a consistent format.
Resolves #6471