The GitHub Actions job "CI" on rocketmq-dashboard.git/rocketmq-studio has 
failed.
Run started by GitHub user lizhimins (triggered by lizhimins).

Head commit for run:
296550b176bb3f70d936f37e25146fa56feb9d25 / Yu Xinqiang <[email protected]>
fix: unify system topic/group filtering across providers (#1402, #1408)

* [ISSUE #1401] Unify system consumer group filtering across providers

Three separate isSystemGroup/isSystemConsumerGroup implementations had
inconsistent filtering rules, causing system groups like CID_ONSAPI_*
and CID_HOUSEKEEPING to leak into topic consumer lists and inflate
dashboard group counts.

This fix:
- Extracts a shared SystemGroupFilter utility class consolidating all
  system group patterns
- Replaces all three implementations with calls to the shared method
- Fixes the CID_ONS_ vs CID_ONSAPI_ prefix bug in MetadataProvider
- Adds missing %RETRY%, %DLQ%, CID_SYS_, CID_HOUSEKEEPING filters to
  DashboardProvider

Fixes #1401

* [ISSUE #1407] Unify system topic filtering across providers

Two separate isSystemTopic implementations had inconsistent filtering
rules, causing system topics to leak into topic lists and inflate
dashboard topic counts. Key discrepancies:

- CID_ prefix was in MetadataProvider but not DashboardProvider
- broker_ prefix was in DashboardProvider but not MetadataProvider
- DashboardProvider had hardcoded broker-a/broker-b names
- DashboardProvider did not check live broker names as system topics

This fix:
- Extracts a shared SystemTopicFilter utility class
- Replaces both implementations with calls to the shared method
- Removes hardcoded broker-a/broker-b from DashboardProvider
- Adds CID_, broker_, BenchmarkTest to the unified filter

Fixes #1407

Report URL: 
https://github.com/apache/rocketmq-dashboard/actions/runs/31408946608

With regards,
GitHub Actions via GitBox

Reply via email to