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 #2

Open
wants to merge 8 commits into
base: main
Choose a base branch
from
Open

Ut #2

wants to merge 8 commits into from

Conversation

wenqiq
Copy link
Owner

@wenqiq wenqiq commented Aug 26, 2022

Add example

Signed-off-by: Wenqi Qiu [email protected]

@wenqiq wenqiq force-pushed the ut- branch 5 times, most recently from 0213562 to 1ffc8a9 Compare August 26, 2022 13:36
Add example

debug codecov

Signed-off-by: Wenqi Qiu <[email protected]>
Signed-off-by: Wenqi Qiu <[email protected]>
@codecov-commenter
Copy link

Codecov Report

Merging #2 (1841abb) into main (02150d8) will decrease coverage by 0.10%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main       #2      +/-   ##
==========================================
- Coverage   15.94%   15.84%   -0.11%     
==========================================
  Files          40       43       +3     
  Lines       18324    18453     +129     
==========================================
+ Hits         2922     2924       +2     
- Misses      15181    15308     +127     
  Partials      221      221              
Flag Coverage Δ
unit-tests 15.84% <100.00%> (-0.11%) ⬇️
Impacted Files Coverage Δ
util/add.go 100.00% <100.00%> (ø)
util/stream.go 0.00% <0.00%> (ø)
util/util.go 0.00% <0.00%> (ø)

wenqiq and others added 6 commits August 30, 2022 01:40
Signed-off-by: Wenqi Qiu <[email protected]>
…ands (antrea-io#27)

To support OFPGC_INSERT_BUCKET and OFPGC_REMOVE_BUCKET commands,
CommandBucketId in GroupMod should be either the bucket ID of one
of the buckets already existing in the group, or one of the special
bucket identifiers, which are added in this PR.

Signed-off-by: Hongliang Liu <[email protected]>
To follow antrea's convention, klog is being used
and logrus is abandoned.

Signed-off-by: Yifeng Sun <[email protected]>
Signed-off-by: Wenqi Qiu <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants