Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

vmtests: disable TestFastK8s #3356

Merged
merged 1 commit into from
Jan 30, 2025
Merged

vmtests: disable TestFastK8s #3356

merged 1 commit into from
Jan 30, 2025

Conversation

kkourt
Copy link
Contributor

@kkourt kkourt commented Jan 30, 2025

TestFastK8s was introduced in 7043d06.

TestFastK8s has been flaky (see issue 2972). Not sure why this is the case. Maybe some timing issue with respect to the k8s controller and fake client initialization.

This commit disables it for vmtests since it does not load any bpf programs and so it is not kernel dependent.

Fixes: #2972

TestFastK8s was introduced in 7043d06.

TestFastK8s has been flaky (see issue 2972). Not sure why this is the
case. Maybe some timing issue with respect to the k8s controller and
fake client initialization.

This commit disables it for vmtests since it does not load any bpf
programs and so it is not kernel dependent.

Signed-off-by: Kornilios Kourtis <kornilios@isovalent.com>
@kkourt kkourt requested a review from a team as a code owner January 30, 2025 08:55
@kkourt kkourt requested a review from jrfastab January 30, 2025 08:55
@kkourt kkourt added the release-note/misc This PR makes changes that have no direct user impact. label Jan 30, 2025
@mtardy mtardy merged commit 4f35118 into main Jan 30, 2025
39 of 40 checks passed
@mtardy mtardy deleted the pr/kkourt/testfastk8s-disable branch January 30, 2025 11:29
@kkourt kkourt mentioned this pull request Jan 30, 2025
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-note/misc This PR makes changes that have no direct user impact.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

pkg.watcher.TestFastK8s might be flaky
2 participants