chore: update concourse checkout scripts#440
Conversation
WalkthroughConcourse CI scripts and .gitignore were updated to move several eloq module references into a new Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches
🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In `@concourse/scripts/tag.sh`:
- Around line 9-12: Update the symlink created for eloq_log_service in the
tag.sh script: change the target path used in the ln -s invocation that
currently creates src/mongo/db/modules/eloq/eloq_log_service so it instead
creates src/mongo/db/modules/eloq/data_substrate/eloq_log_service, matching the
other build scripts (ensure the ln -s line referencing
$WORKSPACE/eloq_logservice_src is modified accordingly).
🧹 Nitpick comments (1)
concourse/scripts/git-checkout.sh (1)
35-46: Good addition of error handling; minor concern withcheckout -bwhen branch already exists remotely.The explicit error handling for missing release branches is a good improvement. However,
git checkout -b "$REL_BRANCH" "origin/$REL_BRANCH"will fail if a local branch with the same name already exists (e.g., from a previous run). Consider usinggit checkout "$REL_BRANCH"orgit switch -c "$REL_BRANCH" --track "origin/$REL_BRANCH"with appropriate error handling.♻️ Suggested improvement for idempotent checkout
if git ls-remote --heads origin "$REL_BRANCH" | grep -q "$REL_BRANCH"; then - git checkout -b "$REL_BRANCH" "origin/$REL_BRANCH" + git checkout "$REL_BRANCH" 2>/dev/null || git checkout -b "$REL_BRANCH" "origin/$REL_BRANCH" else
Summary by CodeRabbit
Chores
Bug Fixes
✏️ Tip: You can customize this high-level summary in your review settings.