unbridled-41 commented on PR #3145:
URL:
https://github.com/apache/rocketmq-dashboard/pull/3145#issuecomment-5543774707
## Verification evidence (supplementary)
**Why CI is red on this PR:** the repository's CI workflow has failed at
startup (`startup_failure`, zero jobs executed) for every branch and event
since 2026-08-28 — including upstream's own pushes to `rocketmq-studio` and all
contributor PRs (dozens of consecutive runs). GitHub's run annotation reads
"This run likely failed because of a workflow file issue"; the last `ci.yml`
change was PR #2621 (2026-08-27). This branch does not modify `ci.yml` (`git
diff origin/rocketmq-studio..HEAD -- .github/workflows/ci.yml` → empty), so the
failures are a pre-existing repo-wide condition unrelated to this change.
CI-equivalent verification was therefore run locally:
**Regression test (red → green)**
- Red, before the fix: `mvn test -Dtest=NativeAlertProcessorTest` → `Tests
run: 18, Failures: 1`.
`resolvesMissingMetricEvenWhenAnotherMetricSharesTheSameLabelsTest` failed with
Mockito reporting `Wanted but not invoked:
alertStateRepository.save(AlertStateKey[ruleId=1, fingerprint=1519e4f4…],
<Capturing argument: AlertRuleState>)`; the only recorded interaction was
`findActive(…)` at
`NativeAlertProcessor.reconcileMissingActiveStates(NativeAlertProcessor.java:117)`
— i.e. the stale FIRING state was never resolved because the same-label
`consumer.lag.total` sample kept the fingerprint present.
- Green, after the fix: same command → `Tests run: 18, Failures: 0`.
**Module tests**
- `mvn test -Dtest='NativeAlert*Test'` → 40/40 pass
(NativeAlertProcessorTest 18, NativeAlertRulePolicyTest 12,
NativeAlertRuleScopeMatcherTest 3, NativeAlertEvaluationTransactionTest 1,
NativeAlertRuleTestServiceTest 4, NativeAlertMetricCatalogServiceTest 2).
- `mvn test '-Dtest=org.apache.rocketmq.studio.ops.alert.*Test'` → all
alert-package suites pass (AlertServiceTest 73, AlertStateMachineTest 7,
ClusterAlertRuleControllerTest 6, AlertRuleEvaluatorTest 5,
repositories/DTO/transfer suites included).
**Full suite (CI backend-test equivalent)**
- `mvn -B -ntp test` on this branch → `Tests run: 2036, Failures: 3`. The 3
failures are identical to the pristine `rocketmq-studio` baseline measured
before this change:
-
`AuthCorsIntegrationTest.shouldRejectNonAdminMutationBeforeControllerExecution:122`
(expected 403, got 401)
- `AuthCorsIntegrationTest.shouldStillRejectAnonymousProtectedRequests:110`
- `AliyunInstanceProviderTest.getGroupProgressShouldMapLagRowsTest:316`
(expected size 2, was 1)
2036 = 2035 baseline tests + the 1 new regression test. **Zero new
failures; the full-suite compile also covers the CI backend-build job.**
--
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]
