fix: allow datanode's server id to be updated

[yiwangdr]

issue: #31516

background: the server id field in data node is redundant. session id already provides the source of truth.

[mergify]

@yiwangdr E2e jenkins job failed, comment /run-cpu-e2e can trigger the job again.

[codecov]

Codecov Report

Attention: Patch coverage is 83.33333% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 81.75%. Comparing base (c0555d4) to head (37a41cf).
Report is 60 commits behind head on master.

Additional details and impacted files

@@           Coverage Diff           @@
##           master   #31597   +/-   ##
=======================================
  Coverage   81.75%   81.75%           
=======================================
  Files         998      998           
  Lines      124172   124175    +3     
=======================================
+ Hits       101519   101525    +6     
+ Misses      18792    18782   -10     
- Partials     3861     3868    +7     

Files	Coverage Δ
internal/datanode/data_node.go	81.33% <100.00%> (-0.09%)	⬇️
internal/datanode/event_manager.go	82.64% <100.00%> (ø)
internal/distributed/datanode/service.go	82.92% <100.00%> (ø)
internal/datacoord/server.go	71.62% <75.00%> (+0.02%)	⬆️
internal/distributed/datacoord/service.go	88.83% <50.00%> (ø)

... and 29 files with indirect coverage changes

[XuanYang-cn]

from the log of the change, stil using zero in msgdispatcher

https://grafana-4am.zilliz.cc/explore?panes=%7B%22r9R%22:%7B%22datasource%22:%22vhI6Vw67k%22,%22queries%22:%5B%7B%22refId%22:%22A%22,%22expr%22:%22%7Bpod%3D%5C%22msop-31597-2-pr-milvus-standalone-85b7d85f78-4k9kl%5C%22%7D%20%7C%3D%20%5C%22datanode%5C%22%20%7C%3D%20%5C%22nodeID%5C%22%22,%22queryType%22:%22range%22,%22datasource%22:%7B%22type%22:%22loki%22,%22uid%22:%22vhI6Vw67k%22%7D,%22editorMode%22:%22code%22%7D%5D,%22range%22:%7B%22from%22:%22now-7d%22,%22to%22:%22now%22%7D%7D%7D&schemaVersion=1&orgId=1

[yiwangdr]

from the log of the change, stil using zero in msgdispatcher https://grafana-4am.zilliz.cc/explore?panes=%7B%22r9R%22:%7B%22datasource%22:%22vhI6Vw67k%22,%22queries%22:%5B%7B%22refId%22:%22A%22,%22expr%22:%22%7Bpod%3D%5C%22msop-31597-2-pr-milvus-standalone-85b7d85f78-4k9kl%5C%22%7D%20%7C%3D%20%5C%22datanode%5C%22%20%7C%3D%20%5C%22nodeID%5C%22%22,%22queryType%22:%22range%22,%22datasource%22:%7B%22type%22:%22loki%22,%22uid%22:%22vhI6Vw67k%22%7D,%22editorMode%22:%22code%22%7D%5D,%22range%22:%7B%22from%22:%22now-7d%22,%22to%22:%22now%22%7D%7D%7D&schemaVersion=1&orgId=1

Nice catch. Done

[yiwangdr]

rerun ut

[yiwangdr]

rerun ut

[mergify]

@yiwangdr E2e jenkins job failed, comment /run-cpu-e2e can trigger the job again.

[mergify]

@yiwangdr ut workflow job failed, comment rerun ut can trigger the job again.

[mergify]

@yiwangdr E2e jenkins job failed, comment /run-cpu-e2e can trigger the job again.

[yiwangdr]

/run-cpu-e2e

[mergify]

@yiwangdr E2e jenkins job failed, comment /run-cpu-e2e can trigger the job again.

[yiwangdr]

/run-cpu-e2e

[czs007]

/run-cpu-e2e

[mergify]

@yiwangdr E2e jenkins job failed, comment /run-cpu-e2e can trigger the job again.

[XuanYang-cn]

/lgtm
/approve

[XuanYang-cn]

/run-cpu-e2e

[mergify]

@yiwangdr E2e jenkins job failed, comment /run-cpu-e2e can trigger the job again.

[sre-ci-robot]

New changes are detected. LGTM label has been removed.

[sre-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

Approval requirements bypassed by manually added approval.

This pull-request has been approved by: XuanYang-cn, yiwangdr

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:

• internal/datacoord/OWNERS
• internal/datanode/OWNERS
• internal/distributed/OWNERS

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