Merge pull request kubernetes-client#2174 from yue9944882/flake/leader-election-test

k8s-ci-robot · web-flow · commit 493857293c13 · 2022-03-18T13:13:55.000-07:00
Fixes flakiness in leader-election tests
diff --git a/extended/src/main/java/io/kubernetes/client/extended/leaderelection/LeaderElector.java b/extended/src/main/java/io/kubernetes/client/extended/leaderelection/LeaderElector.java
@@ -223,6 +223,9 @@ private void renewLoop() {
             log.debug("failed to tryAcquireOrRenew", t);
           }
           renewResult = false;
+        } catch (Throwable t) {
+          log.error("Unexpected exception when renewing lease in the background", t);
+          renewResult = false;
         } finally {
           future.cancel(true); // make the lease worker doesn't overlap
         }

Original file line number	Diff line number	Diff line change
`@@ -223,6 +223,9 @@ private void renewLoop() {`
`223`	`223`	`log.debug("failed to tryAcquireOrRenew", t);`
`224`	`224`	`}`
`225`	`225`	`renewResult = false;`
	`226`	`+ } catch (Throwable t) {`
	`227`	`+ log.error("Unexpected exception when renewing lease in the background", t);`
	`228`	`+ renewResult = false;`
`226`	`229`	`} finally {`
`227`	`230`	`future.cancel(true); // make the lease worker doesn't overlap`
`228`	`231`	`}`