Skip to content

Add: max-parallel: 1 for language matrix jobs - #387

Merged
OriolAbril merged 1 commit into
numfocus:mainfrom
AR21SM:fix/bugs
Aug 28, 2025
Merged

Add: max-parallel: 1 for language matrix jobs#387
OriolAbril merged 1 commit into
numfocus:mainfrom
AR21SM:fix/bugs

Conversation

@AR21SM

@AR21SM AR21SM commented Aug 28, 2025

Copy link
Copy Markdown
Member

Description

Changes Made

Matrix Job Concurrency Control:

  • Added max-parallel: 1 constraint to the language matrix strategy in deploy-book job
  • Forces sequential execution of language builds instead of parallel processing
  • Applies to all language variants processed through the matrix configuration

Technical Rationale

Problem: Parallel language matrix jobs can cause deployment conflicts when multiple builds attempt to write to GitHub Pages simultaneously, potentially leading to:

  • Race conditions during deployment
  • Incomplete or corrupted deployments
  • Build failures due to concurrent write operations ( see )

Solution: Sequential job execution prevents concurrent write conflicts.

Implementation Details

Before:

strategy:
  matrix:
    language: ${{ fromJSON(needs.get-info.outputs.languages) }}

After:

strategy:
  max-parallel: 1
  matrix:
    language: ${{ fromJSON(needs.get-info.outputs.languages) }}

Impact

Performance Trade-off:

  • Total deployment duration will increase proportionally to the number of supported languages
  • Eliminates deployment race conditions and ensures consistent site state

Deployment Stability:

  • Guarantees sequential language deployment to respective subdirectories
  • Prevents GitHub Pages service conflicts during multi-language publishing

@netlify

netlify Bot commented Aug 28, 2025

Copy link
Copy Markdown

Deploy Preview for stupendous-kringle-a86e81 ready!

Name Link
🔨 Latest commit 3a9e38d
🔍 Latest deploy log https://app.netlify.com/projects/stupendous-kringle-a86e81/deploys/68b060f479422c0008f1ffb7
😎 Deploy Preview https://deploy-preview-387--stupendous-kringle-a86e81.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@OriolAbril
OriolAbril merged commit 8fa1875 into numfocus:main Aug 28, 2025
4 of 5 checks passed
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

Successfully merging this pull request may close these issues.

2 participants