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 kube-router #8423

Merged
merged 1 commit into from
Sep 27, 2023
Merged

Update kube-router #8423

merged 1 commit into from
Sep 27, 2023

Conversation

rbrtbnfgl
Copy link
Contributor

@rbrtbnfgl rbrtbnfgl commented Sep 25, 2023

Proposed Changes

Types of Changes

Update Kube-router to fix #8321

Verification

Testing

Linked Issues

#8422

User-Facing Change

update kube-router to v2.0.0-rc7 to fix performance issues

Further Comments

@brandond
Copy link
Contributor

brandond commented Sep 25, 2023

@manuelbuil
Copy link
Contributor

Could you add something to the the User-Facing Change section? For example: update kube-router npc to v2.0.0-rc7 to fix performance issues

@codecov
Copy link

codecov bot commented Sep 25, 2023

Codecov Report

All modified lines are covered by tests ✅

Comparison is base (3194dc7) 44.73% compared to head (f86d460) 51.52%.
Report is 11 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #8423      +/-   ##
==========================================
+ Coverage   44.73%   51.52%   +6.78%     
==========================================
  Files         140      143       +3     
  Lines       14679    15229     +550     
==========================================
+ Hits         6566     7846    +1280     
+ Misses       7006     6168     -838     
- Partials     1107     1215     +108     
Flag Coverage Δ
e2etests 48.70% <ø> (?)
inttests 44.73% <ø> (+<0.01%) ⬆️
unittests 19.84% <ø> (?)

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

see 58 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@rbrtbnfgl
Copy link
Contributor Author

From the diff I probably did some mistake when I rebased. I'll try to fix it

@brandond
Copy link
Contributor

looks kinda like you merged upstream into your branch, instead of rebasing on top of upstream.

@rbrtbnfgl
Copy link
Contributor Author

No I rebased them but some conflicted commits were wrongly edited.

Signed-off-by: Roberto Bonafiglia <roberto.bonafiglia@suse.com>
@brandond
Copy link
Contributor

lgtm!

@rbrtbnfgl rbrtbnfgl merged commit ad20631 into k3s-io:master Sep 27, 2023
19 checks passed
@rbrtbnfgl rbrtbnfgl deleted the kube-router-updater4 branch September 27, 2023 08:41
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.

None yet

5 participants