Skip to content

fix(ci): duplicate run-id#2409

Merged
whutchinson98 merged 1 commit intomainfrom
hutch/fix-ci-runner-contention
Apr 7, 2026
Merged

fix(ci): duplicate run-id#2409
whutchinson98 merged 1 commit intomainfrom
hutch/fix-ci-runner-contention

Conversation

@whutchinson98
Copy link
Copy Markdown
Member

No description provided.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Apr 7, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: 993dcd30-8b13-445d-9a9f-230e59b90ed9

📥 Commits

Reviewing files that changed from the base of the PR and between bc366d9 and b60314e.

📒 Files selected for processing (2)
  • .github/workflows/deploy-all-services.yml
  • .github/workflows/deploy-cloud-storage-on-push.yml

📝 Walkthrough

Summary by CodeRabbit

  • Chores
    • Enhanced deployment workflow configurations to provide improved isolation and tracking during service deployment processes.

Walkthrough

Updated the runner label configuration in two GitHub workflow files to make the job isolation key more granular. The run-id value now includes the current matrix service name, appending it to the existing run ID instead of using only the run ID.

Changes

Cohort / File(s) Summary
Workflow Runner Configuration
.github/workflows/deploy-all-services.yml, .github/workflows/deploy-cloud-storage-on-push.yml
Updated the runs-on array's run-id value in the deploy-services job to include matrix service information. Changed from ${{github.run_id}} to ${{github.run_id}}-${{ matrix.service }} for more granular job isolation.
🚥 Pre-merge checks | ✅ 1 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive No description was provided, making it impossible to assess whether the description relates to the changeset. Add a description explaining why the run-id is being made more granular by appending the matrix.service value to prevent runner contention issues.
✅ Passed checks (1 passed)
Check name Status Explanation
Title check ✅ Passed The title 'fix(ci): duplicate run-id' follows conventional commits format with a clear 'fix' prefix, appropriate scope 'ci', and is under 72 characters.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@whutchinson98 whutchinson98 merged commit 3a50f88 into main Apr 7, 2026
22 checks passed
@whutchinson98 whutchinson98 deleted the hutch/fix-ci-runner-contention branch April 7, 2026 16:11
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.

1 participant