-
-
Notifications
You must be signed in to change notification settings - Fork 314
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
make jcstress default targets enabled: #5668
Comments
We will not enable jcstress tests in our regular test runs at the project as we are conserving our machine resources. Other vendors are welcome to run the disabled targets, or enable on a per vendor basis. Given our constraints, but also that vendors are not constrained, I would leave this issue open temporarily for a week or so for others awareness, then close as won't fix. |
Yah, From the results i posted, I was thinking the same. I agree for sure with what you wrote, just few thoughts I would like to understand better:
|
In my opinion, these tests are good for testing GC changes in the upstream repo, so my inclination would be that GC devs would run these when testing new GC features or major changes (via Trestle pipeline). They could additionally be run on a not very often schedule or against tagged releases, but considered 'blocking' or 'required' tests (run in order to fill the gaps between times where they are run as part of developer testing).
Being concerned and having the human resources / time to investigate are 2 different matters. Should there be resources, I would start them off by looking at profile of the test nodes (how the configuration of the machines differ, etc) as I am certain there would be many differences. |
Fair enough. Issue done by me. TY! |
Thougts?
The text was updated successfully, but these errors were encountered: