youngkermit8-coder opened a new pull request, #1569:
URL: https://github.com/apache/rocketmq-dashboard/pull/1569

   ## What changed
   
   - derive `healthyClusters` from the collected `ClusterStatus.healthy` 
overviews instead of copying `totalClusters`;
   - assert the healthy summary count for both available and unavailable 
broker-runtime-metrics paths.
   
   ## Why
   
   When runtime stats retrieval failed, the per-cluster overview was correctly 
marked `warning`, but the summary still reported every discovered cluster as 
healthy. This made the dashboard health count contradict the cluster cards and 
overstate availability.
   
   ## Validation
   
   Validated this commit on top of the backend compilation fix in #1502 with 
JDK 21:
   
   ```text
   mvn -B -ntp -Dtest=RocketMQDashboardProviderTest clean test
   Tests run: 8, Failures: 0, Errors: 0, Skipped: 0
   Checkstyle: 0 violations
   BUILD SUCCESS
   
   mvn -B -ntp -DskipTests package
   Checkstyle: 0 violations
   BUILD SUCCESS
   ```
   
   Closes #1568.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to