jamesfredley commented on PR #15467:
URL: https://github.com/apache/grails-core/pull/15467#issuecomment-4348090868

   ## CI status update (commit 2ca302b164 + bbf5e5320c)
   
   After the latest commits, all 31 in-scope CI jobs pass. Key fixes applied to 
the failing checks:
   
   ### Fixed in bbf5e53 - `Validate Dependency Versions`
   
   The 8.0.x merge added `grails-base-bom`, `grails-hibernate5-bom` and 
`grails-micronaut-bom` to `GrailsDependencyValidatorPlugin.BOM_PROJECT_NAMES` 
and updated the Micronaut test examples to use 
`enforcedPlatform(grails-micronaut-bom)`. The validator's `detectBomPath()` 
returned the **first** BOM dependency it encountered. Combined with 
`applyGrailsBom()` injecting `platform(grails-bom)` on every declarable 
configuration, the validator was using `grails-bom` as the reference (Groovy 
4.0.31 / Spock 2.3-groovy-4.0) while the actually-resolved versions reflected 
`grails-micronaut-bom` (Groovy 5.0.5 / Spock 2.4-groovy-5.0). Fix: 
`detectBomPath()` now prefers an `enforcedPlatform` declaration over a regular 
`platform` so the validator references the BOM whose constraints actually win 
at resolution.
   
   ### Re-run resolved - `Functional Tests (Java 21, indy=true)`
   
   A Geb timing flake on `UserControllerSpec > User list` in 
`grails-test-examples-scaffolding` (Spring Security OAuth2 redirect to `/login` 
not yet ready when Geb's 10s wait expired). Same test passed on Java 21 
indy=false, Java 25 indy=false, all five Build Grails-Core matrices, and after 
a re-run on indy=true. Not a regression introduced by this PR.
   
   ### Pre-existing failure (out of scope) - `build_grails` (CI - Groovy Joint 
Validation Build)
   
   This workflow has been failing on every 8.0.x build since the BOM split PR 
`apache/grails-core#15608` merged into 8.0.x on 2026-04-28 (last 6 8.0.x 
commits all fail this workflow). The joint workflow's `sed -i` only rewrites 
`'groovy.version' :` lines in `dependencies.gradle`; it doesn't touch 
`'spock.version'`. After the BOM split, the Micronaut profile pins Spock 
2.4-groovy-5.0 which is incompatible with the Groovy 4.0.32-SNAPSHOT the 
workflow injects, breaking `compileIntegrationTestGroovy` for 
`grails-test-examples-issue-11767`, `-micronaut-groovy-only` and `-micronaut`. 
This is a 8.0.x infrastructure issue (not introduced or affected by this PR) 
and should be fixed separately - either by extending the sed substitution or by 
excluding Micronaut projects from the joint workflow when they require a 
different Spock major.
   
   ### Summary
   | Status | Count | Notes |
   |---|---|---|
   | ✅ pass | 31 | including my fix for the validator |
   | ⏭️ skip | 5 | release-only jobs |
   | ❌ fail | 1 | `build_grails` - pre-existing 8.0.x issue, not caused by 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]

Reply via email to