Aias00 commented on PR #6361: URL: https://github.com/apache/shenyu/pull/6361#issuecomment-4645710890
Follow-up for https://github.com/apache/shenyu/actions/runs/27117367880: The websocket backend is now healthy: the healthcheck waited for the default deployment, it became available, and `localhost:30095/actuator/health` returned 200. The remaining failure moved to `Run test`: `WebsocketPluginTest` got `WebsocketNotConnectedException`. The ingress controller log showed the cause: `EndpointsReconciler` repeatedly threw `NullPointerException` at `needUpdateSelectorId.addAll(selectorIdList)` because it always looked up `divide` selectors for endpoint updates. For websocket ingress there is no divide selector, so the websocket selector kept its empty upstream handle and the plugin logged `websocket upstream configuration error`. I pushed `e2bfcb988` so endpoint reconciliation skips missing selector caches and refreshes both `divide` and `websocket` selector handles with plugin-appropriate upstream JSON. Added `EndpointsReconcilerTest` to cover the websocket endpoint update path. -- 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]
