yyqdbngt opened a new pull request, #1961: URL: https://github.com/apache/rocketmq-dashboard/pull/1961
## What is the purpose of the change `RocketMQClusterProvider` (the `@Primary` cluster provider used by `ClusterService.listClusters()`) never set a `ClusterVO.type`, leaving it `null` for every cluster discovered from a live NameServer. The AI tool catalog's `rmq.cluster.list` handler projects `type` with `requiredEnumName(...)`, which throws `IllegalStateException` on a null value — so the most basic cluster tool failed with a 500 on real clusters. ## Brief changelog - Set `ClusterType.V4_DIRECT` in `buildClusterVO`, matching the sibling `RealClusterProvider` default. This provider only discovers broker topology (proxies are always empty), so a V4 default is the accurate classification. - Assert a non-null `type` in the safe-defaults discovery test. ## Verifying this change - `mvn -q -Dtest=RocketMQClusterProviderTest test` - `mvn -q test` (1056/1056) - `git diff --check` -- 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]
