From 90922f5c946b13c852af9b5166b5898e32a3f35c Mon Sep 17 00:00:00 2001 From: Adrian Moisey Date: Tue, 19 Nov 2024 12:20:02 +0200 Subject: [PATCH] Fix linting issue in test Lint error was introduced in https://github.com/kubernetes/kubernetes/pull/128305 and backported in https://github.com/kubernetes/kubernetes/pull/128806 It was merged before I could fix it, so I thought I'd make a follow up PR to fix it. I won't be backporting this lint as it's only in tests (unless it makes sense to backport?) --- pkg/controller/nodeipam/ipam/range_allocator_test.go | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/pkg/controller/nodeipam/ipam/range_allocator_test.go b/pkg/controller/nodeipam/ipam/range_allocator_test.go index bc8ebcdaf2a..c349c2e9744 100644 --- a/pkg/controller/nodeipam/ipam/range_allocator_test.go +++ b/pkg/controller/nodeipam/ipam/range_allocator_test.go @@ -926,7 +926,9 @@ func TestNodeDeletionReleaseCIDR(t *testing.T) { rangeAllocator.nodesSynced = test.AlwaysReady rangeAllocator.recorder = testutil.NewFakeRecorder() - rangeAllocator.syncNode(tCtx, tc.nodeKey) + if err := rangeAllocator.syncNode(tCtx, tc.nodeKey); err != nil { + t.Fatalf("failed to run rangeAllocator.syncNode") + } if len(rangeAllocator.cidrSets) != 1 { t.Fatalf("Expected a single cidrSet, but got %d", len(rangeAllocator.cidrSets))