Skip to content

test: [3.0] backport add function field search readiness fixes - #55224

Open
zhuwenxing wants to merge 2 commits into
milvus-io:3.0from
zhuwenxing:backport-add-function-search-readiness-3.0
Open

zhuwenxing wants to merge 2 commits into
milvus-io:3.0from
zhuwenxing:backport-add-function-search-readiness-3.0

Conversation

@zhuwenxing

Copy link
Copy Markdown
Contributor

Backport #52317 and #52131 to retry search readiness errors after adding function fields, preserve exact ordered topK assertions, and verify five successful rounds after the first hit.

Includes the upstream 512-row BM25 regression case and five L1-to-L2 marker changes.

Validation: Ruff 0.15.11 lint/format checks, Python compilation, diff checks, and six isolated helper checks passed. Collection selected 24 L0/L1/L2 cases and no L3 cases using PyMilvus 3.1.0rc39. Live E2E with the pinned 3.0 SDK was not run. The license workflow’s Go-only scope does not cover this Python-only change.

pr: #52317
pr: #52131

xiaofan-luan and others added 2 commits November 6, 2026 14:43
…ng loops

Backport milvus-io#52317 to tolerate the asynchronous query-node loading window while preserving ordered result assertions.

(cherry picked from commit 025d2e3)

Signed-off-by: xiaofanluan <xf@hjjaq.com>
Signed-off-by: zhuwenxing <wenxing.zhu@zilliz.com>
Backport milvus-io#52131 to record readiness errors, preserve exact ordered result checks, and verify five successful rounds after the initial hit.

(cherry picked from commit abe8696)

Signed-off-by: lyyyuna <yiyang.li@zilliz.com>
Signed-off-by: zhuwenxing <wenxing.zhu@zilliz.com>
@sre-ci-robot sre-ci-robot added the do-not-merge/work-in-progress Don't merge even CI passed. label Nov 6, 2026
@sre-ci-robot sre-ci-robot added the size/XL Denotes a PR that changes 500-999 lines. label Nov 6, 2026
@sre-ci-robot

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
To complete the pull request process, please ask for approval from zhuwenxing after the PR has been reviewed.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@mergify mergify Bot added dco-passed DCO check passed. kind/test labels Nov 6, 2026
@zhuwenxing
zhuwenxing marked this pull request as ready for review November 6, 2026 06:48
@sre-ci-robot sre-ci-robot removed the do-not-merge/work-in-progress Don't merge even CI passed. label Nov 6, 2026
@sre-ci-robot

Copy link
Copy Markdown
Contributor

[ci-v2-notice]
Notice: ci-v2 system is enabled for this PR (3.0 branch).

To rerun ci-v2 checks, comment with:

  • /ci-rerun-code-check-macos // for Code Checker MacOS (GitHub Actions)
  • /ci-rerun-build-ut-cov // for ci-v2/build-ut-cov (build + unit tests)
  • /ci-rerun-e2e-amd // for ci-v2/e2e-amd (e2e tests)
  • /ci-rerun-e2e-dist-wp // for ci-v2/e2e-dist-wp (Tencent distributed woodpecker-service boundary)
  • /ci-rerun-gosdk // for ci-v2/go-sdk (Go SDK E2E tests)
  • /ci-rerun-gosdk-std // for ci-v2/go-sdk-std (Go SDK E2E, standalone)
  • /ci-rerun-gosdk-dist-wp // for ci-v2/go-sdk-dist-wp (distributed + Woodpecker service)

If you have any questions or requests, please contact @zhikunyao.

@sre-ci-robot sre-ci-robot added the do-not-merge/need-milestone generate by v2-label-manager label Nov 6, 2026
@sre-ci-robot

Copy link
Copy Markdown
Contributor

[INFO] PR Label Summary by Default
[SUCCESS] PR #52317 merged to master

[WARNING] Milestone not set

You can set milestone by commenting:
/set-milestone
Example:
/set-milestone 2.5.0

Use /refresh-label to update related check and label manually

@mergify mergify Bot added the ci-passed label Nov 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/test ci-passed dco-passed DCO check passed. do-not-merge/need-milestone generate by v2-label-manager kind/test sig/testing size/XL Denotes a PR that changes 500-999 lines.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants