Skip to content

perf: tune healthcheck intervals for squid and api-proxy containers#1803

Draft
Copilot wants to merge 2 commits intomainfrom
copilot/tune-healthcheck-intervals-squid-api-proxy
Draft

perf: tune healthcheck intervals for squid and api-proxy containers#1803
Copilot wants to merge 2 commits intomainfrom
copilot/tune-healthcheck-intervals-squid-api-proxy

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 8, 2026

Squid and api-proxy healthchecks use conservative intervals (5s interval, 5-10s start_period) that add ~6-8s idle wait to every sandbox invocation. Both services typically become healthy in 1-2s.

  • src/docker-manager.ts: Set interval: '1s', start_period: '2s', timeout: '1s' for both squid and api-proxy healthchecks. retries: 5 unchanged.
  • docs/api-proxy-sidecar.md, src/types.ts: Update docs/examples to match.
Service interval start_period timeout
Squid 5s → 1s 10s → 2s 3s → 1s
API Proxy 5s → 1s 5s → 2s 3s → 1s

Copilot AI changed the title [WIP] Tune healthcheck intervals for squid and api-proxy containers perf: tune healthcheck intervals for squid and api-proxy containers Apr 8, 2026
Copilot AI requested a review from Mossaka April 8, 2026 18:57
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 8, 2026

Documentation Preview

Documentation build failed for this PR. View logs.

Built from commit c43877c

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 8, 2026

✅ Coverage Check Passed

Overall Coverage

Metric Base PR Delta
Lines 86.22% 86.31% 📈 +0.09%
Statements 86.08% 86.17% 📈 +0.09%
Functions 87.50% 87.50% ➡️ +0.00%
Branches 78.81% 78.86% 📈 +0.05%
📁 Per-file Coverage Changes (1 files)
File Lines (Before → After) Statements (Before → After)
src/docker-manager.ts 86.6% → 86.9% (+0.36%) 86.0% → 86.4% (+0.35%)

Coverage comparison generated by scripts/ci/compare-coverage.ts

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 8, 2026

Smoke test results — PASS ✅

  • ✅ GitHub MCP: "chore: upgrade smoke-copilot workflow to gh-aw v0.67.3" / "fix: normalize API target env vars to bare hostnames via URL parsing"
  • ✅ Playwright: github.com title contains "GitHub"
  • ✅ File write: /tmp/gh-aw/agent/smoke-test-claude-24152900341.txt created
  • ✅ Bash: file content verified

💥 [THE END] — Illustrated by Smoke Claude

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 8, 2026

Smoke Test Results

PR titles: "fix: normalize API target env vars to bare hostnames via URL parsing"; "feat: forward OIDC env vars into agent container"
GitHub MCP (last 2 merged PRs): ✅
SafeInputs GH CLI (safeinputs-gh): ❌ (tool not available in this run)
Playwright (github.com title contains GitHub): ❌ (EACCES writing Playwright log output)
Tavily search: ❌ (Tavily MCP tool not available)
File write + bash cat verify: ✅
Discussion query/comment: ❌ (github-discussion-query tool not available)
Build (npm ci && npm run build): ✅
Overall status: FAIL

🔮 The oracle has spoken through Smoke Codex

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 8, 2026

Smoke Test: GitHub Actions Services Connectivity ✅

All checks passed:

Service Check Result
Redis host.docker.internal:6379 PING +PONG
PostgreSQL host.docker.internal:5432 pg_isready accepting connections ✅
PostgreSQL smoketest DB SELECT 1 returned 1

🔌 Service connectivity validated by Smoke Services

@github-actions github-actions bot mentioned this pull request Apr 8, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 8, 2026

Smoke Test Results — PR #1803

Test Result
GitHub MCP connectivity
GitHub.com HTTP connectivity
File write/read

PR: perf: tune healthcheck intervals for squid and api-proxy containers
Author: @Copilot | Assignees: @Mossaka, @Copilot

Overall: PASS

📰 BREAKING: Report filed by Smoke Copilot

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 8, 2026

Chroot Version Comparison Results

Runtime Host Version Chroot Version Match?
Python Python 3.12.13 Python 3.12.3
Node.js v24.14.1 v20.20.2
Go go1.22.12 go1.22.12

Overall: ❌ Not all tests passed — Python and Node.js versions differ between host and chroot environments.

Tested by Smoke Chroot

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

perf: tune healthcheck intervals for squid and api-proxy containers

2 participants