Remote log callback delivers no lines on the component-split branch (#1622)

Đang mở
  • Có gì không ổn?
#1,878 2 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ó
4/5
Thời gian dự kiến
3-5 ngày
Mức phù hợp với người mới
52/100
Loại issue
Lỗi
Độ rõ ràng
Khá rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
cmake, cpp, grpc
Lĩnh vực
backend-api-design, distributed-systems, testing-qa

Hướng nghiên cứu

Chạy ./build.sh libcuopt, sau đó thực thi cpp/build/tests/linear_programming/C_API_TEST với bộ lọc CpuOnlyWithServerTest.log_callback_remote, kiểm tra cổng 18500 trước. Theo dõi việc đăng ký callback xuyên suốt cuopt_mathopt và cuopt_grpc, bao gồm các instance logger theo từng component từ #1778. Được xem là hoàn tất khi remote solve vẫn thành công và các dòng log được stream đến callback.

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

Mô tả

Description

On #1622 (feat/split-routing-lp-libs), CpuOnlyWithServerTest.log_callback_remote fails:

Expected remote solve to deliver log lines; got 0 calls
cpp/tests/linear_programming/c_api_tests/c_api_tests.cpp:797: Failure

The remote solve itself succeeds — CpuOnlyWithServerTest.lp_solve and .mip_solve pass, returning Optimal with the correct objective. Only the streamed log lines never reach the callback registered through cuOptSetLogCallback.

Not caused by the cuopt_base removal

I hit this while removing cuopt_base from that branch and checked before attributing it. Rebuilding the branch with cuopt_base restored (cpp/CMakeLists.txt and cpp/src/CMakeLists.txt from the pre-removal commit) reproduces the failure identically, so it predates that change.

It is specific to the component split: the same test passes on split/4-cuopt-client-library, which carries a monolithic libcuopt.so and is otherwise up to date with main.

Where to look

cuopt_mathopt no longer hard-links gRPC on that branch. It holds nullable callback slots that libcuopt_grpc.so's ELF constructor fills via register_remote_solvers(), and the log callback has to cross the same boundary. Two candidates:

  • the log callback is registered on one side of that boundary and read on the other, so the streaming path sees a null or default-constructed callback
  • per-component loggers (#1778) mean the log lines the gRPC client streams are written to a different logger instance than the one the C API callback is attached to

The second is worth checking first, since #1778 gave each component library its own logger and the remote log path spans cuopt_mathopt and cuopt_grpc.

Reproduce

./build.sh libcuopt
cd cpp/build
./tests/linear_programming/C_API_TEST --gtest_filter='CpuOnlyWithServerTest.log_callback_remote'

Note that the fixture forks a cuopt_grpc_server on port 18500. A leaked server from an earlier run will make lp_solve and mip_solve fail too, with Error solving problem: 6, which is a separate and purely local artifact — check ss -ltnp | grep 18500 before drawing conclusions.

Related: #1622, #1778, #1510.

Ngôn ngữ chính
Cuda
Star
1.1k
Fork
249
Merge trung bình
1 ngày 15 giờ
Pull request đã merge (30 ngày)
126

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 NVIDIA/cuopt

Tất cả issue của NVIDIA/cuopt

Issue tương tự

Thêm issue về Backend & API Design

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.