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

Bug: The build healthcheck system is double-counting failures #3272

Closed
adamfarley opened this issue Mar 3, 2023 · 4 comments
Closed

Bug: The build healthcheck system is double-counting failures #3272

adamfarley opened this issue Mar 3, 2023 · 4 comments
Assignees

Comments

@adamfarley
Copy link
Contributor

Summary

It appears that the system for reporting on the health of overnight builds is somehow counting failures twice.

Example: https://ci.adoptium.net/job/nightlyBuildAndTestStats_temurin/981//console

Search for openjdk8-pipeline, see the double entry, and also note that the total failures at the end are "8", despite there being only one set of four failures in the double-printed jdk8 pipeline.

How it should work

It should count each failed build only once.

Next steps

To look at the code that fetches data from TRSS and figure out why we're receiving two sets of the same data, or why we're duplicating the data we receive.

@adamfarley adamfarley self-assigned this Mar 3, 2023
@smlambert
Copy link
Contributor

Likely related to the fact that ci.adoptopenjdk.net got renamed to ci.adoptium.net and we are currently monitoring both in TRSS, so double-counting. I have not yet deleted the configuration in TRSS that has us monitoring ci.adoptopenjdk.net, as I was testing whether I could just mark the monitoring to "No" but hitting a known bug adoptium/aqa-test-tools#775. I will remove the ci.adoptopenjdk.net monitoring completely which should fix this issue.

@adamfarley
Copy link
Contributor Author

Thanks Shelley. 🙂

@smlambert
Copy link
Contributor

Please keep this open until we confirm its fixed by my changes, I will plan to make those config changes to TRSS on the weekend.

@adamfarley
Copy link
Contributor Author

I double-checked the results, and I'm not seeing the double-counting anymore. Good job Shelley! 😀

@adoptium adoptium deleted a comment Apr 3, 2023
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

No branches or pull requests

2 participants