ci(machines): mark container mounts as safe git directories

[njzjz-bot]

Problem

• PR fix: mark workspace as safe for git in machines.yml CI workflow #587 only marked the host runner workspace as a safe Git directory.
• The failing build happens inside CI containers where the repository is mounted at /dpdispatcher, so git still aborts with a dubious ownership error during setuptools/vcs-versioning introspection.

Change

• Keep the workflow-level safe-directory fix added in fix: mark workspace as safe for git in machines.yml CI workflow #587.
• Mark /dpdispatcher as a safe Git directory inside the ssh, ssh_rsync, slurm, and pbs CI scripts before installing/building the package.

Notes

• This replaces PR fix: mark workspace as safe for git in machines.yml CI workflow #587 because its head branch is in the upstream repository and is not writable from the bot account.
• The root cause from the failed log is fatal: detected dubious ownership in repository at '/dpdispatcher'.

Authored by OpenClaw (model: custom-chat-jinzhezeng-group/gpt-5.4)

[coderabbitai]

Warning

Rate limit exceeded

@njzjz-bot has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 18 minutes and 50 seconds before requesting another review.

Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 18 minutes and 50 seconds.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 28324123-8674-4174-a9b3-195111022ba7

📥 Commits

Reviewing files that changed from the base of the PR and between f7541f8 and 571e25a.

📒 Files selected for processing (5)

• .github/workflows/machines.yml
• ci/pbs.sh
• ci/slurm.sh
• ci/ssh.sh
• ci/ssh_rsync.sh

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 48.33%. Comparing base (f7541f8) to head (571e25a).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files

@@           Coverage Diff           @@
##           master     #588   +/-   ##
=======================================
  Coverage   48.33%   48.33%           
=======================================
  Files          40       40           
  Lines        3958     3958           
=======================================
  Hits         1913     1913           
  Misses       2045     2045           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[Copilot]

Pull request overview

Fixes CI failures in the “machines” GitHub Actions workflow caused by Git’s “dubious ownership” protections when setuptools_scm introspects the repo inside containers mounted at /dpdispatcher.

Changes:

• Add a workflow step to mark $GITHUB_WORKSPACE as a Git safe directory.
• Update CI container scripts (ssh, ssh_rsync, slurm, pbs) to mark /dpdispatcher as a Git safe directory before installing/building.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
ci/ssh.sh	Configure /dpdispatcher as a safe Git directory inside the test container prior to install/test.
ci/ssh_rsync.sh	Same safe-directory configuration inside the rsync-enabled test container.
ci/slurm.sh	Configure /dpdispatcher as safe inside the Slurm controller container before build/test.
ci/pbs.sh	Configure /dpdispatcher as safe inside the PBS master container before build/install.
.github/workflows/machines.yml	Add workflow-level safe-directory config for the host runner workspace.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[njzjz]

LGTM