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

[Flaky test] TestReconcile #5796

Closed
antoninbas opened this issue Dec 13, 2023 · 1 comment · Fixed by #5798
Closed

[Flaky test] TestReconcile #5796

antoninbas opened this issue Dec 13, 2023 · 1 comment · Fixed by #5798
Labels
kind/bug Categorizes issue or PR as related to a bug. kind/failing-test Categorizes issue or PR as related to a consistently or frequently failing test.

Comments

@antoninbas
Copy link
Contributor

I have seen this test fail for an unrelated PR (dependency upgrade):

2023-12-13T18:38:12.1706687Z I1213 18:38:11.910789   25042 server.go:733] "Reconciliation for CNI server"
2023-12-13T18:38:12.1707824Z I1213 18:38:11.911057   25042 pod_configuration.go:566] Removed interfaces for container c4d85a6e-99e6-11ee-a3e4-6045bdc75c39
2023-12-13T18:38:12.1708953Z W1213 18:38:11.911190   25042 pod_configuration_linux.go:57] Interface for Pod test/p4 not found in the interface store
2023-12-13T18:38:12.1709778Z --- FAIL: TestReconcile (0.00s)
2023-12-13T18:38:12.1711494Z     controller.go:98: missing call(s) to *testing.MockClient.InstallPodFlows(is equal to iface1 (string), is anything, is anything, is anything, is anything, is anything) /home/runner/work/antrea/antrea/pkg/agent/cniserver/server_linux_test.go:701
2023-12-13T18:38:12.1713170Z     controller.go:98: aborting test due to missing call(s)

@tnqn could this be related to recent change which affected the reconciliation logic?

This is the link to the failed workflow: https://github.com/antrea-io/antrea/actions/runs/7199573923/job/19611559921?pr=5790

@antoninbas antoninbas added kind/bug Categorizes issue or PR as related to a bug. kind/failing-test Categorizes issue or PR as related to a consistently or frequently failing test. labels Dec 13, 2023
@tnqn
Copy link
Member

tnqn commented Dec 14, 2023

@antoninbas thanks for reporting it. Yes, it's related to the recent change. #5798 should fix it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Categorizes issue or PR as related to a bug. kind/failing-test Categorizes issue or PR as related to a consistently or frequently failing test.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants