mirror of
https://github.com/tiennm99/ccs.git
synced 2026-07-17 16:16:37 +00:00
Revert "ci(ai-review): move PR-Agent jobs to ubuntu-latest"
This reverts the runs-on change for ai-review.yml. All CI/CD must stay on home infra (claw, docker LXC, etc); GitHub-hosted runners are forbidden. The qodo-ai/pr-agent docker action mount issue will be addressed by either configuring the self-hosted runner image to use path-identical bind mounts so nested-docker volume sources resolve correctly, or by switching to native CLI invocation (pip install pr-agent) instead of the docker action. Tracked separately; not blocking PR #1209.
This commit is contained in:
@@ -7,7 +7,7 @@ function resolvePath(relativePath: string) {
|
||||
}
|
||||
|
||||
describe('PR-Agent review lane migration', () => {
|
||||
test('keeps ai-review.yml as the PR-Agent workflow on a GitHub-hosted runner', () => {
|
||||
test('keeps ai-review.yml as the PR-Agent workflow on the self-hosted cliproxy runner', () => {
|
||||
const workflowPath = resolvePath('../../../../.github/workflows/ai-review.yml');
|
||||
const prAgentConfigPath = resolvePath('../../../../.pr_agent.toml');
|
||||
|
||||
@@ -19,14 +19,7 @@ describe('PR-Agent review lane migration', () => {
|
||||
const appTokenUsages = workflow.match(/uses: actions\/create-github-app-token@v1/g) ?? [];
|
||||
|
||||
expect(workflow).toContain('name: AI Code Review');
|
||||
// AI review jobs run on GitHub-hosted runners. Self-hosted runners that
|
||||
// run inside a Docker container (myoung34/github-runner) cannot host
|
||||
// qodo-ai/pr-agent because the action's docker run command uses a volume
|
||||
// mount path (/tmp/runner/work/_temp/_github_workflow) that resolves on
|
||||
// the host filesystem, not inside the runner container, leaving
|
||||
// /github/workflow/event.json missing in the action.
|
||||
expect(workflow).toContain('runs-on: ubuntu-latest');
|
||||
expect(workflow).not.toContain('runs-on: [self-hosted, cliproxy]');
|
||||
expect(workflow).toContain('runs-on: [self-hosted, cliproxy]');
|
||||
expect(workflow).toContain('uses: qodo-ai/pr-agent');
|
||||
expect(appTokenUsages).toHaveLength(2);
|
||||
expect(workflow).toContain('OPENAI.API_BASE');
|
||||
|
||||
Reference in New Issue
Block a user