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

fix ovn nb load balancer health check unittest #3726

Closed
wants to merge 1 commit into from

Conversation

zhangzujian
Copy link
Member

@zhangzujian zhangzujian commented Feb 18, 2024

Pull Request

What type of this PR

  • Tests

Which issue(s) this PR fixes

panic: runtime error: index out of range [0] with length 0 [recovered]
        panic: runtime error: index out of range [0] with length 0

goroutine 821 [running]:
testing.tRunner.func1.2({0x1691840, 0xc00038f548})
        /home/linuxbrew/.linuxbrew/Cellar/go/1.22.0/libexec/src/testing/testing.go:1631 +0x24a
testing.tRunner.func1()
        /home/linuxbrew/.linuxbrew/Cellar/go/1.22.0/libexec/src/testing/testing.go:1634 +0x377
panic({0x1691840?, 0xc00038f548?})
        /home/linuxbrew/.linuxbrew/Cellar/go/1.22.0/libexec/src/runtime/panic.go:770 +0x132
github.com/kubeovn/kube-ovn/pkg/ovs.(*OvnClientTestSuite).testDeleteLoadBalancerHealthCheckOp.func1(0xc000938680)
        /home/zhang/kube-ovn/pkg/ovs/ovn-nb-load_balancer_health_check_test.go:324 +0x2da
testing.tRunner(0xc000938680, 0xc0003c6820)
        /home/linuxbrew/.linuxbrew/Cellar/go/1.22.0/libexec/src/testing/testing.go:1689 +0xfb
created by testing.(*T).Run in goroutine 90
        /home/linuxbrew/.linuxbrew/Cellar/go/1.22.0/libexec/src/testing/testing.go:1742 +0x390
FAIL    github.com/kubeovn/kube-ovn/pkg/ovs     0.364s
?       github.com/kubeovn/kube-ovn/pkg/webhook [no test files]
ok      github.com/kubeovn/kube-ovn/pkg/util    (cached)
FAIL
make: *** [Makefile:961: ut] Error 1

WHAT

copilot:summary

copilot:poem

HOW

copilot:walkthrough

@zhangzujian zhangzujian added the test automation tests label Feb 18, 2024
@zhangzujian zhangzujian changed the title fix unittest fix ovn nb load balancer health check unittest Feb 18, 2024
@zhangzujian zhangzujian marked this pull request as ready for review February 18, 2024 06:46
Signed-off-by: zhangzujian <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
test automation tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant