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

Update egress agent test #646

Closed
wants to merge 1 commit into from
Closed

Update egress agent test #646

wants to merge 1 commit into from

Conversation

lou-lan
Copy link
Collaborator

@lou-lan lou-lan commented Jul 30, 2023

No description provided.

@lou-lan lou-lan added the release/none should not recoded in new release label Jul 30, 2023
@codecov
Copy link

codecov bot commented Jul 30, 2023

Codecov Report

Merging #646 (ddb2ab6) into main (75589ad) will increase coverage by 5.51%.
The diff coverage is 55.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #646      +/-   ##
==========================================
+ Coverage   10.45%   15.96%   +5.51%     
==========================================
  Files          19       16       -3     
  Lines        3864     3808      -56     
==========================================
+ Hits          404      608     +204     
+ Misses       3384     3095     -289     
- Partials       76      105      +29     
Flag Coverage Δ
unittests 15.96% <55.00%> (+5.51%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Changed Coverage Δ
pkg/agent/vxlan.go 36.55% <0.00%> (+36.55%) ⬆️
pkg/controller/egress_cluster_info.go 0.00% <0.00%> (ø)
pkg/utils/ip/ip.go 33.22% <78.57%> (ø)

... and 4 files with indirect coverage changes

@lou-lan lou-lan force-pushed the update/agent/test branch 13 times, most recently from 4f2c8c5 to 99c55d9 Compare July 30, 2023 15:32
@lou-lan lou-lan closed this Jul 31, 2023
@lou-lan lou-lan deleted the update/agent/test branch September 3, 2024 03:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release/none should not recoded in new release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant