Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

🤖 perf: Daytona keepalive refreshes a shared sandbox once per workspace instead of once per sandbox

Đang mở
  • Có gì không ổn?
#7,236 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

Chưa có ai nhận issue này.

Đánh giá

Độ khó
3/5
Thời gian dự kiến
1-2 ngày
Mức phù hợp với người mới
68/100
Loại issue
Lỗi
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
typescript
Lĩnh vực
backend

Hướng nghiên cứu

Start with src/node/services/daytona/daytonaReconciler.ts around line 135 and read how the 60-second sweep resolves sandbox references and checks workspace activity. Add a reconciler test for an owner and two active forks sharing one sandbox. Done when a sweep makes one last-activity call for that sandbox, with the SSH token behavior also handled once per sandbox.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

Follow-up from the Codex review of #7100 (thread: https://github.com/coder/xum/pull/7100#discussion_r4353806733, src/node/services/daytona/daytonaReconciler.ts:135).

Problem

Forks and sub-agents of a Daytona workspace run as worktrees in the owner's sandbox, so several workspaces reference one sandbox. The reconciler's 60 s sweep walks workspaces, not sandboxes: every active workspace on the same sandbox sends its own POST /sandbox/{id}/last-activity (and may check its token), one after another. A large sub-agent tree multiplies control-plane calls and sweep latency for no gain, since one refresh per sandbox is enough.

Proposed fix

Group the stored references by resolved sandbox id, ask whether any workspace in the group is active, and refresh activity (and the SSH token, keyed by the stored alias) once per sandbox per sweep. Add a reconciler test with an owner and two active forks on one sandbox that expects one last-activity call.

Why it is deferred

#7100 runs its review loop under a P0/P1-only rule; P2 findings get a tracked follow-up.

Ngôn ngữ chính
TypeScript
Star
2.1k
Fork
139
Merge trung bình
3 giờ 45 phút
Pull request đã merge (30 ngày)
953

Chuẩn bị môi trường

  • Có Dockerfile hoặc tệp Docker Compose
  • Không có mẫu pull request
  • Không có hướng dẫn đóng góp

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của coder/xum

Tất cả issue của coder/xum

Issue tương tự

Thêm issue về TypeScript

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.