watch: a rebuild queued during a failed deploy is never deployed
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 2/5
- Thời gian dự kiến
- 1-3 giờ
- Mức phù hợp với người mới
- 78/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ệ
- node.js, typescript
Hướng nghiên cứu
Bắt đầu tại packages/cli/src/watch.ts và theo dõi cách triggerDeploy xử lý các nhánh thành công và thất bại của promise. Tái hiện lỗi bằng cách lưu một thay đổi trong khi deploy đang chạy, sau đó đảm bảo deploy được xếp hàng sẽ chạy sau cả hai kết quả; xác minh rằng nó không bị chặn bởi trạng thái giữ deploy được mô tả trong #47.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Problem
In packages/cli/src/watch.ts, if a rebuild happens while a deploy is running, triggerDeploy sets deployQueued = true. The .then handler starts the queued deploy, but the .catch handler only resets isDeploying and never checks deployQueued.
So if the running deploy fails, a change saved during it is silently left undeployed until the next file change. That's often exactly the fix for the failure.
Repro
- In
notation watch, start a deploy that will fail. - While it's running, save a change.
- The failed deploy finishes, and the saved change is not deployed until another edit.
Fix
Drain the queue in both paths, e.g. move the deployQueued check into a .finally, or call the same "done" function from .then and .catch.
Note that while #47 is open, the queued deploy would then hang on the deployment hold, so the two are worth fixing together.
- Ngôn ngữ chính
- TypeScript
- Star
- 54
- Fork
- 1
- Merge trung bình
- 1 giờ 45 phút
- Pull request đã merge (30 ngày)
- 1
Chuẩn bị môi trường
- Không có Dockerfile hay tệp Docker Compose
- Không có mẫu pull request
- Đọc hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- 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.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của notationlabs/notation
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 42/100
notationlabs/notation#47 ·
-
state provider configĐang mở
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 35/100
notationlabs/notation#34 ·
-
envs as configĐang mở
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 35/100
notationlabs/notation#33 ·
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 45/100
notationlabs/notation#16 ·
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 45/100
notationlabs/notation#15 ·
Tất cả issue của notationlabs/notation
Issue tương tự
-
DQ assisgnment closing at CRĐang mởenhancement
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
equinor/prisma-decision-web#919 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
[DC] OneLogin sync imports a partial user list when an API request failsCó thể đã có người làm @RaphaelFakhri đã nhận hôm nay. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 84/100
bitwarden/directory-connector#1276 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 62/100
home-assistant/installer#324 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Mondriaan
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100
knaw-huc/textannoviz#733 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
Maintainer thường phản hồi trong vòng 1 ngày