[TECH DEBT] Remove deprecated Tornado IOLoop channel arguments in 3009.2

Đang mở
  • Có gì không ổn?
#70,612 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 3 ngày

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

Đánh giá

Độ khó
4/5
Thời gian dự kiến
3-5 ngày
Mức phù hợp với người mới
55/100
Loại issue
Tái cấu trúc
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
python
Lĩnh vực
backend, testing

Hướng nghiên cứu

Start with salt/channel/client.py and salt/utils/channel.py, then inspect the deprecation in PR #70604 and the tests covering these channel APIs. Remove Tornado IOLoop argument support from the listed entry points while preserving omitted and explicit asyncio loop behavior, loop identity and caller ownership. Done means Tornado inputs raise TypeError and normal asyncio calls remain warning-free.

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

Mô tả

tech-debt
Description of the tech debt to be addressed

PR #70604 deprecates passing a Tornado IOLoop as io_loop to client-channel and worker-pool transport APIs. During the migration period, these APIs warn and use the supplied Tornado object's underlying asyncio loop. Removal is scheduled for Salt 3009.2 and tracked by warn_until((3009, 2), ...).

Complete the removal in 3009.2. Native asyncio loop arguments remain supported. Omitting io_loop continues to select the current thread's asyncio loop.

Cleanup checklist
  • Reject Tornado IOLoop arguments with a clear TypeError in AsyncReqChannel.factory(), AsyncPubChannel.factory() and AsyncPubChannel.__init__() in salt/channel/client.py.
  • Apply the same removal to create_client_transport() and create_request_client() in salt/utils/channel.py.
  • Remove the expired channel-argument warning and obsolete compatibility code/imports. Keep shared aioloop() behavior still needed by unrelated, unmigrated callers.
  • Replace legacy warning/compatibility tests with tests that reject Tornado inputs. Retain coverage for omitted arguments, explicit asyncio arguments, selected-loop identity and caller ownership; normal asyncio calls must remain warning-free.
Timing and related work

Target release: 3009.2. Depends on the deprecation introduced in #70604. Part of #70366.

This follow-up concerns channel loop arguments. It does not schedule removal of the Tornado package or its other APIs.

Versions Report

N/A — planned cleanup for Salt 3009.2.

Ngôn ngữ chính
Python
Star
15.8k
Fork
5.6k
Merge trung bình
5 ngày 9 giờ
Pull request đã merge (30 ngày)
99

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

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 saltstack/salt

Tất cả issue của saltstack/salt

Issue tương tự

Thêm issue về Python

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.