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

Support re-config pod networks #316

Merged
merged 4 commits into from
Jan 27, 2022
Merged

Support re-config pod networks #316

merged 4 commits into from
Jan 27, 2022

Conversation

l1b0k
Copy link
Collaborator

@l1b0k l1b0k commented Jan 26, 2022

This PR include several improvement relate to terway-controlplane

@l1b0k l1b0k self-assigned this Jan 26, 2022
@l1b0k l1b0k added the kind/feature feature label Jan 26, 2022
@codecov-commenter
Copy link

codecov-commenter commented Jan 26, 2022

Codecov Report

Merging #316 (6930fae) into main (d78ed63) will increase coverage by 0.40%.
The diff coverage is 8.07%.

Impacted file tree graph

@@           Coverage Diff            @@
##            main    #316      +/-   ##
========================================
+ Coverage   4.12%   4.52%   +0.40%     
========================================
  Files        107     111       +4     
  Lines      10313   10561     +248     
========================================
+ Hits         425     478      +53     
- Misses      9854   10048     +194     
- Partials      34      35       +1     
Flag Coverage Δ
unittests 4.52% <8.07%> (+0.40%) ⬆️

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

Impacted Files Coverage Δ
cmd/terway-controlplane/terway-controlplane.go 4.47% <0.00%> (ø)
daemon/daemon.go 2.74% <0.00%> (ø)
daemon/eni-multi-ip.go 0.00% <ø> (ø)
daemon/k8s.go 0.00% <0.00%> (ø)
examples/maxpods/maxpods.go 19.35% <0.00%> (ø)
pkg/aliyun/aliyun.go 0.00% <0.00%> (ø)
pkg/aliyun/client/ecs.go 0.00% <0.00%> (ø)
pkg/aliyun/client/errors/errors.go 33.33% <ø> (ø)
pkg/aliyun/client/fake/fake_client.go 0.00% <0.00%> (ø)
pkg/aliyun/client/types.go 0.00% <0.00%> (ø)
... and 25 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d78ed63...6930fae. Read the comment docs.

@l1b0k l1b0k requested a review from BSWANG January 26, 2022 06:01
@l1b0k l1b0k merged commit cdd1a81 into AliyunContainerService:main Jan 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants