KAFKA-18628: Add node.id to the metrics reporter config override

[mingyen066]

DynamicMetricReporterState overrides only broker.id when configuring
metrics reporters. Since broker.id will no longer be passed to plugins
from 5.0 (KIP-1232), pass node.id alongside it.

This also fixes an inconsistency: reconfigurables are handed
KafkaConfig#valuesFromThisConfig, so a reporter recreated by a dynamic
update saw node.id as an Integer while broker.id was a String.
Both are now passed as strings, matching what reporters see at startup.

Raised in
#22977 (comment).

Reviewers: Chia-Ping Tsai chia7712@gmail.com

[chia7712]

LGTM

[chia7712]

@mingyen066 would you mind adding a test to ensure the props passed to the reporter include both id configs?

[mingyen066]

@chia7712 Added the test — it caught a real issue: without the fix, a dynamic update passes node.id to reporters as an Integer instead of a String. Title and description updated.