Skip to content

Sync deployment jobs#62

Merged
FScholPer merged 1 commit intoeclipse-score:mainfrom
elektrobit-contrib:serialize-doc-deployment
Feb 17, 2026
Merged

Sync deployment jobs#62
FScholPer merged 1 commit intoeclipse-score:mainfrom
elektrobit-contrib:serialize-doc-deployment

Conversation

@lurtz
Copy link
Contributor

@lurtz lurtz commented Feb 16, 2026

Deployment jobs modify the gh-pages branch and concurrent modification needs to be prevented. Otherwise we get random build failures.

Note: I have not tested this myself and I actually do not want to because testing it is quite time consuming. This solutions is what came to my mind after analyzing the logs of eclipse-score/docs-as-code#404

Closes eclipse-score/docs-as-code#404

Deployment jobs modify the `gh-pages` branch and concurrent modification needs to be prevented. Otherwise we get random build failures.
@AlexanderLanin
Copy link
Member

AlexanderLanin commented Feb 16, 2026

Is this better/worse/different than job.concurrency?

@FScholPer FScholPer merged commit 216fe31 into eclipse-score:main Feb 17, 2026
2 checks passed
@lurtz
Copy link
Contributor Author

lurtz commented Feb 17, 2026

Is this better/worse/different than job.concurrency?

@AlexanderLanin I was not aware of that. Its documentation says the following:

When a concurrent job or workflow is queued, if another job or workflow using the same concurrency group in the repository is in progress, the queued job or workflow will be pending. Any existing pending job or workflow in the same concurrency group, if it exists, will be canceled and the new queued job or workflow will take its place.

This seems not to be the behavior we want for updating the gh-pages branch. This would mean some pull requests will have eventually no documentation deployed to that branch.

I think a better application of this option is e.g. checking branches while new commits are pushed.

@lurtz lurtz deleted the serialize-doc-deployment branch February 17, 2026 09:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

race condition

3 participants