Skip to content

Conversation

@aoiasd
Copy link
Contributor

@aoiasd aoiasd commented Nov 17, 2025

relate: #45345
pr: #45337

@sre-ci-robot sre-ci-robot added the size/M Denotes a PR that changes 30-99 lines. label Nov 17, 2025
@mergify mergify bot added dco-passed DCO check passed. kind/bug Issues or changes related a bug labels Nov 17, 2025
@sre-ci-robot
Copy link
Contributor

[ci-v2-notice]
Notice: We are gradually rolling out the new ci-v2 system.

  • Legacy CI jobs remain unaffected, you can just ignore ci-v2 if you don't want to run it.
  • Additional "ci-v2/*" checkers will run for this PR to ensure the new ci-v2 system is working as expected.
  • For tests that exist in both v1 and v2, passing in either system is considered PASS.

To rerun ci-v2 checks, comment with:

  • /ci-rerun-code-check // for ci-v2/code-check
  • /ci-rerun-build // for ci-v2/build
  • /ci-rerun-ut-integration // for ci-v2/ut-integration
  • /ci-rerun-ut-go // for ci-v2/ut-go
  • /ci-rerun-ut-cpp // for ci-v2/ut-cpp
  • /ci-rerun-ut // for all ci-v2/ut-integration, ci-v2/ut-go, ci-v2/ut-cpp
  • /ci-rerun-e2e-arm // for ci-v2/e2e-arm

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 17, 2025
@sre-ci-robot
Copy link
Contributor

[INFO] PR Label Summary by Default
[SUCCESS] PR #45337 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

@aoiasd
Copy link
Contributor Author

aoiasd commented Nov 17, 2025

/set-milestone 2.5.7

@sre-ci-robot
Copy link
Contributor

[ERROR] Failed to set milestone: 2.5.7

@mergify
Copy link
Contributor

mergify bot commented Nov 17, 2025

@aoiasd cpu-e2e job failed, comment /run-cpu-e2e can trigger the job again.

@codecov
Copy link

codecov bot commented Nov 17, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 76.60%. Comparing base (cb08db0) to head (dcd35ee).
⚠️ Report is 277 commits behind head on 2.6.

❌ Your project status has failed because the head coverage (76.60%) is below the target coverage (77.00%). You can increase the head coverage or adjust the target coverage.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##              2.6   #45618      +/-   ##
==========================================
- Coverage   76.99%   76.60%   -0.39%     
==========================================
  Files        1700     1856     +156     
  Lines      262533   287764   +25231     
==========================================
+ Hits       202142   220447   +18305     
- Misses      53550    59971    +6421     
- Partials     6841     7346     +505     
Components Coverage Δ
Client 78.03% <65.76%> (-0.11%) ⬇️
Core 83.65% <79.25%> (+1.44%) ⬆️
Go 74.62% <52.20%> (-0.77%) ⬇️
Files with missing lines Coverage Δ
internal/core/src/common/Schema.h 98.95% <ø> (ø)
internal/core/src/segcore/Collection.cpp 70.45% <100.00%> (-1.29%) ⬇️
internal/core/src/segcore/Collection.h 100.00% <100.00%> (ø)

... and 647 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@sre-ci-robot
Copy link
Contributor

[INFO] PR Label Summary by Default
[SUCCESS] PR #45337 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

@aoiasd
Copy link
Contributor Author

aoiasd commented Nov 19, 2025

/set-milestone 2.6.5

@sre-ci-robot
Copy link
Contributor

[ERROR] Failed to set milestone: 2.6.5

@aoiasd
Copy link
Contributor Author

aoiasd commented Nov 19, 2025

/set-milestone 2.6.6

@sre-ci-robot sre-ci-robot added this to the 2.6.6 milestone Nov 19, 2025
@sre-ci-robot sre-ci-robot removed the do-not-merge/need-milestone generate by v2-label-manager label Nov 19, 2025
@sre-ci-robot
Copy link
Contributor

[INFO] Set milestone to: 2.6.6

@sre-ci-robot
Copy link
Contributor

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

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

@mergify
Copy link
Contributor

mergify bot commented Nov 19, 2025

@aoiasd go-sdk check failed, comment rerun go-sdk can trigger the job again.

@mergify
Copy link
Contributor

mergify bot commented Nov 19, 2025

@aoiasd cpu-e2e job failed, comment /run-cpu-e2e can trigger the job again.

@aoiasd
Copy link
Contributor Author

aoiasd commented Nov 20, 2025

/run-cpu-e2e

@aoiasd
Copy link
Contributor Author

aoiasd commented Nov 20, 2025

rerun go-sdk

@aoiasd
Copy link
Contributor Author

aoiasd commented Nov 20, 2025

/ci-rerun-ut-integration

@mergify
Copy link
Contributor

mergify bot commented Nov 20, 2025

@aoiasd go-sdk check failed, comment rerun go-sdk can trigger the job again.

@sre-ci-robot
Copy link
Contributor

[ci-v2-notice]
Notice: We are gradually rolling out the new ci-v2 system.

  • Legacy CI jobs remain unaffected, you can just ignore ci-v2 if you don't want to run it.
  • Additional "ci-v2/*" checkers will run for this PR to ensure the new ci-v2 system is working as expected.
  • For tests that exist in both v1 and v2, passing in either system is considered PASS.

To rerun ci-v2 checks, comment with:

  • /ci-rerun-code-check // for ci-v2/code-check
  • /ci-rerun-build // for ci-v2/build
  • /ci-rerun-ut-integration // for ci-v2/ut-integration
  • /ci-rerun-ut-go // for ci-v2/ut-go
  • /ci-rerun-ut-cpp // for ci-v2/ut-cpp
  • /ci-rerun-ut // for all ci-v2/ut-integration, ci-v2/ut-go, ci-v2/ut-cpp
  • /ci-rerun-e2e-arm // for ci-v2/e2e-arm

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

@aoiasd
Copy link
Contributor Author

aoiasd commented Nov 20, 2025

rerun go-sdk

@aoiasd
Copy link
Contributor Author

aoiasd commented Nov 20, 2025

/run-cpu-e2e

@sre-ci-robot
Copy link
Contributor

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

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

@mergify mergify bot added the ci-passed label Nov 21, 2025
@yanliang567 yanliang567 modified the milestones: 2.6.6, 2.6.7 Nov 21, 2025
Copy link
Contributor

@congqixia congqixia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@sre-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: aoiasd, congqixia

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

The pull request process is described here

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

@sre-ci-robot sre-ci-robot merged commit b92f17b into milvus-io:2.6 Nov 25, 2025
14 of 19 checks passed
@sre-ci-robot
Copy link
Contributor

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

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved ci-passed dco-passed DCO check passed. kind/bug Issues or changes related a bug lgtm size/M Denotes a PR that changes 30-99 lines.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants