jdaugherty commented on PR #15793: URL: https://github.com/apache/grails-core/pull/15793#issuecomment-4844135462
> > Stabilizes recurring CI flaky tests identified by mining ~100 recent failures of the `gradle.yml` and `groovy-joint-workflow.yml` workflows. Uses the project's existing Spock `@Retry` convention (already used in `GitHubCreateControllerSpec` and `MicronautErsatzRoundtripSpec`) plus root-cause synchronization hardening where feasible. > > All changes are confined to functional/example test apps under `grails-test-examples/`; no framework runtime behavior is altered. > > @jdaugherty Fredley did address the deterministic issues from the non-deterministic. @borinquenkid I'm ok with the wait's being added, I am not ok with the retry. We need to address the tests to make them non-flaky by adding waits. There are still `@Retry` changes in this PR. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
