You can not select more than 25 topics Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.

Terry Howe a7f84439aa test(kube): fix flaky WaitForDelete test by avoiding informer sync race ... The previous fix (increasing timeout / reducing deletion delay) did not work because the flakiness is not a timing problem at all. Root cause: fluxcd/cli-utils HasSynced() returns true after the initial list item is *popped* from DeltaFIFO, which is before AddFunc delivers the ResourceUpdateEvent to the collector. This creates a race where the SyncEvent can arrive at the statusObserver *before* the pod's Current status is recorded. When that happens: - statusObserver sees pod as Unknown - Unknown is skipped for WaitForDelete (by design, to handle resources that were already deleted before watching started) - AggregateStatus([], NotFoundStatus) == NotFoundStatus → cancel() - The watch context is cancelled before DeleteFunc can fire - Final check: pod still Current → error The test intent is to verify that waitForDeleteCtx (not the cancelled generalCtx) is selected. A non-existent resource satisfies this: - With waitForDeleteCtx=Background(): informer syncs with empty list → Unknown → cancel → success ✓ - With generalCtx (cancelled, wrong): context immediately done → ctx.Err() appended → error returned ✓ Remove the goroutine-based deletion and the pod creation to eliminate the race while preserving the context-selection assertion. Signed-off-by: Terry Howe <terrylhowe@gmail.com>		6 days ago
..
fake	chore(pkg): enable perfsprint linter	2 months ago
client.go	pkg/kube: remove legacy import comments	1 month ago
client_test.go	chore(pkg): enable perfsprint linter	2 months ago
converter.go	pkg/kube: remove legacy import comments	1 month ago
factory.go	pkg/kube: remove legacy import comments	1 month ago
interface.go	pkg/kube: introduce support for custom kstatus readers	3 months ago
options.go	feat(kstatus): fine-grained context options for waiting	3 months ago
ready.go	pkg/kube: remove legacy import comments	1 month ago
ready_test.go	fix(kube): remove legacy import comments from test files	1 month ago
resource.go	pkg/kube: remove legacy import comments	1 month ago
resource_policy.go	pkg/kube: remove legacy import comments	1 month ago
resource_test.go	fix(kube): remove legacy import comments from test files	1 month ago
result.go	ref(*): Refactors kube client to be a bit more friendly	7 years ago
roundtripper.go	fixing error handling from a previous PR	1 year ago
roundtripper_test.go	test(pkg/kube/roundtripper): Add unit tests for roundtripper.go	8 months ago
statuswait.go	pkg/kube: remove legacy import comments	1 month ago
statuswait_test.go	test(kube): fix flaky WaitForDelete test by avoiding informer sync race	6 days ago
wait.go	Merge pull request #31873 from mmorel-35/perfsprint-pkg-5-3294250	1 month ago
wait_test.go	chore: fix unnecessary-format issues from revive	3 weeks ago