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

Stabilize tls_stress_test metrics queries under exhausted session caps

Đã đóng
  • Có gì không ổn?
#8,167 1 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
48/100
Loại issue
Lỗi
Độ rõ ràng
Khá rõ ràng
Mức độ hoạt động
Ít trao đổi
Công nghệ
python
Lĩnh vực
networking, testing-qa

Hướng nghiên cứu

Bắt đầu trong tests/connections.py tại get_session_metrics(), run_connection_caps_tests() và hàm lồng nhau create_connections_until_exhaustion(), sau đó kiểm tra tests/infra/clients.py về việc chuyển đổi httpx.ReadError. Chạy tls_stress_test nhiều lần thông qua tests.sh trên Azure Linux 4 và xác định hành vi dự kiến của các session hiện có khi đạt đến các cap. Hoàn thành khi các lỗi metric nhất thời không còn khiến test bị flaky, trong khi các assertion về session đang hoạt động, đỉnh, soft-cap và hard-cap vẫn có thể xử lý được.

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

Mô tả

ci failed-test testing

Summary

tls_stress_test is intermittently failing in AL4 VMSS Virtual C while the
connection-cap scenario deliberately exhausts sessions and file descriptors.
After expected connection failures, the test performs an unguarded
GET /node/metrics through clients[0]. That request may be reset or receive
503 SessionCapExhausted, failing the whole CI job.

This has occurred on unrelated PRs and predates the PR where it was most
recently observed.

Most recent occurrence

tests/connections.py:231 run_connection_caps_tests
tests/connections.py:176 create_connections_until_exhaustion
  r = clients[0].get("/node/metrics")
tests/infra/clients.py:696
  raise CCFIOException from exc
infra.clients.CCFIOException

The underlying exception was:

ConnectionResetError: [Errno 104] Connection reset by peer
httpx.ReadError: [Errno 104] Connection reset by peer

The normal VMSS Virtual C job passed on the same PR commit; only the AL4 job
failed:

Evidence that this is pre-existing and intermittent

The same request at tests/connections.py:176 failed with
infra.clients.CCFIOException on July 26:

That recovery PR did not modify tests/connections.py or the TLS/session-cap
implementation. Its preceding AL4 run passed, this run failed, and its next 11
AL4 runs passed without a change to tests/connections.py.

Related failures in the same connection-cap scenario have also returned
503 SessionCapExhausted from metrics queries:

The metrics request at tests/connections.py:176 dates from 2021. The loop
immediately before it already treats CCFConnectionException,
CCFIOException, and RuntimeError as expected while searching for the
session/file-descriptor limit, but the subsequent metrics request has no
equivalent handling.

Relevant code

  • tests/connections.py
    • get_session_metrics()
    • run_connection_caps_tests()
    • nested create_connections_until_exhaustion()
    • particularly the unguarded clients[0].get("/node/metrics")
  • tests/infra/clients.py
    • conversion of httpx.ReadError to CCFIOException

Investigation notes

Determine which invariant is intended:

  1. An already admitted session must remain usable while new sessions are
    rejected. If so, the reset/503 may expose a CCF session-cap bug and the test
    should retain a strict assertion.
  2. A reset or temporary 503 is valid while the test is deliberately exhausting
    file descriptors and session caps. If so, the test should explicitly wait
    for cap state to settle and retry the metrics query using an appropriate
    existing session, rather than failing on a timing-dependent request.

Avoid simply swallowing the exception. Preserve the assertions on active/peak
session metrics and ensure a persistent inability to query metrics still fails
with useful diagnostics.

Also check whether the test should use the existing get_session_metrics()
helper consistently rather than making a separate direct request.

Suggested validation

  1. Build Debug on Azure Linux 4, matching the failing workflow.
  2. Run tls_stress_test repeatedly through the repository tests.sh wrapper.
  3. Confirm the connection-cap assertions still detect real cap regressions.
  4. Confirm transient reset/503 behavior no longer fails the test if it is
    considered valid.
  5. Run the complete AL4 bucket C job.

Acceptance criteria

  • The intended behavior of existing sessions at the soft/hard cap is explicit
    in the test.
  • tls_stress_test is stable under repeated AL4 runs.
  • Persistent metrics-query failure remains actionable and fails the test.
  • Session active/peak/soft-cap/hard-cap assertions are preserved.
Ngôn ngữ chính
C++
Star
876
Fork
264
Merge trung bình
1 ngày 8 giờ
Pull request đã merge (30 ngày)
138

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

Mở trong Codespaces

Khởi chạy dev container của dự án ngay trên trình duyệt, bằng tài khoản GitHub của bạn.

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 microsoft/CCF

Tất cả issue của microsoft/CCF

Issue tương tự

Thêm issue về C++

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.