antrea
antrea copied to clipboard
[ExternalNode] Implement SupportBundleCollection on Controller
- Add internal objects to sync the suport bundle collection request from Controller to Agent.
- Implement support bundle collection on Controlelr side.
Signed-off-by: wenyingd [email protected]
Codecov Report
Merging #4184 (e101c6c) into main (835e142) will decrease coverage by
2.51%
. The diff coverage is75.31%
.
:exclamation: Current head e101c6c differs from pull request most recent head 76b96dd. Consider uploading reports for the commit 76b96dd to get more accurate results
@@ Coverage Diff @@
## main #4184 +/- ##
==========================================
- Coverage 64.38% 61.87% -2.52%
==========================================
Files 393 389 -4
Lines 55538 55142 -396
==========================================
- Hits 35760 34117 -1643
- Misses 17188 18498 +1310
+ Partials 2590 2527 -63
Flag | Coverage Δ | |
---|---|---|
integration-tests | 34.89% <57.14%> (+0.28%) |
:arrow_up: |
kind-e2e-tests | 45.15% <10.95%> (-3.72%) |
:arrow_down: |
unit-tests | 44.21% <71.27%> (-3.94%) |
:arrow_down: |
Impacted Files | Coverage Δ | |
---|---|---|
cmd/antrea-controller/controller.go | 0.00% <0.00%> (ø) |
|
pkg/apis/controlplane/types.go | 100.00% <ø> (ø) |
|
pkg/apis/controlplane/v1beta2/helper.go | 100.00% <ø> (ø) |
|
pkg/features/antrea_features.go | 60.00% <ø> (ø) |
|
pkg/controller/supportbundlecollection/validate.go | 70.83% <70.83%> (ø) |
|
...g/controller/supportbundlecollection/controller.go | 75.92% <75.92%> (ø) |
|
...istry/controlplane/supportbundlecollection/rest.go | 79.48% <79.48%> (ø) |
|
...roller/supportbundlecollection/store/collection.go | 81.69% <81.69%> (ø) |
|
pkg/apis/controlplane/register.go | 85.18% <100.00%> (-6.49%) |
:arrow_down: |
pkg/apis/controlplane/v1beta2/conversion.go | 66.66% <100.00%> (ø) |
|
... and 114 more |
/test-all /test-ipv6-all /test-windows-all
Seems that we also need to update pkg/apis/controlplane/v1beta2/conversion.go, like other internal objects?
Thanks for catching it, updated.
/test-all /test-windows-all /test-multicluster-e2e
/test-all
LGTM, typo in commit message: Controlelr
updated.
/test-all
Update commit message only.
/test-all