zqr10159 commented on PR #4294: URL: https://github.com/apache/hertzbeat/pull/4294#issuecomment-5137729891
Standalone verification is now complete on head `183cf6bb21`. - The PR targets `master` and is exactly one commit above it; the former #4263 dependency is included in that standalone commit. - `required` is the shipped Java and YAML default. `optional` remains available only as an explicit mixed-version rollout override, with removal after convergence documented in the PR. - GitHub checks are green: Backend Build, Backend Maven E2E, Backend Image E2E, DOC CI, License Checker, and labeler. For transparency, the first Maven E2E attempt stopped after 2m42s in the unrelated `CommonHttpClientVirtualThreadTest.dispatchConnectionPoolCleanupClosesExpiredAndIdleConnections` timing test, where a cleanup call was observed twice. This PR does not modify that client or test, and the complete class passed locally. GitHub attempt 2 passed the full Maven E2E job without any code change in 12m45s: https://github.com/apache/hertzbeat/actions/runs/30592359568 The dependency, CI coverage, and secure steady-state default blockers from the review are therefore resolved. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
