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

Added basic develocity setup #25322 #25336

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

pzygielo
Copy link
Contributor

@pzygielo pzygielo commented Jan 21, 2025

@pzygielo

This comment was marked as resolved.

@pzygielo
Copy link
Contributor Author

Test failure analytics to better understand trends and causes around slow, failing, and flaky tests

Seems to be flaky test:

Is there already some value hidden in these build scans (compared here) that could help us understand that better and fix HttpRedirectTest?

@pzygielo pzygielo force-pushed the bdcs branch 2 times, most recently from 514f452 to e5a45b9 Compare January 23, 2025 19:24
@pzygielo pzygielo mentioned this pull request Jan 23, 2025
@ribafish
Copy link
Contributor

ribafish commented Jan 24, 2025

@pzygielo Re: flaky tests: No, Develocity by default doesn't gather more data by default, other than what's reported to the console log and part of the failure stacktace. If you know what data you'd be interested in, you can extend the build scan.

It looks like the org.glassfish.grizzly.config.HttpRedirectTest tests are the only flaky tests picked up - at least from the runs published until now.

Unrelated to this issue, but I'd recommend adding the profiles the build was executed with (at least on CI) as a custom value - this would allow you to also filter out just the runs with a specific profile, which might be useful since pretty much all your CI builds run install. I'm happy to add this improvement if you agree.

@pzygielo pzygielo marked this pull request as ready for review January 26, 2025 06:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants