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.30] Backports for 2024-11 #11262

Merged

Conversation

Signed-off-by: Brad Davidson <[email protected]>
(cherry picked from commit e34fe03)
Signed-off-by: Brad Davidson <[email protected]>
* Use clientv3.NewCtxClient instead of New to avoid automatic retry of all RPCs
* Only timeout status requests; allow defrag and alarm clear requests to run to completion.
* Only clear alarms on the local cluster member, not ALL cluster members

Signed-off-by: Brad Davidson <[email protected]>
(cherry picked from commit 095e34d)
Signed-off-by: Brad Davidson <[email protected]>
Signed-off-by: Brad Davidson <[email protected]>
(cherry picked from commit a39e191)
Signed-off-by: Brad Davidson <[email protected]>
Signed-off-by: Brad Davidson <[email protected]>
(cherry picked from commit bc60ff7)
Signed-off-by: Brad Davidson <[email protected]>
Signed-off-by: Brad Davidson <[email protected]>
(cherry picked from commit b67249e)
Signed-off-by: Brad Davidson <[email protected]>
Add new flag that is passed through to the device_ownership_from_security_context parameter in the containerd CRI config. This is not possible to change without providing a complete custom containerd.toml template so we should add a flag for it.

Signed-off-by: Brad Davidson <[email protected]>
(cherry picked from commit 56fb3b0)
Signed-off-by: Brad Davidson <[email protected]>
@brandond brandond requested a review from a team as a code owner November 6, 2024 19:29
Signed-off-by: Brad Davidson <[email protected]>
(cherry picked from commit 917761c)
Signed-off-by: Brad Davidson <[email protected]>
External CLI actions cannot short-circuit on --help or --version, so we
cannot skip loading the config file if these flags are present when
running these wrapped commands. The behavior of just returning the
override flag name instead of the requested flag value was breaking
data-dir lookup when running wrapped commands.

Signed-off-by: Brad Davidson <[email protected]>
(cherry picked from commit ff5c633)
Signed-off-by: Brad Davidson <[email protected]>
Signed-off-by: Brad Davidson <[email protected]>
(cherry picked from commit 9c32f83)
Signed-off-by: Brad Davidson <[email protected]>
@brandond brandond force-pushed the 2024-11-backports_release-1.30 branch from e85b2f4 to 30ed6d0 Compare November 6, 2024 19:52
Copy link

codecov bot commented Nov 6, 2024

Codecov Report

Attention: Patch coverage is 63.81579% with 55 lines in your changes missing coverage. Please review.

Project coverage is 41.86%. Comparing base (ddea433) to head (30ed6d0).
Report is 9 commits behind head on release-1.30.

Files with missing lines Patch % Lines
pkg/etcd/resolver.go 45.83% 23 Missing and 3 partials ⚠️
pkg/etcd/etcd.go 70.12% 18 Missing and 5 partials ⚠️
pkg/configfilearg/defaultparser.go 72.22% 5 Missing ⚠️
pkg/etcd/snapshot.go 0.00% 1 Missing ⚠️
Additional details and impacted files
@@               Coverage Diff                @@
##           release-1.30   #11262      +/-   ##
================================================
- Coverage         46.23%   41.86%   -4.38%     
================================================
  Files               178      179       +1     
  Lines             18627    18716      +89     
================================================
- Hits               8613     7836     -777     
- Misses             8663     9682    +1019     
+ Partials           1351     1198     -153     
Flag Coverage Δ
e2etests 33.98% <2.70%> (-7.85%) ⬇️
inttests 34.74% <42.56%> (+0.10%) ⬆️
unittests 13.55% <59.86%> (+0.53%) ⬆️

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 185400d into k3s-io:release-1.30 Nov 6, 2024
30 checks passed
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.

2 participants