antrea
antrea copied to clipboard
Resolve reconnect issue between Agent and OVS
- Bump up ofnet version
- Delete existing OpenFlow groups before groups are replayed in reconnections.
Fixes #4092
Signed-off-by: wenyingd [email protected]
Codecov Report
Merging #4091 (93774c0) into main (d1c6a43) will increase coverage by
0.04%
. The diff coverage isn/a
.
@@ Coverage Diff @@
## main #4091 +/- ##
==========================================
+ Coverage 65.70% 65.74% +0.04%
==========================================
Files 304 304
Lines 46604 46779 +175
==========================================
+ Hits 30621 30757 +136
- Misses 13557 13622 +65
+ Partials 2426 2400 -26
Flag | Coverage Δ | |
---|---|---|
e2e-tests | 39.39% <ø> (?) |
|
integration-tests | 34.95% <ø> (+<0.01%) |
:arrow_up: |
kind-e2e-tests | 48.35% <ø> (-0.99%) |
:arrow_down: |
unit-tests | 44.41% <ø> (+0.03%) |
:arrow_up: |
Impacted Files | Coverage Δ | |
---|---|---|
pkg/agent/cniserver/ipam/antrea_ipam.go | 51.08% <0.00%> (-22.95%) |
:arrow_down: |
pkg/agent/flowexporter/exporter/certificate.go | 27.77% <0.00%> (-22.23%) |
:arrow_down: |
pkg/controller/networkpolicy/crd_utils.go | 74.88% <0.00%> (-14.84%) |
:arrow_down: |
...agent/flowexporter/connections/deny_connections.go | 67.74% <0.00%> (-13.98%) |
:arrow_down: |
pkg/agent/cniserver/ipam/ipam_service.go | 51.68% <0.00%> (-12.36%) |
:arrow_down: |
pkg/agent/flowexporter/exporter/exporter.go | 67.82% <0.00%> (-10.15%) |
:arrow_down: |
pkg/agent/flowexporter/utils.go | 72.34% <0.00%> (-8.52%) |
:arrow_down: |
pkg/ipam/poolallocator/allocator.go | 49.76% <0.00%> (-5.96%) |
:arrow_down: |
.../flowexporter/connections/conntrack_connections.go | 76.66% <0.00%> (-4.77%) |
:arrow_down: |
pkg/agent/cniserver/server.go | 73.94% <0.00%> (-3.41%) |
:arrow_down: |
... and 37 more |
/test-all /test-windows-all
/test-all /test-windows-all /test-ipv6-all /test-ipv6-only-all
/test-flexible-ipam-e2e /test-multicluster-e2e
Update go.mod/go.sum to refer to antrea.io/ofnet since the dependent patch is merged. All the tests are passed before this update.
/test-all