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

ut: ovn sb suite #4549

Merged
merged 10 commits into from
Sep 27, 2024
Merged

ut: ovn sb suite #4549

merged 10 commits into from
Sep 27, 2024

Conversation

zcq98
Copy link
Member

@zcq98 zcq98 commented Sep 23, 2024

Pull Request

What type of this PR

Examples of user facing changes:

  • Tests

Which issue(s) this PR fixes

Fixes #(issue-number)

@dosubot dosubot bot added size:XXL This PR changes 1000+ lines, ignoring generated files. test automation tests labels Sep 23, 2024
@zcq98
Copy link
Member Author

zcq98 commented Sep 24, 2024

@bobz965 Could you help review this?

pkg/ovs/ovn-nb-acl_test.go Outdated Show resolved Hide resolved
pkg/ovs/ovn-nb-suite_test.go Show resolved Hide resolved
pkg/ovs/ovn-nb-suite_test.go Show resolved Hide resolved
pkg/ovs/ovn-sb-chassis_test.go Outdated Show resolved Hide resolved
pkg/ovs/ovn-sb-chassis_test.go Outdated Show resolved Hide resolved
pkg/ovs/ovn-sb-chassis_test.go Outdated Show resolved Hide resolved
@zcq98 zcq98 requested a review from bobz965 September 24, 2024 07:31
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Sep 25, 2024
@bobz965
Copy link
Collaborator

bobz965 commented Sep 25, 2024

@oilbeater 大佬,这个帮忙review下,大部分都是命名替换,新增了一个 sb 的测试

@oilbeater oilbeater merged commit 49cba10 into kubeovn:master Sep 27, 2024
62 checks passed
@zcq98 zcq98 deleted the ut-ovn-sb branch October 9, 2024 00:50
bobz965 pushed a commit that referenced this pull request Oct 25, 2024
Signed-off-by: zcq98 <[email protected]>

---------

Signed-off-by: zcq98 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm This PR has been approved by a maintainer size:XXL This PR changes 1000+ lines, ignoring generated files. test automation tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants