jamesfredley opened a new pull request, #16110:
URL: https://github.com/apache/grails-core/pull/16110

   ## Depends On
   
   This PR must not merge until apache/grails-github-actions#110 is merged to 
`asf` and the updated `deploy-github-pages@asf` ref is verified.
   
   ## Summary
   
   Remove the job-level `grails-docs-publish` queue from snapshot and release 
documentation publishing. Once the action-level optimistic retry is available, 
independent documentation jobs can run in parallel and let Git's normal 
fast-forward rules coordinate updates to the destination branch.
   
   The diff removes only the two queue declarations added by #15988. It does 
not change documentation generation, credentials, destination paths, action 
references, or workflow-level release coordination.
   
   ## Why Remove the Queue
   
   GitHub Actions concurrency groups coordinate runs only within one 
repository. Grails Core, `grails-static-website`, and future publishers such as 
Grails Forge can all target the same website repository, but a 
`grails-docs-publish` group in Grails Core cannot coordinate those other 
repositories. The manual `release-publish-docs.yml` workflow is also a 
publisher and has never used this group.
   
   Within Grails Core, the static group makes unrelated release documentation 
jobs serial:
   
   - the [8.0.0-M5 release 
run](https://github.com/apache/grails-core/actions/runs/30769584508) waited 18 
minutes 7 seconds for its documentation job to start;
   - the [7.2.2 release 
run](https://github.com/apache/grails-core/actions/runs/30769598969) waited 11 
minutes 15 seconds; and
   - the [7.1.5 release 
run](https://github.com/apache/grails-core/actions/runs/30769609958) was still 
queued after more than 6 minutes when observed.
   
   This turned a parallel release sequence into a serial one and required 
manual retriggering.
   
   ## Replacement Safety
   
   apache/grails-github-actions#110 retries only genuine non-fast-forward 
races. It fetches the new destination tip into the existing shallow checkout, 
verifies descendant history, rebases the unpublished local deployment commit, 
and retries a normal push. It never force-pushes or auto-resolves true 
conflicts.
   
   Merge order is therefore mandatory:
   
   1. Merge apache/grails-github-actions#110 to `asf`.
   2. Verify `deploy-github-pages@asf` contains the retry implementation.
   3. Merge this PR into `7.0.x`.
   4. Merge forward through the maintained branches normally.
   
   ## Preserved Concurrency
   
   The workflow-level `release-pipeline-${{ 
github.event.release.target_commitish }}` group remains unchanged. This PR 
removes only the documentation publishing mutex.
   
   ## Verification
   
   - Both edited workflow files parse successfully as YAML.
   - `git diff --check` passes.
   - The diff is exactly 12 deletions across two workflow files.
   - `grails-docs-publish` no longer appears in `.github/workflows`.
   - `release-publish-docs.yml` is byte-identical.
   - No Gradle task was run because this is a YAML-only deletion.
   


-- 
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