Aias00 commented on PR #6361: URL: https://github.com/apache/shenyu/pull/6361#issuecomment-4645493200
Second websocket follow-up after run 27115745087 / job 80022215094: - The stale `/new/feature/get/mapping/path` issue is confirmed fixed, but probing `http://localhost:31188/actuator/health` still failed with repeated 404s. - The diagnostic logs show the ingress controller registered the backend as a websocket upstream: `protocol: ws://`, `upstreamUrl: 10.244.0.6:8001`. - That means the websocket preflight should not use an HTTP actuator request through the websocket backend path. The real test path is `ws://localhost:30095/myWs`, and the stable preflight signal is that the ingress controller itself is up. New commit `edb7ace53`: - changes websocket preflight to `http://localhost:30095/actuator/health` - removes the now-unused kind hostPort mapping for `31188` - adds a `startupProbe` to the websocket example so liveness does not restart the app while it is still starting Verification before push: - `git diff --check` - Ruby YAML parse for websocket Kubernetes manifest and kind config - kind hostPort mapping consistency check for all ingress service lists - `./mvnw -f shenyu-integrated-test/pom.xml -pl shenyu-integrated-test-k8s-ingress-websocket -am -DskipTests -Pit -Dmaven.javadoc.skip=true test-compile` -- 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]
