Record migration warnings where they arise
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ó
- 4/5
- Thời gian dự kiến
- 1-2 ngày
- Mức phù hợp với người mới
- 45/100
Hướng nghiên cứu
Start with cutover.py, references.py, and cleanup.py, then trace their Jobs and existing warning handling, including the changes in #60. Update operation return values and callers so each pass logs warnings it adds, including on failure; preserve history and deduplication, and revise retry messages. Update the related docstrings and tests for interrupted operations and repeated passes.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Proposed Changes
Update the migration operations in cutover.py, references.py and cleanup.py to record warnings through MigrationRun.record_warnings() where they arise, and return only counts or other operation results.
Update the cutover, references and cleanup Jobs to log warnings newly added by their own pass, including when a later operation fails. Preserve the run's warning history and deduplication, and update retry messages to point to that history when a warning has already been recorded.
Update callers, docstrings and tests for the revised return values, interrupted operations and repeated passes. This follows up on the warning-handling changes in #60.
Justification
Migration helpers currently use different conventions: some return warnings for their callers to record, while others both record and return them. This adds warning-list plumbing and can lose earlier warnings if later work raises before the list is recorded.
Recording warnings in the operation that produces them gives migration one convention, simplifies callers and preserves already committed warnings when later work fails.
- Ngôn ngữ chính
- Python
- Star
- 8
- Fork
- 0
- Merge trung bình
- 14 giờ 51 phút
- Pull request đã merge (30 ngày)
- 11
Chuẩn bị môi trường
- Không có Dockerfile hay tệp Docker Compose
- 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 netbox-community/netbox-scripts
-
app: scripts status: backlog type: documentation
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 86/100
netbox-community/netbox-scripts#68 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
app: scripts severity: low status: backlog type: bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
netbox-community/netbox-scripts#64 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
app: scripts severity: low status: backlog type: bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
netbox-community/netbox-scripts#62 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
app: scripts status: accepted type: documentation
Độ khó 2/5 Nửa ngày Mức phù hợp với người mới 82/100
netbox-community/netbox-scripts#40 · 1 bình luận · 1 người được giao ·
Maintainer thường phản hồi trong vòng 1 ngày
-
app: scripts severity: low status: backlog type: bug
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 58/100
netbox-community/netbox-scripts#67 ·
Maintainer thường phản hồi trong vòng 1 ngày
Tất cả issue của netbox-community/netbox-scripts
Issue tương tự
-
priority middle
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 62/100
KATO-Hiro/AtCoderClans#12940 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 76/100
huggingface/huggingface_hub#5283 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 1/5 Dưới một 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
-
enhancement
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 86/100
ccxt/ccxt#31045 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
bug example
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 86/100
Maintainer thường phản hồi trong vòng 1 ngày