fix: pass response instead of status to CheckRPCCall to prevent nil deref

[sunby]

Pass whole response to merr.CheckRPCCall instead of resp.Status to prevent nil pointer dereference when broker returns (nil, err).

issue: #48538

[sre-ci-robot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: sunby
To complete the pull request process, please assign wxyucs after the PR has been reviewed.
You can assign the PR to them by writing /assign @wxyucs in a comment when ready.

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

Details

Needs approval from an approver in each of these files:

• internal/datacoord/OWNERS

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]

[ci-v2-notice]
Notice: New ci-v2 system is enabled for this PR.

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-build-all // for ci-v2/build-all (multi-arch builds)
• /ci-rerun-buildenv // for ci-v2/build-env (build milvus-env builder images)
• /ci-rerun-ut-integration // for ci-v2/ut-integration, will rerun ci-v2/build
• /ci-rerun-ut-go // for ci-v2/ut-go, will rerun ci-v2/build
• /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, will rerun ci-v2/build
• /ci-rerun-e2e-arm // for ci-v2/e2e-arm
• /ci-rerun-e2e-default // for ci-v2/e2e-default
• /ci-rerun-ciloop // for ci-v2/ciloop (build + unit tests in one pipeline)
• /ci-rerun-gosdk // for ci-v2/go-sdk (Go SDK E2E tests, AMD)
• /ci-rerun-gosdk-arm // for ci-v2/go-sdk-arm (Go SDK E2E tests, ARM)

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

[codecov]

Codecov Report

❌ Patch coverage is 33.33333% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 77.62%. Comparing base (276db8f) to head (e09f813).
⚠️ Report is 7 commits behind head on master.

Files with missing lines	Patch %	Lines
internal/datacoord/index_service.go	0.00%	1 Missing and 1 partial ⚠️

❌ Your patch check has failed because the patch coverage (33.33%) is below the target coverage (80.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files

@@            Coverage Diff             @@
##           master   #48539      +/-   ##
==========================================
- Coverage   77.64%   77.62%   -0.02%     
==========================================
  Files        2112     2112              
  Lines      351505   351494      -11     
==========================================
- Hits       272924   272850      -74     
- Misses      70250    70307      +57     
- Partials     8331     8337       +6     

Components	Coverage Δ
Client	79.25% <ø> (ø)
Core	84.01% <ø> (ø)
Go	75.79% <58.59%> (-0.03%)	⬇️

Files with missing lines	Coverage Δ
internal/datacoord/ddl_callbacks_import.go	78.84% <100.00%> (ø)
internal/datacoord/index_service.go	87.77% <0.00%> (ø)

... and 29 files with indirect coverage changes

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[sunby]

/ci-rerun-e2e-arm

[sunby]

/ci-rerun-e2e-arm
/ci-rerun-e2e-default

[sunby]

/ci-rerun-e2e-default

[sunby]

/ci-rerun-e2e-default
/ci-rerun-go-sdk

[sunby]

/ci-rerun-go-sdk

[sunby]

/ci-rerun-go-sdk

[sunby]

/ci-rerun-go-sdk

[sunby]

/ci-rerun-go-sdk

[sunby]

/ci-rerun-go-sdk

[sunby]

/ci-rerun-go-sdk

[sunby]

/ci-rerun-go-sdk

[sunby]

/ci-rerun-go-sdk

[sunby]

/ci-rerun-go-sdk

[sunby]

/ci-rerun-go-sdk

[sunby]

/ci-rerun-go-sdk

[sunby]

/ci-rerun-go-sdk

[sunby]

/ci-rerun-go-sdk