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

[release-1.28] More backports for 2024-06 release cycle #10289

Merged

Conversation

brandond
Copy link
Member

@brandond brandond commented Jun 4, 2024

github-actions bot and others added 3 commits June 4, 2024 18:50
* chore: Bump Local Path Provisioner version

Made with ❤️️ by updatecli

(cherry picked from commit 1268779)
Signed-off-by: Brad Davidson <brad.davidson@rancher.com>
Signed-off-by: Koen de Laat <koen.de.laat@philips.com>
(cherry picked from commit 79ba10f)
Signed-off-by: Brad Davidson <brad.davidson@rancher.com>
If proxy.SetAPIServerPort was called multiple times, all calls after the
first one would cause the apiserver address to be set to the default
server address, bypassing the local load-balancer. This was most likely
to occur on RKE2, where the supervisor may be up for a period of time
before it is ready to manage node password secrets, causing the agent
to retry.

Signed-off-by: Brad Davidson <brad.davidson@rancher.com>
(cherry picked from commit 1661f10)
Signed-off-by: Brad Davidson <brad.davidson@rancher.com>
@brandond brandond requested a review from a team as a code owner June 4, 2024 18:58
Copy link

codecov bot commented Jun 4, 2024

Codecov Report

Attention: Patch coverage is 20.83333% with 19 lines in your changes missing coverage. Please review.

Project coverage is 36.26%. Comparing base (a29d555) to head (fb8e15e).

Files Patch % Lines
pkg/agent/proxy/apiproxy.go 10.00% 9 Missing ⚠️
pkg/agent/config/config.go 11.11% 5 Missing and 3 partials ⚠️
pkg/agent/loadbalancer/loadbalancer.go 66.66% 1 Missing ⚠️
pkg/certmonitor/certmonitor.go 0.00% 1 Missing ⚠️
Additional details and impacted files
@@               Coverage Diff                @@
##           release-1.28   #10289      +/-   ##
================================================
- Coverage         42.68%   36.26%   -6.42%     
================================================
  Files               163      160       -3     
  Lines             16609    14051    -2558     
================================================
- Hits               7089     5096    -1993     
+ Misses             8260     7960     -300     
+ Partials           1260      995     -265     
Flag Coverage Δ
e2etests 36.26% <20.83%> (-9.64%) ⬇️
unittests ?

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

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

@brandond brandond merged commit 7de7adb into k3s-io:release-1.28 Jun 4, 2024
27 checks passed
@brandond brandond deleted the 2024-06-backports2_release-1.28 branch June 6, 2024 21:07
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.

3 participants