XComp commented on PR #24132: URL: https://github.com/apache/flink/pull/24132#issuecomment-1916845790
Ok, looks like the timeout was caused by some changes to the MockServer (which was moved into the `kubernetes-client` repository). The mocked GET request contains some parameters. The order of the parameters matters when mocking the requests (...which is annyoying. I didn't find a way to make this order-agnostic). Anyway, updating the GET path makes the test succeed again. For 1.19, I'm gonna go ahead and upgrade `kubernetes-client` to v6.9.2 to include all `LeaderElector`-related bug fixes. I don't see a reason to upgrade to v6.10.0 ([release notes](https://github.com/fabric8io/kubernetes-client/releases/tag/v6.10.0)). There are no compelling changes and I rather keep the version bump small: We would have to apply more code changes on our side (due to https://github.com/fabric8io/kubernetes-client/commit/fd50fa96#diff-375ebadb4285b8214fe6209c8e1758b3cd21f17f9637fb1206173026c0c033d3R65 ). Any objections from your side, @gyfora ? -- 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]
